- 1
- 2
- 3
- 4
- 5
- 6
- 7
if($user_key !== '') {
....
} elseif ($user_key == '') {
.....
} else {
exit('error');
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+163
if($user_key !== '') {
....
} elseif ($user_key == '') {
.....
} else {
exit('error');
}
Еще один шедевр
wmmorgun 09.12.2010 11:38 # +4
revis0r 09.12.2010 11:44 # +5
Oleg_quadro 09.12.2010 12:30 # 0
http://govnokod.ru/4058#comment43817
TarasB 09.12.2010 13:27 # −6
revis0r 09.12.2010 13:57 # 0
wmmorgun 09.12.2010 13:58 # 0
Плюс там однозначно до exit дело не дойдет, ибо делается строгое неравенство к '', то есть сначала проверяется тип поля, а потом значение, так что в elseif попадет только в одном случае, когда $user_key === '', т. е. только в случае пустой строки. В случай null, 0 попадем в "then"
Анонимус 09.12.2010 14:23 # +1
ничего. Плохо когда нет статической.
Вернее это тоже хорошо, когда у Вас программа из ста строк кода.
Если строк больше, то без статической типизации жизнь превращается в ад. А именно такой и становится жизнь любого пыхапешника, если он пытается программировать а не говнокодить
KirAmp 09.12.2010 19:12 # 0
Если вы захотите сложить овец с баранами то при динамики они сложутся и ошибку будите искать не один час.
А при статике у вас компилятор (или IDE) выдаст ошибку типов
Анонимус 09.12.2010 19:20 # −1
а без статической типизации надо будет руками менять везде
wmmorgun 09.12.2010 19:33 # 0
Анонимус 09.12.2010 19:36 # −1
т.е. документация (доки) лечит косяки отсутствия статической типизации?
ничего не скажешь -- красивое решение)))
>>равда и делать это приходилось пару раз.
в крупных проектах рефакторинг делается часто. Меняются классы, методы, сигнатуры, методы подымаются выше/ниже по иерархии.. Или в мире пхп делать рефакторинг не принято?)
bugmenot 09.12.2010 21:06 # −1
Lure Of Chaos 10.12.2010 17:14 # +1
конечно же нет ))) обычно все кидают и переписывают с нуля!
именно поэтому каждый пишет свой фреймворк каждый раз, и даже сам пыхоавтор против них!
paranoid 09.12.2010 15:27 # −2
перед тем, как на что-то срать, чтоб не казаться полным мудаком нужно сначала хотя бы минимально разобраться в сути процесса.
TarasB 09.12.2010 15:33 # −5
Анонимус 09.12.2010 15:34 # −1
dwinner 09.12.2010 11:56 # +4
KirAmp 09.12.2010 13:21 # 0
Lure Of Chaos 09.12.2010 17:34 # +1
типа
и далее по коду = )