Комментарии к записи: Задерка между итерациями

Автор: coldFlame

coldFlame — Thu, 31 Jul 2008 11:02:18 +0000

Все правильно, но цикл do-while значит «выполнять, пока» – в данном случае «выполнять, пока *нет* задачи».

А sleep должен стоять вторым параметром, чтоб при последней итерации он не отрабатывал – зачем спать, если уже выполнено условие выхода из цикла?

Одним запросом дело не обходится, ибо вся соль в том, что в задачи в базу заносит другой скрипт.

Автор: Alex

Alex — Wed, 30 Jul 2008 18:09:06 +0000

Не, ну может конечно я не врубаюсь в семантику цикла из примера, но ИМХО условие для while неправильное :)
Давай упростим и выбосим sleep из условия. Когда должен продолжаться цикл? Видимо пока mysql_fetch_assoc возвращает в $task не FALSE. Ну так это записывается как:
} while ($task); // если напишем !$task, то вылетим после первой итерации в случае непустого рекордсета

Дальше уже подключим sleep:
} while (sleep(1) || $task); // ну sleep первым чтоб он по принципу вычисления с лева на право участвовал в каждой итерации

Вообще если цикл есть обработка строк из одного запроса, то надо тело цикла переписать:
$result = mysql_query(‘…’);
do {
$task = mysql_fetch_assoc($result);
} while (sleep(1) || $task);
mysql_free_result($result);

Автор: coldFlame

coldFlame — Wed, 30 Jul 2008 17:26:48 +0000

Нет-нет, do-while выполняется, пока условие истинно, так что именно !$task.

А насчет второго замечания вы правы. Только еще правильнее будет
} while (!$task && !sleep(1))
или, по правилу де Моргано
} while (!($task || sleep(1))

Спасибо, обновил пост.

Автор: Alex

Alex — Wed, 30 Jul 2008 13:55:00 +0000

} while (!$task || sleep(1));
- чето тут перемудрили. Такой цикл сработает с единственной задержкой после последней итерации (которая вероятно тоже будет единственной из-за !$task). ПХП ж не расчитывает второй аргумент ИЛИ если первый дал 1.
ИМХО правильнее было бы так:
} while (sleep(1) || $task);

Автор: Алексей Токарь

Алексей Токарь — Wed, 30 Jul 2008 09:11:05 +0000

Данный подход имеет как минимум две проблемы.
первая – при беглом просмотре кода, не всегда удастся заметить этот sleep
вторая – сама неочевидность данного решения. Человек не знающий о возвращаемом значении этого sleep будет долго разбираться что же это тут такое происходит

А ведь хорошим тоном считается писать код для людей а не для машин :)