Какой из вариантов лучше и почему?

Слава Шевцов · 2009-04-21T20:27:39.0000000Z

Есть две функции работы с Курлом. Взяты из боевых скриптов. В обеих идут проверки инициализации. Функции разные, но суть одинаковая - нужно инициировать Курл. Автор у них один. Какой вариант лучше с точки зрения синтаксиса, удобства чтения, поддержки, отлова ошибок и почему? function my_download($url,$post){ $ret = false; if( function_exists('curl_init') ){ if( $curl = curl_init() ){ if( !curl_setopt($curl,CURLOPT_URL,$url) ) return $ret; if( !curl_setopt($curl,CURLOPT_RETURNTRANSFER,true) ) return $ret; if( !curl_setopt($curl,CURLOPT_CONNECTTIMEOUT,30) ) return $ret; if( !curl_setopt($curl,CURLOPT_USERAGENT,'yandex-positions (http://www.samborsky.com/yandex-positions/)') ) return $ret; if( !curl_setopt($curl,CURLOPT_HEADER,false) ) return $ret; if( !curl_setopt($curl,CURLOPT_POST,true) ) return $ret; if( !curl_setopt($curl,CURLOPT_POSTFIELDS,$post) ) return $ret; if( !curl_setopt($curl,CURLOPT_ENCODING,"gzip,deflate") ) return $ret; $ret = curl_exec($curl); curl_close($curl); } } } function get_download($url){ $ret = false; if( $curl = curl_init() ){ if( curl_setopt($curl,CURLOPT_URL,$url) ){ if( curl_setopt($curl,CURLOPT_RETURNTRANSFER,true) ){ if( curl_setopt($curl,CURLOPT_CONNECTTIMEOUT,60) ){ if( curl_setopt($curl,CURLOPT_HEADER,false) ){ if( curl_setopt($curl,CURLOPT_ENCODING,"gzip,deflate") ){ if( curl_setopt($curl,CURLOPT_FOLLOWLOCATION,true) ){ $ret = curl_exec($curl); } } } } } } curl_close($curl); } return $ret; }

255

malls

21 апреля 2009, 21:22

#21

присоединяюсь к мнению что хороших вариантов там просто нет. параноидальный говнокод.

386

dkameleon

21 апреля 2009, 22:04

#22

Слава Шевцов:
В $a оказывалась какая-нибудь фигня.

Это значит надо проверить оперативку и срочно заменить.

Так как ФАТ да и файлы часто тоже через оперативку гоняются.

А парить мозг всякой мистикой не нужно.

Дизайн интерьера (http://balabukha.com/)

255

malls

21 апреля 2009, 22:09

#23

dkameleon:
Это значит надо проверить оперативку и срочно заменить.

.... "прокладку" между оперативкой и клавой!

dkameleon:
А парить мозг всякой мистикой не нужно.

вот это верно в этом деле как известно чудес не бывает - бывают только баги!

[Удален]

21 апреля 2009, 22:32

#24

Ох как всё запущено...

http://phpfaq.ru/debug3

(dkameleon и Апокалипсис правы короче)

370

Слава Шевцов

22 апреля 2009, 06:04

#25

Зингельшухер, спасибо за ссылку. Не знал, что такое уже сделали в php. dkameleon, Апокалипсис - вы правы.

Неизменность точки зрения неизменно порождает иллюзию понимания.

VK приобрела 70% в структуре компании-разработчика red_mad_robot

Тренды маркетинга в 2024 году: мобильные продажи, углубленная аналитика и ИИ