- 1
- 2
- 3
- 4
if(strlen(trim($row['article']))!=0)
{
// Some action
}
"PHP" / Говнокод #9230
0
empty? не, это из будущего.
Суки! Сколько можно это писать!
Запостил: 
Zerstoren,
(Updated )
Ну а если в $row['article'] будут пробелы находиться то ваш код с empty не сработает. И вообще empty нужно пользоваться когда толком не знаешь, определена ли переменная. А тут все четко, нужно лишь обрезать строку от лишних боковых пробелов и глянуть, осталось ли что то. Не говнокод, минусую!
Да, еще если $row['article']="0" то empty тоже не сработает!
>>Ну а если в $row['article'] будут пробелы находиться то ваш код с empty не сработает.
Я ничего не сказал про trim, он здесь нужен и это правильно.
>>И вообще empty нужно пользоваться когда толком не знаешь, определена ли переменная
Путаете с isset
>>Да, еще если $row['article']="0" то empty тоже не сработает!
Представьте себе, так и должно быть! Не сработает, но зачем ему срабатывать? Переменная не пуста, у нее длинна в 1 байт.
Соль empty в том, что если будет найден первый байт, она сразу вернет true. В отличии от strlen, который вернет количество байт в строке.
Т.е. Отрицание пустоты, если строка как раз не пуста, то оно выполнит действие.
Во первых, оно быстрее, не значительно, но быстрее.
Во вторых логически правильно
В третьих лаконично.
Вообще, сайт с которого я выдрал этот код - один сплошной артефакт говна.
Только файл index.php весит 110кб. Перемешка HTML, JS, CSS и PHP кода.
Вы сами пробовали исполнить такой то код?
так что emty в самый раз
Вместо того, чтобы обсирать других — почитайте мануал.
>> Представьте себе, так и должно быть! Не сработает, но зачем ему срабатывать? Переменная не пуста, у нее длинна в 1 байт.
$var=' '; // пробел
print !empty($var)?'is not empty':'is empty'; // выдает is not empty
print '<br />';
$var='0'; // (string)ноль
print !empty($var)?'is not empty':'is empty'; // выдает is empty
print '<br />';
$var=0; // (int)ноль
print !empty($var)?'is not empty':'is empty'; // выдает is empty
Как видите, все три варианта переменных длинной в 1 байт.. Но результат функции empty не везде одинаков!
Согласен, приведенный говнокод корявенько выглядит. И что приведенный пример с empty() выглядит как минимум более лаконично и более эффективно (по скорости отработки). Но чего сразу кричать "Говнокооод! Говнокоооод!" Все такие, блин, правильные. Соревнование что-ли - кто круче обхамит чужой код? Уж по мелочам то придираться не стоит, наверное.