Как подсчитать количество заказов?

Question

[l55uiz]

Подскажите, как подсчитать количество заказов для юзера?
В таблице orders есть поле user_id, в котором хранится айди юзера, взятый из сессии. И нужно вывести сколько было заказов
Данный код выводит мне 0

public function countOrders()
  {
    try
    {  
      $stmt = $this->runQuery("SELECT count(user_id) FROM orders GROUP BY user_id WHERE user_id =:user_id");
      $stmt->bindValue(":user_id", $_SESSION['user_session'], PDO::PARAM_STR);
      echo $stmt->rowCount(); 
    }
    catch(PDOException $e)
    {
      echo 'Ошибка.';
    }     
  }

Answer 1

[ThunderCat]

$stmt = $this->runQuery("SELECT count(id) FROM orders WHERE user_id =:user_id");

а то вы чет перемудрили

echo $stmt->rowCount();
это зачем? учитывая что вы выбираете уже каунт?

Comments

[l55uiz]

Написал так:

$stmt = $this->runQuery("SELECT count(id) FROM orders WHERE user_id =:user_id ");
      $stmt->bindValue(":user_id", $_SESSION['user_session'], PDO::PARAM_STR);
      $stmt->execute();  
      $countOrders = $stmt->fetchAll(PDO::FETCH_ASSOC);
      return $countOrders;

Вывожу в html так

foreach ($Cart->countOrders() as $orders) {
    echo $orders[count(uid)] ;
   }

Выводит пустоту, однако print_r($Cart->countOrders()) показывает 14.

[ThunderCat]

l55uiz: обратите внимание что в результате запроса вы получите не сторки, а ЧИСЛО. Одно. Количество заказов.

[l55uiz]

ThunderCat: все, спасибо. Убрал Aall, и вывел просто $orders, все работает

Answer 2

[FanatPHP]

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

Как это должно выглядеть на самом деле:

public function countOrders($user_id)
{
      $stmt = $this->pdo->prepare("SELECT count(1) FROM orders WHERE user_id =?");
      $stmt->execute([$user_id]);
      return $stmt->fetchColumn(); 
}
// и использование
echo $obj->countOrders($_SESSION['user_session']);

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

Comments

[l55uiz]

Только сейчас смог посмотреть твой коммент. Ну не совсем уж ад, если говорить честно. В интернете еще хуже чем мой код есть. Как же это называть?
Я отталкивался от большинства примеров(на форумах, на курсах, на бурж. сайтах) как писать похожий pdo запрос.
Где же найти примеры как у тебя, раз ты говоришь что мой код - ад?
Например, ты пишешь неименованный плейсхолдер, я его именую. Тут особой разницы нету. (это так, чисто для инфы).
Далее, я вывожу не колонку(fetchColumn(), а число через Fetch).
И не произвожу вывод в html типа "echo $obj->countOrders($_SESSION['user_session']);", а просто вывожу переменную <?=$orders?>

Почему это плохо? Мне просто интересно.
П.с. это не обсырание, а заинтересованность и аргументирование.

[FanatPHP]

l55uiz: ну-ну, врать-то не надо. Ты выводишь

foreach ($Cart->countOrders() as $orders) {
echo $orders[count(uid)] ;
}

плюс ты используешь идиотскую функцию runQuery() вместо нормальной pdo::prepare()
плюс ловишь исключение и тут же вываливаешь на экран идиотское слово "ашыпка, насяльинка"
плюс твоя функция считает только для сессии, а когда понадобится посчитать для другого юзера, придется писать еще одну.
Ну ок, не ад, но чистилище. куча мусора и бесполезного кода, и в тоже время очень ограниченная функциональность.

[l55uiz]

FanatPHP: понял, спасибо