Czy ta klasa jest dobra? Opcje

[alex011251]

[PHP] pobierz, plaintext 
<?php
class classSecurityBasicAuth {
	public $zmienna = '';
	public function __construct($url, $komunikat) {
		$handle = curl_init($url);
		curl_setopt($handle, CURLOPT_RETURNTRANSFER, true);
		curl_setopt($handle, CURLOPT_NOBODY, true);
		curl_exec($handle);
 
		$httpCode = curl_getinfo($handle, CURLINFO_HTTP_CODE);
		if ($httpCode == 200) {
			$this -> zmienna = '<img src="'.$url.'" />';
		} else {
			$this -> zmienna = $komunikat;
		}
		curl_close();
	}
	public function funkcja() {
	return $this->zmienna;	
	}
}
 
 
 
 
$security = new classSecurityBasicAuth('http://c.allegrostatic.pl/images/velalayout/434de666.allegro-logo.png', '<img src= "https://upload.wikimedia.org/wikipedia/commons/thumb/3/3e/Nie_istnieje.svg/100px-Nie_istnieje.svg.png"/>');
echo $security->funkcja();
?>
[PHP] pobierz, plaintext 

Uprzedzam. Chodzi czy dobrze konstruuje P Bo ucze sie dopiero

Ten post edytował alex011251 25.05.2013, 17:39:28

[Dipter]

Do każdej metody/właściwości zawartej w tej klasie (lub też tej, po której dziedziczysz, choć nie zawsze) odwołujesz się poprzez $this, to logiczne. Ponadto korzystaj z widoczności w klasach private/protected/public, bo samo "function" nie jest poprawne.

Tak jak napisał CuteOne, przeczytaj kilka poradników o OOP, oraz poznaj zasady takie jak SRP, KISS, DRY i wiele innych, bo w chwili obecnej szukasz problemów na siłę i próbujesz stworzyć coś co nie do końca tak powinno wyglądać.

Przeczytaj też o metodach magicznych, bo nie ma czegoś takiego jak __descruct, w dodatku nie wywołuje się tej metody z zewnątrz.