0

Латаю ферму, используется класс $db, не могу тупо выполнить запрос внутри цикла, данные перебиваются запросом внутри while

<?
$db->Query("SELECT * FROM db_task WHERE moder = '1' AND kol_z >= 1 $catt ORDER BY up_date DESC");
while($task = $db->FetchArray()) {

  $db->Query("SELECT id FROM db_task_user WHERE id_task = '$task[id]' LIMIT 1"); // эта штука перебивает похоже $db и выводит пустоту, на самом деле данные есть
  if($db->NumRows() == 0) {//если это и строку выше закомментить, всё норм
  ?>
    Norm<br>
  <? 
  }

} ?>
11
  • кто там учил вас писать так <? ?>
    – Naumov
    Commented 4 авг. 2016 в 15:22
  • @Naumov скрипт не мой, знаю что говнокод, есть решение? Commented 4 авг. 2016 в 15:23
  • @Rammsteinik, перепишите запрос к базе так, чтобы остался один единственный запрос вне цикла.
    – Visman
    Commented 4 авг. 2016 в 15:25
  • 1
    @Rammsteinik а как насчет LEFT JOIN? :)
    – Firepro
    Commented 4 авг. 2016 в 15:29
  • 1
    @Rammsteinik Конечно в where не должно быть таких условий. Либо в where пишите (B.task_user = 1 OR B.task_user is null) что бы были все записи. То же касается и поля status, которое так же надо проверять на NULL. Либо вообще переносите условие в ON: select * from db_task A left join db_task_user B on B.id_task=A.id and B.task_user = 1 and B.status<>1
    – Mike
    Commented 4 авг. 2016 в 16:48

1 ответ 1

3

Судя по $db->FetchArray()- это одна из детских болезней всех классописателей - сохранение состояния.

$db сохраняет внутри себя результат запроса, при вызове в цикле эта переменная перезаписывается, и в итоге всё встаёт колом.

В рамках данного класса проблема не лечится.
Решение - выкинуть эту самопальную писульку и использовать PDO.

Хотя конечно вложенный запрос в любом случае - зло, и его надо переписывать на джойн.

8
  • это понял давно, как увидел класс, используя его нет решения? Commented 4 авг. 2016 в 15:30
  • @Ипатьев, такая проблема и у mysql_* присутствует :)
    – Visman
    Commented 4 авг. 2016 в 15:32
  • Как раз дописал строчку про это. про вложенные запросы можно забыть. единственный вариант - переписать на джойн, как советуют выше. Commented 4 авг. 2016 в 15:32
  • @Visman ни разу. Не надо путать магическое соединение с БД (которое там есть) с персистивным результатом запроса (которого там нет и никогда не было). Результат всегда явно хранится в переменной типа ресурс и перезапишется только если программист сам это сделает. Commented 4 авг. 2016 в 15:33
  • 1
    @Visman Если внутри цикла по mysql_fetch_assoc догадаться присвоить результат другого запроса переменной с тем же именем, то да - цикл сломается ;) Commented 4 авг. 2016 в 15:46

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.