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 (IMG:style_emoticons/default/tongue.gif) P Bo ucze sie dopiero

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

[Dipter]

Nie wiem dlaczego konstruktor robi aż tyle u Ciebie, przekaż w nim zaledwie adres obrazka, następnie stwórz metodę, w której odpalisz całego cURL, następnie przetworzysz informację, a dopiero później sprawdzaj czy obrazek jest faktycznie obrazkiem.

W chwili obecnej zrobiłeś opakowanie przy pomocy klasy na coś, co mogłoby spokojnie być zwykłą funkcją.