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]

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?

[alex011251]

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

Ten post edytował alex011251 25.05.2013, 18:41:16

[Dipter]

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.

[alex011251]

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 

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

[alex011251]

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

[Dipter]

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.

[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

[CuteOne]

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

[alex011251]

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?

[CuteOne]

[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

Ten post edytował CuteOne 25.05.2013, 22:41:30

[alex011251]

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 

Ten post edytował alex011251 25.05.2013, 22:51:25

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