[php] operacje na stringach, pliki Opcje

[neoangin]

Witam,

[PHP] pobierz, plaintext 
class zamien
{
	function zam($path, $co, $naco)
	{
		$handle = fopen($this->path, "r+");
		$contents = fread($handle, filesize($this->path));
		$nowy = str_replace($this->co, $this->naco, $contents);
		rewind($handle);
		fwrite($handle, $nowy);
		fclose($handle);
	}
}
 
$zamm = new zamien();
$pierwszy = $zamm->zam("text.txt", "a", "A");
[PHP] pobierz, plaintext 

Wyrzuca takie bledy:

Notice: Undefined property: zamien::$path in C:\wamp\www\php_obiektowy\czytanie_i_zamiana.php on line 7

Warning: fopen() [function.fopen]: Filename cannot be empty in C:\wamp\www\php_obiektowy\czytanie_i_zamiana.php on line 7

Notice: Undefined property: zamien::$path in C:\wamp\www\php_obiektowy\czytanie_i_zamiana.php on line 8

Warning: fread() expects parameter 1 to be resource, boolean given in C:\wamp\www\php_obiektowy\czytanie_i_zamiana.php on line 8

Notice: Undefined property: zamien::$co in C:\wamp\www\php_obiektowy\czytanie_i_zamiana.php on line 9

Notice: Undefined property: zamien::$naco in C:\wamp\www\php_obiektowy\czytanie_i_zamiana.php on line 9

Warning: rewind() expects parameter 1 to be resource, boolean given in C:\wamp\www\php_obiektowy\czytanie_i_zamiana.php on line 10

Warning: fwrite() expects parameter 1 to be resource, boolean given in C:\wamp\www\php_obiektowy\czytanie_i_zamiana.php on line 11

Warning: fclose() expects parameter 1 to be resource, boolean given in C:\wamp\www\php_obiektowy\czytanie_i_zamiana.php on line 12

A jak dam to nie w klasie:

[PHP] pobierz, plaintext 
	$path = "text.txt";
	$handle = fopen($path, "r+");
 
	$contents = fread($handle, filesize($path));
	echo $contents;
	echo "<br />";
	echo "Po:<br />";
	$nowy = str_replace("a","A",$contents);
	echo $nowy;
	rewind($handle);
	fwrite($handle,$nowy);
	fclose($handle);
 
[PHP] pobierz, plaintext 

To jest okej wszystko i skrypt sie wykonuje - czy ktoś coś zaradzi?

[wookieb]

Komunikaty są dość jasne. Jak nie umiesz angielskiego to przepuść je przez translator i się do nich zastosuj.

[Berg]

Nie wiem gdzie czytałeś o obiektówce ale zdecydowanie nie doczytałeś. Do argumentów metody nie odwołujesz się przez $this tylko jak do normalnej zmiennej... Czyli zamiast np. $this->path powinno być $path.

Tak na marginesie to po co upychasz jedną metodę w klasę? Jeśli już koniecznie chcesz stosować OOP to ta metoda powinna być raczej statyczna.

Ten post edytował Berg 2.09.2010, 15:05:59

[zordon]

wszystko masz napisane jak na dłoni.
ten kod to przerabianie na siłę kodu proceduralnego na obiektowy lub odwrotnie. Albo zrezygnuj z klasy i posługuj się parametrami, które przekazujesz do funkcji, albo przekaż te parametry do konstruktora klasy, przypisz do właściwości i dopiero operuj na właściwościach.