Почему PHP игнорирует условие?

Question

[Ayk72]

Добрый день!

Необходимо написать код, который подсчитает кол-во запятых в каждой записи mysql.
На сервере PHP 5.3.

Сделал циклический запрос и условие:

$count = substr_count($row[category], ',' );
if ($count <= 3) $cat = 40;

Если сделать вывод $count, то он правильно покажет количество запятых, но в условии он тупо игнорирует условие меньше или равно! И тупо всем назначает 40. Почему?

Comments

[Ayk72]

Значение 100% присвоено $count, выводит и считает прекрасно, а в условии не работает, что за бред?

[Ayk72]

Всем спасибо! Проблема решена! unset

Answer 1

[Cyapa]

Попробуйте что-нибудь из этого:
- Назвать счетчик по другому
- Таки воспользоваться блочным оператором (поставить фигурные скобки)

Comments

[Ayk72]

Сделал, результат тотже

[Cyapa]

Ayk72: тогда нескромный вопрос, а до этого кода переменная $cat чем инициализирована?

[Ayk72]

Cyapa: вообще код достаточно простой, т.к. нихера не работает. Попросили сделать цикл на DLE, чтобы присвоить $cat новостям, у которых более 3 категорий присвоено, вот полностью код.

$z = $db->query("SELECT id,title,category FROM _post ORDER BY id DESc");

while($row = $db->get_row($z)){
$num = substr_count($row[category], ",");
if ($num < '2') {$cat = 40;
#$db->query( "UPDATE _post set cat='{$cat}' where id='{$row[id]}'" );
}
}
$db->free();

$row[category] заполнен у каждой новости типа - 6,19,21

[Ayk72]

но он тупо присваивает 40 вообще всем

[Вадим]

Ayk72: а что ж ты хочешь то, переменная не сбрасывается перед следующей итерацией цикла. И если первая запись подходит под условие, то всегда будет 40

[Cyapa]

Ух...

$z = $db->query
("
SELECT `id`, `title`, `category`
FROM `_post`
ORDER BY `id` DESC
");

$rows = array();

while($row = $db->get_row($z))
{
$num = substr_count($row['category'], ',');
if($num < 2)
{
$rows[] = '{' . $row['id'] . '}';
}
}

if(count($rows))
{
$db->query
("
UPDATE `_post`
SET `cat` = '{40}'
WHERE `id` IN (" . impole(',', $rows) . ")
");
}

$db->free();

[Ayk72]

Вадим:
$nume = substr_count($row['category'], ",");
if ($nume <= 2) {$cat = 40;}
unset($nume);

не срабатывает, объясни, я туплю.

[Cyapa]

implode*

[Вадим]

Ayk72: Для начала объясни где эта переменная используется? Кроме цикла нигде больше?

[Ayk72]

Вадим: нигде.

[Ayk72]

Вадим: это полностью код

[Ayk72]

Вадим: решил, благодарю! unset для cat

[Cyapa]

Ayk72: Тот код, что я привел, должен работать.

И да, unset тут ни к чему.

[Ayk72]

Cyapa: да, благодарю. Просто я такой вид кода выбрал потому что там еще десятки условий будут. И этот код на 1 раз, чисто спарсить и удалить.

[Вадим]

Ayk72: мой вариант ниже.

Answer 2

[Reistlin]

1.$row['category'] а не $row[category]
2.что значит тупо всем назначает 40?у вас $cat строка,если вы там записали 40,то и будет показывать.Я так понимаю у вас это в цикле происходит?если да,то после того как 1 раз $count оказался меньше или равно 3,$cat всегда будет 40

Comments

[Ayk72]

1. Это я просто методом тыка искал проблему.
2. Как исправить эту проблему? Выше код полностью, посмотрите, пожалуйста.

[Ayk72]

Он каждой новости присваивает уникальный $count. Проверял, даже unset ставил, толку нет.

[Reistlin]

я не работал с DLE.но предположу,что:
1.запросы об изменении делаются не методом query с вызовом потом метода free,посмотрите как в движке реализованы изменения данных
2. "UPDATE _post set cat='{$cat}' where id='{$row[id]}'" ); это не правильно,здесь не указана таблица с которой вы работаете,название поле странное(_post set cat) https://ru.wikipedia.org/wiki/Update_(SQL)

[Ayk72]

Изнаур Бексултанов: это просто убрал, таблица в коде у меня правильно. Update тоже у DLE так реализовано. Тут код рабочий 100% прост if не работает почему-то.

[Reistlin]

тогда просто попробуй:_post set cat='40',а не создавать отдельные переменные

Answer 3

[Вадим]

Если я правильно понял задачу, то вот

$posts = $db->query("
    SELECT id, title, category
    FROM _post
    ORDER BY id DESC
");
$postsToUpdate = array();

while($post = $db->get_row($posts)) { //как-то странно в дле получение записи реализовано
    if(substr_count($row['category'], ',') <=2) $postsToUpdate[] = $post['id'];
}
if(!empty($postsToUpdate)) $db->query("UPDATE _post SET cat = 40 WHERE id IN (".implode(',', $postsToUpdate).")");

Answer 4

[Анатолий Кирсанов]

Да нет инициализации $cat и все. Тоже самое можно сделать и в ветке else условного оператора.
В Вашей ситуации срабатывает конкретный набор данных - на первой выбранной из БД строке 3 и более запятых. А дальше уже нет разницы сколько их потому, что в $cat уже записано 40 и это значение не сбрасывается.