- 1
- 2
- 3
- 4
if( !$this->dirsmodel->isExist(array('alias' => $_POST['alias']))) {
$this->messages[] = 'Такой алиас уже существует';
return $this->redirect("admin/dirs/#" . $cat_id);
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+148
if( !$this->dirsmodel->isExist(array('alias' => $_POST['alias']))) {
$this->messages[] = 'Такой алиас уже существует';
return $this->redirect("admin/dirs/#" . $cat_id);
}
Говнокод тонкий особый. Кто найдёт получит пиченьку
u1tr0n 11.09.2010 13:55 # +2
DrFreez 11.09.2010 14:27 # 0
Wave 11.09.2010 16:18 # 0
u1tr0n 11.09.2010 17:51 # +2
З.Ы. если ты покушаешься на мою пиченьку, то я ее уже съел ;)
Lure Of Chaos 12.09.2010 01:24 # 0
1. isExist() можно было бы просто exists()
2. что оно требует передачи именно массива - хотя, может быть именно тут так и надо, а обычно достаточно проверки только на ключ
bildja 12.09.2010 01:31 # 0
2. Не, тут же не на ключ, а на всю пару
Lure Of Chaos 12.09.2010 01:36 # 0
2. ну по аналогии с картами(хэшами) - что бы проверить, существует ли пара ключ-значение, достаточно проверить существование ключа. Конечно же, тут isExist может быть универсальнее, всмысле 'alias' => проверяет сущестование именно такого аляса, а может проверять еще чье-то.....
bildja 12.09.2010 01:39 # 0
2. Вот-вот, скорей всего там эта универсальность, так как это ж не аляса похоже метод
Lure Of Chaos 12.09.2010 01:40 # 0
bildja 12.09.2010 01:42 # 0
Анонимус 12.09.2010 03:26 # 0
Если метод не удовлетворяет биновыми требованиям (тоесть не get и не is), то до него может быть недобраца через expressions language в jsp и всяких других шаблонах, если не начинается с set -- его не засетит какой-нить там струтс итд..
Короче соглашения лучше соблюдать, даже если получается кошмар типа isExists.
В этом плане даже проперти си шарпа выглядят на такими уж убогими
Lure Of Chaos 12.09.2010 10:09 # 0
Lure Of Chaos 12.09.2010 01:49 # 0
я требую продолжения банкета! (ц)
bugmenot 12.09.2010 15:46 # −1
Lure Of Chaos 12.09.2010 17:13 # +2
оффтоп: ой, а у вас кто на авке?
bugmenot 12.09.2010 19:06 # −1
3.14159265 12.09.2010 20:37 # +1
просто немногие с говнокода ходят на чаны.
bugmenot 12.09.2010 21:21 # −1
да ну, старый мем, уже в интервебах должно быть узнаваемо :)
ZX_Spectrum 12.09.2010 23:36 # +2
bugmenot 12.09.2010 23:59 # −1
inkanus-gray 13.09.2010 00:13 # +2
http://images.encyclopediadramatica.com/images/thumb/f/f2/Trollface_More_HD.png/657px-Trollface_More_HD.png
http://gravatar.com/avatar/b94d9cb39fdb23c4d78ba35b35ba9c4e?size=51 2
Одно лицо! Не узнал, богатым будете! Новая причёска?
inkanus-gray 13.09.2010 01:55 # +1
http://tinyurl.com/tro11face
(в URL две единицы)