Często zdarza mi się korzystać z Curl'a, jak wiadomo jest to średnio przyjemne dlatego postanowiłem na szybko opakować go na swój sposób, moim zdaniem pracuje się całkiem przyjemnie jednak pytanie czy takie podejście dobre?

Zamysł był taki, aby było prościej i szybciej, czy się udało?

[PHP] curl.php - pobierz, plaintext 
class curl_exception extends Exception {}
 
class curl {
 
	private $handle;
	private $url='';
	private $path=''; // cookie path
	private $userAgent='';
 
	public function __construct($url='') {
		if (!function_exists('curl_init')) {
			throw new curl_exception('Curl jest wyłączony');
		}
 
		$this->handle = curl_init();
 
		if (!empty($url)) {
			$this->setUrl($url);
		}
	}
 
	public function setUrl($url) {
		$this->url = $url;
		$this->setOpt(CURLOPT_URL, $url);
		return $this;
	}
 
	public function getUrl() {
		return $this->url;
	}
 
	public function setCookiePath($path) {
		if (!opendir($path)) {
			throw new curl_exception('Brak dostępu do ścieżki');
		}
		$this->path = $path;
 
		$this->setOpt(CURLOPT_COOKIEJAR, $path.'/cookie.txt')->setOpt(CURLOPT_COOKIEFILE, $path.'/cookie.txt');
 
		return $this;
	}
 
	public function getCookiePath() {
		return $this->path;
	}
 
	public function setOpt($option, $value) {
		curl_setopt($this->handle, $option, $value);
		return $this;
	}
 
	public function setPost($string) {
		$this->setOpt(CURLOPT_POST, true)->setOpt(CURLOPT_POSTFIELDS, $string);
		return $this;
	}
 
	public function setUserAgent($agent) {
		$this->userAgent = $agent;
		$this->setOpt(CURLOPT_USERAGENT, $agent);
		return $this;
	}
 
	public function getUserAgent() {
		return $this->userAgent;
	}
 
	public function exec() {
		if (!empty($this->url)) {
			$exec = curl_exec($this->handle);
			if ($this->getErrno()) {
				throw new curl_exception($this->getError());
			}else{
				return $exec;
			}
		}else{
			throw new curl_exception('Url jest wymagany!');
		}
	}
 
	private function getError() {
		return curl_error($this->handle);
	}
 
	private function getErrno() {
		return curl_errno($this->handle);
	}
 
	public function getHandle() {
		return $this->handle;
	}
 
}
 
class curl_multi {
 
	private $handle;
	private $handles = array();
 
	public function __construct() {
		if (!function_exists('curl_multi_init')) {
			throw new curl_exception('Curl jest wyłączony');
		}
		$this->handle = curl_multi_init();
	}
 
	public function addHandle(curl $obj) {
		curl_multi_add_handle($this->handle, $obj->getHandle());
		$this->handles[] = $obj;
		return $this;
	}
 
	public function clean() {
		foreach($this->handles as $obj) {
			curl_multi_remove_handle($this->handle, $obj->getHandle());
		}
		return $this;
	}
 
	public function exec() {
		do {
			$status = curl_multi_exec($this->handle, $running);
		}while($running > 0);
 
		return $this;
	}
 
	public function getContent() {
		$content = array();
 
		foreach ($this->handles as $obj) {
			$content[$obj->getUrl()] = curl_multi_getcontent($obj->getHandle());
		}
 
		return $content;
	}
 
 
}	
[PHP] curl.php - pobierz, plaintext 

curl_post -> tablica + http_build_query
curl i curl_multi to na dobrą sprawę nakładka 1:1 na funkcję curl_*() nie ułatwiająca niczego, a jedynie ograniczająca.

Do tego "pierdoły" typu niepoprawne nazewnictwo czy złe użycie wyjątków.

A o tym nie wiedziałem tak szczerze.

Gdzie ogranicza? jest set uniwersalny i set'y do najczęściej używanych (przeze mnie) url, cookie, post

Hmm, jeżeli klasa nie może pracować z powodu braku uprawnień czy wyłączonego curl'a to uznałem że powinna wyrzucać wyjątek.

