Цикл while для выборки из базы?

Question

[Андрей Бойченко]

Добрый вечер! Объясните чем плох данный подход? Я имею ввиду то, что здесь для выборки из базы используется цикл while и какие есть альтернативы? Спасибо!

<?php
$cnn = mysqli_connect("host","user","pass","db");
$query = "SELECT * FROM `my_sql_table`";
$res = mysqli_query($cnn,$query);
while($row = mysqli_fetch_array($res))
{
echo "Номер: ".$row['id']."
\n";
echo "Имя: ".$row['firstname']."
\n";
echo "Фамилия: ".$row['surname']."
\n";
}
?>

Answer 1

[Алексей Приказчиков]

Чем вам плох данный пример? Довольно таки классическое решение. Можно заменить на FOR, но тогда не будет, так наглядно. do ... while может наоборот всё испортить, так как если даже выборка будет пустая, всё равно выполнится и выведет вам:

Номер: *
Имя: *
Фамилия: *

Звёздочками отметил, то где вам PHP выведет предупреждения, если включён вывод всех сообщений об ошибках.

Comments

[Андрей Бойченко]

Много раз видел на тостере как такой подход ругали, правда аргументов по этому поводу тоже приведено не было, в любом случае спасибо!

[Алексей Приказчиков]

Андрей Бойченко: не за что. Почитайте, про циклы.

Это же типо мат. часть ;-) Вы их плохо понимаете.

Answer 2

[ThunderCat]

Извините, я прерву предыдущих ораторов, создалось впечатление что автор не совсем то же что отвечающие имел в виду, и, если я верно понял вопрос, отвечу:

Для ВЫБОРКИ из базы здесь НЕ ИСПОЛЬЗУЕТСЯ цикл, цикл используется для вывода РЕЗУЛЬТАТОВ ВЫБОРКИ из базы, весь набор уже получен и теперь только выводится в цикле. Выборка осуществляется в строке
$res = mysqli_query($cnn,$query);
после этого в $res хранится ссылка на ресурс со всей выборкой.
mysqli_fetch_array($res) создает массив из строчки, на которую в данный момент указывает внутренний указатель и переводит данный указатель на следующую строку выборки. RTFM. И да пребудет с Вами сила чтения мануала )

А делать запрос в цикле - да, дурной тон и дурная идея.

Answer 3

[Андрей Буров]

а кто сказал что плох?

PS: плохо только mysqli_ mysql_ использовать...

Comments

[DevMan]

mysqli_ использовать норм

[Алексей Приказчиков]

DevMan: жаль, что нельзя лайкнуть коммент ) Тоже не вижу ничего плохо в mysqli_, может Человек привык работать в Объектном подходе? )

[DevMan]

Алексей Приказчиков: mysqli_ тоже позволяет работать в объектном стиле.
у pdo есть свои преимущества, но это не делает mysqli_ отстоем.

[Андрей Буров]

DevMan, Алексей Приказчиков: я абсолютно не против mysqli_. Просто может привык к PDO...

PS: у автора в вопросе используется mysqli_connect, но mysql_fetch_array

[GDim]

$res = mysqli_query($cnn,$query);
while($row = mysql_fetch_array($res))

Алексей Приказчиков: видимо Андрей Буров имел в виду что в цикле используется mysql_, а при запросе mysqli_

[Алексей Приказчиков]

GDim: вопрос был по циклу, не думал по подключению, хотя да тут косяк есть )

Answer 4

[krypt3r]

Видимо, речь шла о том, что mysqli используется точно так же, как использовался mysql — разницы в коде никакой, только буковка i в вызове функций добавилась. Если в запрос не подставляются данные пользователя, использование mysqli_query(), модет быть, и оправдано. Но в ином случае следует юзать prepared statements (prepare(), bind_param(), execute()).
Все то же самое касается не только mysqli, но и PDO. Тут на тостере немало было говнокода аля mysql_xxx, заменили только функции mysql_xxx на методы PDO

Comments

[Андрей Бойченко]

О практиках с использования всего, что вы перечислили я знаком. С вероятностью в 100% буду это использовать, но меня по сути интересовал только вопрос с запросов в цикле постольку поскольку я видел в интернет, на том же тостере, что это пережиток времени и дурной тон.