- 1
($property['valid'] == 'ошибка') !== TRUE
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+158
($property['valid'] == 'ошибка') !== TRUE
вот и бери стажеров на работу
neeedle 15.04.2013 17:16 # −3
eth0 15.04.2013 17:51 # 0
bormand 16.04.2013 05:16 # +1
Okay. Сравним с переменной:
neeedle 16.04.2013 13:00 # −2
Ну может быть кривенькое логическое выражение, ну работает оно со статичной строкой, ну и что?
Работает то оно ведь правильно.
false - ошибка.
true - все нормально.
Похоже, что у кого-то перфекционизм.
CRRaD 16.04.2013 13:26 # 0
neeedle 16.04.2013 13:54 # −1
bormand 16.04.2013 17:42 # 0
P.S. Но тред минуснул т.к. бульшиты уже наскучили.
neeedle 16.04.2013 17:53 # 0
Кривенько конечно же, но постить джуниоровский код на говнокодру, не комильфо.
Я думал, что тут какая-то специфика php, что-то очень смешное, а так.
Ничего особенного же. Просто условие сложно написано.
eth0 16.04.2013 17:53 # 0
scriptin 15.04.2013 20:07 # −1
2.
> $property['valid']
> 'ошибка'
inkanus-gray 16.04.2013 00:14 # −2
1. Магическая константа в конце выражения. Избавляемся: TRUE → intval('')
2. Магическая константа в квадратных скобках. Избавляемся: $property['valid'] → (object)$property->valid
3. Слишком мало полосочек и неявное приведение типа. Исправляем: == → ===
4. Слишком мало явных приведений типов. Итого получаем:
anonimb84a2f6fd141 16.04.2013 01:05 # 0
scriptin 16.04.2013 20:21 # 0
guest 10.05.2013 03:07 # 0
anonimb84a2f6fd141 22.04.2013 07:55 # 0
guest 14.05.2013 21:57 # 0
это он тоже сам придумал или ему по-старше подсказали?
Если это у вас такая передача ошибок, то можете повышать своего стажера и увольнять по-старше