- 1
- 2
- 3
- 4
- 5
- 6
- 7
<?php
if($key === false){
return false;
}else{
return $key;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+169
<?php
if($key === false){
return false;
}else{
return $key;
}
TarasB 03.11.2010 15:05 # −2
sbb 03.11.2010 15:07 # +3
TarasB 03.11.2010 15:15 # −1
user654321 03.11.2010 15:20 # 0
А если бы тут все же использовалась такая конструкция, то $key привелось бы булеану и false вернулась бы, если в $key лежал ноль, нулл, false или пустая строка.
istem 03.11.2010 15:59 # 0
переменная либо вообще не определена, либо определена, и никак не должна быть равна "false"
--
хотя 6 строчка выглядит при этом странно...
seonull 05.11.2010 10:31 # 0
Анонимус 06.11.2010 02:11 # +2
в пыхе же с типами -- помойка. один шедевр "===" чего стоит
Анонимус 03.11.2010 16:31 # +1
если переменная $key типа boolean и при этом равна false -- вернуть false.
во всех других случаях (не false, не boolean итд) -- вернуть $key.
строки 3-5 а так же 7 не имеют смысла
Govnocoder#0xFF 03.11.2010 16:34 # +1
user654321 03.11.2010 16:38 # 0
Govnocoder#0xFF 03.11.2010 16:39 # 0
Анонимус 03.11.2010 16:41 # +6
telnet 03.11.2010 16:57 # +5
http://govnokod.ru/56
istem 03.11.2010 17:27 # 0
а тут ещё и null может вернуть...
Morgan 04.11.2010 00:52 # 0
Lure Of Chaos 04.11.2010 23:50 # +5
istem 05.11.2010 00:37 # −1
andremacareno 06.11.2010 17:44 # −1
Кстати, меня терзают смутные сомнения: нужно ли здесь использовать else? Вроде итак выйдет, если "$key !== false" выполнится.
AxisPod 08.11.2010 09:48 # 0
istem 08.11.2010 12:27 # +1
Arigato 08.11.2010 18:07 # −4
user654321 08.11.2010 18:55 # 0
Vasiliy 09.11.2010 11:34 # 0
istem 09.11.2010 18:35 # 0