Еще один забавный случай или:
public function getCurlException()
{
if (curl_errno($this->handle)) {
throw new FooBar_exception(
'Ошибка curl: ' . curl_error($this->handle)
);
}
else return true;
}Эмм.. Что?
Если курл завершил запрос с ошибкой, тогда бросаем исключение.
Если все гут, тогда возвращает тру.
Представляю себе это как-то так:
try {
if ($fooBar->getCurlException()) {
// ой, ничего не случилось
} else {
// дабл-ой, нас тут уже нет
}
} catch (FooBar_exception $e) {
// ...
}Так зачем там нужен ретурн тру?
Пойду грепну где это дергается...
А, ну точно, копипаста, каждые 20-40 строк в каждом методе повторяется:
$options = array(
// везде почти одинаковые параметры
);
$this->connection->openCurlRequest();
// метод, судя по названию, должен ставить одну опцию, а ставит несколько
$this->connection->setOption($options);
$html = $this->connection->execute();
$this->connection->getCurlException(); // о, вот она!Ах, да:
ЛОРовцы, родные!
Ни за что на свете
Не ходите в офис,
ПоХаПе писать!
В ПоХаПе "собаки" (@),
В ПоХаПе нотайсы,
В ПоХаПе большие
Злые копипасты
Будут вас кусать,
Бить и обижать,-
Не ходи ты, ЛОРовец,
ПоХаПе писать.
