Как правильно использовать mysqli_num_rows()?

Question

[Aigerim1108]

В моей задаче на странице должны отображаться комментарии к выбранной записи, а если никто не прокомментировал запись нужно сообщить об этом.

$query_comments = "SELECT * FROM comments WHERE art_id = $note_id";
$s_note = mysqli_query($link, $query_comments);
while ($n = mysqli_fetch_array($s_note))
{
if(mysqli_num_rows($query_comments)==0)
echo "Эту запись еще никто не комментировал";
else
echo $n ['comment'], "<br>";
}

Comments

[Erl]

а зачем mysqli_num_rows?
вы ведь записали результат запроса в $s_note, дак работайте с ним и средствами php

[Дмитрий]

Вы не сделали запрос, а просто передали строку
$s_note вместо $query_comments
Ну и проверку до цикла с фетчем
Ну и фетчить в ассок

Answer 1

[FanatPHP]

Это очень хороший вопрос.
Ответ на него - никак не использовать.

Да, в mysqli действительно есть специальная функция, которая может сказать, какое количество строк вернул запрос SELECT.
Традиционно употребляется в двух случаях:

а) когда это не нужно
б) когда приводит к катастрофическим последствиям

Первый вариант - это когда мы хотим узнать, вернул ли запрос хоть какие-то данные, или нет. Но на этот случай у нас есть сами данные. Зачем отдельно запрашивать их количество, если мы все равно будем эти данные получать в переменную, которую потом и можно будет использовать чтобы узнать, вернул ли запрос что-нибудь или нет.

Второй вариант - если эта функция используется чтобы посчитать, сколько строк лежит в БД. В таком варианте это будет откровенное вредительство, поскольку данных может быть очень много, и все эти данные БД должна сначала получить у себя, а потом отправить в РНР. Заняв всю доступную память или даже вызвав фатальную ошибку нехватки памяти.

Правильным решением этой задачи будет сделать запрос вида SELECT COUNT(*) FROM .... В этом случае БД сама внутри себя посчитает количество строк (очень быстро) и вернёт только одно число, которое не занимает оперативную память вообще.

Вот и получается, что функция mysqli_num_rows() является либо вредной, либо бесполезной

В данном случае надо сначала получить записи из БД

// БЕЗОПАСНО выполняем запрос
$stmt = $link->prepare("SELECT * FROM comments WHERE art_id = ?");
$stmt->bind_param("s", $note_id);
$stmt->execute();
// получаем данные
$result = $stmt->get_result();
$comments = $result->fetch_all(MYSQLI_ASSOC);
?>

А после этого уже их выводить

<?php if ($comments): ?>
    <?php foreach ($comments as $row): ?>
         <?=$row['comment']?><br>
    <?php endforeach ?>
<?php else: ?>
    Эту запись еще никто не комментировал
<?php endif ?>

Как видно, никакой mysqli_num_rows нам не понадобилось

Comments

[N]

Вот и получается, что функция mysqli_num_rows() является либо вредной, либо бесполезной

3 вариант - это при изменении/удалении получить кол-во затронутых записей...

[FanatPHP]

N, это не третий вариант, это совсем другая функция

[N]

FanatPHP, Верно :)

Спутал с mysqli_affected_rows...

Во всём виноваты всякие ORM'ы...отучают от "старой школы"... :)

[Spartak (Web-StyleStudio)]

FanatPHP,

Традиционно употребляется в двух случаях

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

// БЕЗОПАСНО выполняем запрос
$stmt = $link->prepare("SELECT * FROM comments WHERE art_id = ?");
$stmt->bind_param("s", $note_id);
$stmt->execute();
// получаем данные
$result = $stmt->get_result();
$comments = $result->fetch_all(MYSQLI_ASSOC);

Согласен, этот способ лучше, чем тот что использует автор вопроса.

[FanatPHP]

Spartak (Web-StyleStudio), открою маленький секрет.
Если вдруг когда-нибудь понадобится такое число, то его можно получить через count($comments). Это простое и универсальное решение для любых массивов, а не только полученных из БД.

[Spartak (Web-StyleStudio)]

FanatPHP,

Если вдруг когда-нибудь понадобится такое число, то его можно получить через count($comments).

Верно, можно, и не только через count, но вопрос автора был в том: Как правильно использовать mysqli_num_rows()?

[FanatPHP]

Spartak (Web-StyleStudio), я ответил на него, но тебе просто хочется поспорить потому что ты учил пхп по видео на Ютубе, и не можешь себе представить осмысленный подход к написанию программ.

[Spartak (Web-StyleStudio)]

FanatPHP,

тебе просто хочется поспорить

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

[FanatPHP]

Spartak (Web-StyleStudio), да при чем здесь царёк?
Я не виноват в том что любой кто понимает код который пишет, на голову выше серой массы похапешников, лепящих пару функций, которые осилили зазубрить

mysqli_num_rows() - это рудимент из прошлого века, когда хтмл писали вперемешку с SQL
С тех пор прошла почти четверть века, но большинство обезьян так и не смогли осилить модульный подход к программированию, когда получением данных из бд занимается одна функция, а отображением - совсем другая. В которой даже доступа к объекту БД нету, и запросить эту несчастную mysqli_num_rows() неоткуда

Answer 2

[Spartak (Web-StyleStudio)]

$query_comments = "SELECT * FROM `comments` WHERE `art_id` = '{$note_id}'";
$s_note = mysqli_query($link, $query_comments);
if (mysqli_num_rows($s_note)) {
    while ($n = mysqli_fetch_assoc($s_note)) {
        echo $n['comment']."<br>";
    }
} else {
    echo "Эту запись еще никто не комментировал";
}

Comments

[Aigerim1108]

Warning: mysqli_num_rows() expects parameter 1 to be mysqli_result, string given in C:\OpenServer\domains\localhost\lab sql\comments.php on line 23
Эту запись еще никто не комментировал

Ошибка выходит

[Spartak (Web-StyleStudio)]

Aigerim1108,
Поправил запрос

[FanatPHP]

Spartak (Web-StyleStudio), не стыдно такое писать?

[Aigerim1108]

Spartak (Web-StyleStudio),
СПАСИБО!

[Spartak (Web-StyleStudio)]

FanatPHP,

не стыдно такое писать?

Нет

[FanatPHP]

Все еще не стыдно?