[PHP] debugowanie kodu.. Opcje

[-gox-]

Witam, oto moj pierwszy skrypt w OO php5 i mam sporo problemow z jego zdebugowaniem:

[PHP] pobierz, plaintext 
<?php
 
	$t = new telnet('host', '21');
	$t->loop();
 
class telnet {
 
public $fp;
 
public __construct($host, $port) {
	if (!empty($host) or !empty($port)) trigger_error('telnet __construct - brak hosta/portu', E_USER_ERROR);
	if (!preg_match('#^[0-255]{1,3}.[0-255]{1,3}.[0-255]{1,3}.[0-255]{1,3}$#', $host)) {
		$ip = gethostbyname($host);
		if ($ip != $host) $host = $ip;
		else trigger_error("telnet __construct - Bledny host - '$host'", E_USER_ERROR);
	}
	if ($port < 0 or $port > 65536)	trigger_error("telnet __construct - Nieprawidlowy port - '$port'", E_USER_ERROR);
	if ( (self::$fp = fsockopen($host, $port)) == false ) trigger_error("telnet __construct - Blad polaczenia!", E_USER_ERROR);
}
 
public __destruct() {
	if (@fclose(self::$fp))
		return true;
	else return false;
}
 
public function loop() {
	$out = '';
	while(self::$fp) {
		$dane = '';
		if ($dane = self::input('o.txt') != false) {
			fwrite(self::$fp, "$daner");
			while(!feof(self::$fp))$out.= fread(self::$fp, 1024);
		}
		self::ajax($out);
		//sleep(1);
}
	self::close();
	return true;
}
 
private function ajax ($dane) {
	$dane = str_replace(''', ''', $dane);
	echo '<script type="text/javascript">
$('response').innerHTML = '' . $dane . '';
</script>';
	return true;
}
 
private function input ($plik) {
	if ($fp = fopen($plik, 'r') == false)
		return false;
	while(!feof($fp)) $fget.= fgets($fp);
	fclose($fp);
	unlink($plik);
	return $fget;
}
 
}
?>
[PHP] pobierz, plaintext 

Jako pierwszy jest blad: Parse error: syntax error, unexpected T_STRING, expecting T_VARIABLE on line 174

o co mu chodzi?

[-gox-]

linia 174 to ta : if (!preg_match('#^[0-255]{1,3}\.[0-255]{1,3}\.[0-255]{1,3}\.[0-255]{1,3}\$#', $host)) {

[mike]

Stos błędów.

1. Brak słowa kluczowego function dla __construct() i __destruct()
2. Pomylone zdanie logiczne w warunku:

[PHP] pobierz, plaintext 
<?php
//...
if (!empty($host) or !empty($port))
//...
?>
[PHP] pobierz, plaintext 

Powinno być:

[PHP] pobierz, plaintext 
<?php
//...
if (empty($host) || empty($port))
//...
?>
[PHP] pobierz, plaintext 

3. Następnie:

Fatal error: Access to undeclared static property: telnet::$fp

Składową klasy $fp zadeklarowałeś jako zmienną obiektu więc nie możesz odwoływać się do niej statycznie. Powinieneś skorzystać z $this a nie self:

[PHP] pobierz, plaintext 
<?php
//...
if (($this->fp = fsockopen($host, $port)) == false)
//...
?>
[PHP] pobierz, plaintext 

Zmiana dotyczy każdego wystąpienia $fp.
4. Złe odwołania do składowych obiektu. Podobnie jak źle odwołujesz się do fp tak samo źle odwołujesz się do ajax() i ii]close()[/i]. Wszędzie powinno być $this->ajax() i $this->close()
5. Brak w klasie funkcji close() wiec powyższe odwołania i tak nie zadziałają.
6. Wychwytywanie sytuacji wyjątkowych zostaw ... wyjątkom. Niepotrzebnie bawisz się w trigger_error stosowanie wyjątków (Exception) jest dużo wygodniejsze i niesie dużo więcej możliwości. Lepiej tak:

[PHP] pobierz, plaintext 
<?php
// ...
if (empty($host) || empty($port)) {
	throw new Exception(Brak hosta/portu);
}
//...
?>
[PHP] pobierz, plaintext 

7. Linijka:

[PHP] pobierz, plaintext 
<?php
if ($fp = fopen($plik, 'r') == false)
?>
[PHP] pobierz, plaintext 

Jest błędna. Powinno być:

[PHP] pobierz, plaintext 
<?php
if (($fp = fopen($plik, 'r')) == false)
?>
[PHP] pobierz, plaintext 

8. Złe nawyki i standardy kodowania. Wszystkie bloki kodu ujmujemy w klamry. Pisanie nawet jednowierszowych bloków dla instrukcji if, if ... else, ... jest złe.

Więcej punktować mi się już nie chce. Wystarcz Ci.

A co do błędu, który podałeś to u mnie nie występuje.

To jeszcze tylko dorzucę trzy rzeczy.

9. Funkcja input() jest totalnie niepotrzebna. Słyszałeś kiedyś o file_get_contents() (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)
10. Skrypt wali błędami na potęgę jeśli plik o.txt nie istnieje.
11. Nazwę pliku o.txt powinieneś wyrzucić do zmiennej klasy i dać możliwość ustawienia swojej nazwy.