Как посчитать и вывести количество строк по условию?

Question

[Максим]

Добрый день.
У меня выводятся данные на странице в виде таблиц в цикле. Хотел сделать до этого цикла подсчёт количества строк по заданному условию таким способом:

function get_count_reject()
{
    global $link;
if (isset($_POST['order_date'])) {
        $form_date = $_POST['order_date'];
        $sql = "SELECT COUNT(id) as count FROM Users WHERE order_date='$form_date' AND reject=1";
        $result = mysqli_query($link, $sql);
        $count = mysqli_num_rows($result);
        return $count;
    }
else {
    $SQL = "SELECT COUNT(id) as count FROM Users WHERE order_date='2019-04-06' AND reject=1";
    $result = mysqli_query($link, $SQL);
    $count = mysqli_num_rows($result);
    return $count;
}
}

Answer 1

[Rsa97]

Вы указали агрегатную функцию COUNT без группировки по каким-либо полям. Соответственно, вам всегда возвращается одна строка, о чем и говорит mysqli_num_rows.

Comments

[Максим]

Т.е. нужно привести запрос к такому виду?

SELECT * FROM Users WHERE order_date='$form_date' AND reject=1

И как потом подсчитать количество строк в запросе?

[Rsa97]

Максим, Нет, нужно получать не количество строк в ответе, а сам ответ.

[Максим]

Rsa97,

[Максим]

Rsa97, я пытался так сделать, как пример из документации php, но опять же результата нет :(

function get_count_reject()
{
    global $link;
    if (isset($_POST['order_date'])) {
        $form_date = $_POST['order_date'];
        $SQL = "SELECT * FROM Users WHERE order_date='$form_date' AND reject=1";
        $result = mysqli_query($link, $SQL);
        $count = mysqli_num_rows($result);
        return $count;
    }
    else {
        $sql = "SELECT * FROM Users WHERE order_date='2019-04-07' AND reject=1";
        $result = mysqli_query($link, $sql);
        $count = mysqli_num_rows($result);
        return $count;
    }
}

<?php
                            $post1 = get_count_reject();
                            ?>
                            <?php foreach ($post1 as $count): ?>
                            <?php echo $count; ?>
                            <?php endforeach; ?>

[Rsa97]

Максим, Не смотрите больше такие примеры, они вас плохому научат.
1. global - это очень и очень плохо.
2. Подставлять напрямую в текст запроса что-либо, полученное от клиента, - моветон. Данные из POST должны проверяться заранее. Используйте подготовленные запросы с плейсхолдерами.
4. Использовать "магические" константы нехорошо.

<?php
require_once('db.php');

function get_count_reject($link, $date) {
  $req = mysqli_prepare($link, "SELECT COUNT(*) FROM `Users` WHERE `order_date` = ? AND `reject` = 1");
  mysqli_stmt_bind_param($req, 'c', $date);
  mysqli_stmt_bind_result($req, $count);
  mysqli_stmt_execute($req);
  mysqli_stmt_fetch($req);
  mysqli_stmt_close($req);
  return $count;
}

if (isset($_POST['order_date']) && preg_match('/^\d{4}-\d{2}\d{2}$/', $_POST['order_date'])) {
  $date = $_POST['order_date'];
} else {
  $date = date('Y-m-d');
}
$post1 = get_count_reject($link, $count);

[Максим]

global - это очень и очень плохо.

Что не так с глобалом?

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

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

"магические" константы

Что это такое?)

И код почему-то теперь открывает просто пустую страницу -_- Видимо, я что-то не так сделал.

[Rsa97]

Максим,

Что не так с глобалом?

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

Что это такое?)

А откуда у вас дата '2019-04-07'? И будет ли она актуальна через месяц? А в следующем году?

И код почему-то теперь открывает просто пустую страницу -_- Видимо, я что-то не так сделал.

Надо не копипастить код, а разбираться и модифицировать под свои цели.

[Максим]

откуда у вас дата '2019-04-07'?

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

Надо не копипастить код, а разбираться и модифицировать под свои цели.

Это я понимаю, я сам код пока не понял :(

[Vitsliputsli]

Что не так с глобалом?

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

Работа с БД в mysqli очень неудобная и непонятная, так что не удивляйтесь, что сразу не разобрались. Вот пример (минимально работающий, т.е. без проверок и прочего):

spoiler

<?php

$dbOptions = [
	'servername' => '127.0.0.1',
	'username' => 'test',
	'password' => 'test',
	'database' => 'test',
];

$query = "SELECT COUNT(*) FROM `Users` WHERE `order_date` = ? AND `reject` = 1";
$types = 's';
$values = ['2019-04-07'];

$connection = new \mysqli(
	$dbOptions['servername'], 
	$dbOptions['username'], 
	$dbOptions['password'], 
	$dbOptions['database']
);

$stmt = $connection->prepare($query);
$stmt->bind_param($types, ...$values);
$stmt->execute();
$meta = $stmt->result_metadata();

$params = [];
while ($field = $meta->fetch_field()) 
{ 
	$params[] = &$row[$field->name];
}
$stmt->bind_result(...$params);

$result = [];
while ($stmt->fetch()) { 
	$result[] = $row;
}

var_dump($result);

В процедурном варианте не нашел.

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

[Максим]

Vitsliputsli, спасибо за пояснение. Про PDO мне уже говорили ранее, нужно будет изучить.