A racja, nie zauważyłem.
To może bardziej rozwinę:
- Zła klasa curl_exception. Zła bo klasa wyjątku ma symbolizować przyczynę jego wystąpienia nie miejsce jego wyrzucenia. W dodatku dziedziczysz po złej klasie (Exception). Powinieneś skorzystać z wyjątków z SPL-a. Nie ma ich zbyt wielu i nie pokrywają one wszystkich zastosowań, niemniej jednak zawsze powinieneś dziedziczyć po jak najbardziej wyspecjalizowanej klasie, czyli co najwyżej po LogicExcepton lub RuntimeException.
- Metody typu curl::exec() powinny same sprawdzić czy są odpowiednio skonfigurowane oraz czy nie wystąpił błąd po ich odpaleniu i w razie potrzeby wywalić wyjątek. Wyjątki mają być automatycznie wyrzucane, a metod typu getError() nie powinno nawet być.

Szczerze mówiąc nic to nie zmieni, prócz tego, że będę po czymś innym dziedziczył - chyba, że czegoś nie doczytałem.

Fakt, można tutaj dodać sprawdzanie czy url został wprowadzony bo to jedyny wymóg.

getError, getErrno mogę zmienić na prywatne, po odpaleniu exec odpalać je i sprawdzać ew. błędy i wyrzucić wyjątek - czy taki pomysł jest do zaakceptowania?

[offtopic]
Liczyłem właśnie na Twoją krytykę
[/offtopic]

@edit
Wywaliłem curl_post, usunąłem private $post,
Metody: getErrno(), getError() zostały zmienione na prywatne
W setPost drobna zmiana kosmetyczna
Exec według Twoich uwag.

(edytuję pierwszego posta z kodem)



@edit2

Dodane metody:
getCookiePath()
setUserAgent()
getUserAgent()

Edytowane:
setCookiePath() - nie zmieniał ścieżki, teraz to robi.

Popraw metodę exec w curl_multi - aktualna zajeżdża procesor.

Dołóż w środku curl_multi_select albo zrób dokładnie tak jak w manualu do curl_multi_exec

Masz na myśli ten skrawek kodu:

[PHP] pobierz, plaintext 
while ($active && $mrc == CURLM_OK) {
    if (curl_multi_select($mh) != -1) {
        do {
            $mrc = curl_multi_exec($mh, $active);
        } while ($mrc == CURLM_CALL_MULTI_PERFORM);
    }
}
[PHP] pobierz, plaintext 

?

Chociażby. Chodzi o to, żeby wywoływać to curl_multi_select(), bo bez tego ta pętla będzie się wykonywać non-stop zajeżdżając procesor serwera.

Chodzi o poprawne informowanie użytkownika klasy co poszło nie tak, czy zawiódł kod / programista / serwer itp Bardzo łądnie opisane ma to na swoim blogu Zyx.
Rzucając curl_exception - wiem tyle co nic, ot klasa zawiodła, ale czemu? Ok - wiadomość ale to też mi nie daje możliwości zareagowania na odpowiednie błędy, załóżmy sytuację że dostaję curl_exception z Twojej klasy
aby zareagować na odpowiednie błędy muszę sprawdzać wartość message - a tak by wyglądał ocoś w stylu:

[PHP] pobierz, plaintext 
try
{
//operacje curl
}
catch( Exception_Curl_Missing $e)
{
// ok nie ma curl próbujemy z fsockopen
}
catch( Exception_Curl_UrlInvalid $e) //dziedziczy po UnexpectedValueException
{
// ok nie porpaweny url tutaj w zależności czy wymuszamy walidację na osobie obsługującą klasę czy na samą klasę zmieniamy typ wyjątku po którym dziedziczymy
// z UnexpectedValue_Exception na InvalidArgumentException
}
[PHP] pobierz, plaintext 

Bo na razie bardziej te wyjątki służą Tobie jako walidator i provider dla komunikatów błędów niż po coś innego.

Ogólnie bardziej tą klasę widziałbym jako adapter, dla bardziej ogólnej klasy wtedy byś mógł z łatwością wymieniać interejfsy gdy np. nie ma curl używasz fsockopen itp.
Do tego ogólny interfejs by w ogóle użyteczność tej klasy miała jakikolwiek sens, bo na razie jest przydatna Tobie i tylko w momencie używania CURL'a, w momencie wystapienia wyjątku braku curl'a nie mam możliwość podmiany adapter'a i próby ponownego wywołania tylko tworzenie od nowa kolejnej klasy lub kompletne zignorowanie wyjątku co jest ogólnie pomyłką.

Ogólnie z chęcią zobaczę co inni powiedzą na temat mojego myślenia, ponieważ zazwyczaj hamuję się z wypowiadaniem na takie tematy ponieważ mam wrażenie że wciąż w tych klockach jestem za `cienki`, ale jak lepiej się uczyć niż na swoich błędach - chociaż brzmi to nawet logicznie :]