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

[alex011251]

[PHP] pobierz, plaintext 
<?php
class securityBasicAuth {
	public $zmienna = '';
	public function __construct($url) {
		$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);
	}
		public function warunek() {
		if ($httpCode == 200) {
			$this -> zmienna = TRUE;
		} else {
			$this -> zmienna = FALSE;
		}
		}
 
function __descruct() {
 	return $this -> zmienna;
		curl_close();
	}
}
 
$link = 'http://c.allegrostatic.pl/images/velalayout/434de666.allegro-logo.png';
$neg ='https://upload.wikimedia.org/wikipedia/commons/thumb/3/3e/Nie_istnieje.svg/100px-Nie_istnieje.svg.png';
$security = new securityBasicAuth($link);
$wynik =  $security -> __descruct();
 
if($wynik){
	echo '<img src="'.$link.'" />';
}else{
	echo '<img src= "'.$neg.'"/>';
 
}
?>
[PHP] pobierz, plaintext 

Ale nie dziala