[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

1. Co konkretnie chcesz osiągnąć, jakie zadanie ma pełnić ta klasa?
2. Jakie odniesienie ma nazwa klasy do tego co jest w środku?

Klasa blokuje dodawanie "lewych" linkow. BaiscAuth - nazwa komunikatu gdy grafika zabezpieczona . Wiec nazwa adekwatna.

Jak ktos skorzyst z tagu img na forach to to ma sprawdzac czy nie jest lewa grafika. Badz jakis url czy cos jezeli korzysta z innego tagu typu flash itp

Obiekt musze bardziej przystosowac by spelnial funkcje. To jest fakt

Mimo wszystko nadal nie przekonuje mnie nazwa tej klasy, zwłaszcza przedrostek "class".

Co do samego kodu:

[PHP] pobierz, plaintext 
$httpCode = curl_getinfo($handle, CURLINFO_HTTP_CODE);
 
if ($httpCode == 200) {
	$this -> zmienna = '<img src="'.$url.'" />';
} else {
	$this -> zmienna = $komunikat;
}
[PHP] pobierz, plaintext 

Właściwie to nic nie powinieneś opakowywać w HTML i ogólny zarys wyświetlania danego komunikatu/błędu. Klasa raczej powinna sprawdzać czy faktycznie link jest obrazkiem i zwracać true/false - do tego oczywiście odpowiednia metoda. Dlatego też na zewnątrz powinieneś się tym wynikiem posłużyć i wtedy wyświetlić komunikat, przekierować czy cokolwiek innego.

[PHP] pobierz, plaintext 
public function funkcja() {
	return $this->zmienna;	
}
[PHP] pobierz, plaintext 

Niestety nie mówi to zbyt wiele, jedynie po zapisie można stwierdzić, że "zmienna" zostanie zwrócona. Mała rada: nie mieszaj angielskiego z polskim i jeszcze jakim językiem co by Ci do głowy przyszedł. Korzystaj z jednego (z reguły angielski). Do tego precyzuj nazwy metod, klas, zmiennych tak by było zrozumiałe co konkretnie dana rzecz robi.

na nazwy poki co nie patrz.

[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);
		if ($httpCode == 200) {
			$this -> zmienna = TRUE;
		} else {
			$this -> zmienna = FALSE;
		}
		curl_close();
	}
 
	public function open() {
		return $this -> zmienna;
	}
 
}
 
$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 -> open();
 
if($wynik){
	echo '<img src="'.$link.'" />';
}else{
	echo '<img src= "'.$neg.'"/>';
 
}
?>
[PHP] pobierz, plaintext 

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ą.

czyli
construktor
jakas metoda - z curlem
i sprawdzanie w kolejnej metodzie czy jest czy nie ma.


?

OOP myslalem ze bedzie latwiejsze. A dluga droga przedemna

No tak to mniej więcej powinno wyglądać. Każda metoda powinna mieć jakieś zajęcie (czyt. coś wykonuje/zwraca/ustawia/modyfikuje etc.), lecz musi to być konkretne zadanie, nie moż robić wszystkiego.

[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

Zmienne to nie właściwości klasy (zapomniałeś o $this)

Dosyc juz tego mam. Nie na moje nerwy

Mogl bys wskazac w ktorym miejscu?

[PHP] pobierz, plaintext 
       function sprawdz($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);
		if ($httpCode == 200) {
			$zmienna = TRUE;
		} else {
			$zmienna = FALSE;
		}
		curl_close();
 
        return $zmienna;
	}
 
 
        $sprawdz = sprawdz('http://blablabla.pl/');
 
[PHP] pobierz, plaintext 

Dobra rada... narazie pisz strukturalnie a w międzyczasie poczytaj parę poradników o OOP, zobacz jak to wygląda w popularnych frameworkach (CI, Zend, Kohana) i wróć do pisania gdy uznasz, że poznałeś podstawy

Strulturalnie opanowane dlatego chcialem przejsc do OOP . Metoda prob i bledow.
Ale czasem po prostu nie rozumiem . Jezeli cos jest publiczne to powinno wspolgrac. A nie jak ze usune oddzielajace public function

Jak wykonam tak to działa

Ale jak chce oddzielic jak wyzej to juz nie

[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);
 
 
		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 

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.