Pull to refresh
54
0
Aлексей Токарь @AlexeyTokar

Пользователь

Send message
хабраэффект — миф ;)
секунды не хватило? :(
а. уже ответили. просто картинка сильно привлекла внимание :)
а еще можно просто выбрать нужную группу в левом меню…
кстати отличная идея — строить солнечные электростанции в санитарной зоне АЭС
<irony>
подогревать голубей? О_о
</irony>
хабраэффекта нет.
сейчас активность повысилась в два раза и составляет ~8 хитов/сек
да, дейтсивтельно — токаря обвинять не в чем...
нет, но само решение можно было свести к «Объекту запроса», которое является более гибким, чем приведенное
_or_404 уже лучше, но опять же смешиваеются слои приложения. Но это дело вкуса, конечно…
да просто потом прийдется объяснять, почему метод называется именно так и почему в одних случаях случается магия (отображается 404), а в других — обычный поток выполнения
выбрасывать исключения каждый раз, когда нет записи в БД — не кошерно. Иногда нужен действительно пустой массив, если нет данных.
Кстати метод

public function fetchRowIfExists($message = 'Данной страницы нет на сайте')

ужасен. Ужасен по нескольким причинам:
1) метод в зависимости от наличия строки возвращает разные типы данных (массив или строку ошибки)
2) логика обработки отсуствия строки в БД перенесена из контроллера в модель

По хорошему достаточно метода fetchRow(), который возвращает либо запись из БД, либо пустой массив (объект?), которй уже в контроллере проверяется на пустоту и выбрасывается соответствующее исключение, которое обработается центральным обработчиком исключений и покажет 404, либо вручную перенаправит на 404
Фаулер. Объект запросов
можно купить книгу по шаблонам GoF
а еще я выбрал «показать все банкоматы Киева на карте» и моментально об этом пожалел… И это при том, что есть не все банки

Information

Rating
Does not participate
Location
Киев, Киевская обл., Украина
Date of birth
Registered
Activity