- 1
- 2
- 3
- 4
- 5
if (!empty(($_POST['text'])) {
$text = $_POST['text'];
} else {
unset($text);
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+132.1
if (!empty(($_POST['text'])) {
$text = $_POST['text'];
} else {
unset($text);
}
guest 16.02.2009 18:12 # 0
в чем говно? в unset() ?
guest 16.02.2009 20:11 # 0
$sql .= isset($text)?"='".$text."' "?"=NULL".
Или хуже того - дальше в коде направо и налево будет юзатцо $text невзирая на то, что не так давно ему сделан unset.
На самом деле гавно в том, что у пхп-кодеров принято писать в данном случае так:
$text = $_POST['text'];
Т.е. без лишних ифов и поебать на нотисы в логах и плодящиеся как тараканы инъекции. Именно из за таких приходиться каждому полю из базы делать трим, анэскейп, проверять на валидность, целостность и т.п.
Извините, накипело ибо заебали уже "грамотные" пхп программисты..
guest 17.02.2009 05:06 # 0
может быть выдрано из контекста, но мне кажется, что делать unset для несуществующей переменной стрёмно.
[url]http://www.govnokod.com/575[/url]
взято отсюда. не думаю, что эти переменные задефайнены в
[quote=Говногость]Извините, накипело ибо заебали уже "грамотные" пхп программисты.. [/quote]
Как я вас понимаю.
p.s. сам я php программист
guest 17.02.2009 16:28 # 0
guest 17.02.2009 17:32 # 0
$text = $_POST['text'];[/quote]
мм.. где ты видел таких говнокодеров? ниразу такого невидел, и не делал.
guest 17.02.2009 18:00 # 0
[quote=Говногость]мм.. где ты видел таких говнокодеров? ниразу такого невидел, и не делал. [/quote]
Я с ними работаю :). Вчерашний пример (выборочный копипаст):
Стэйт для агента - текст из select-a. Селект с каким то глючным яваскриптовым поиском, поэтому периодически в стейт попадает пробел. Он не пустой, а потому успешно ложицо в базу. После таких скриптов мне приходицо "исправлять" "баги" след образом
[code=c]
@@ -1 +1,2 @@
-select * from agents where `state` is NULL;
+select * from agents where `state` rlike('\w+') or `state` is NULL;
+
[/code
guest 31.07.2009 10:47 # 0
#6 порадовал конечно
общая суть
а насчет пустых элиментов достаточно 28 строку заменить на
guesto 17.06.2016 00:27 # 0