Поможете определиться с правильностью запроса SELECT PDO?

Question

[Марк]

Добрый день.

Посмотрите пожалуйста, где я ошибаюсь?
1)Тут работает, но не то что нужно.

$nRows = $pdo->query('SELECT count(*) FROM Notice')->fetchColumn();
      echo $nRows;

Все ок, но мне нужен WHERE?
2) То что нужно, не работает (((

$sql = $pdo->prepare('SELECT * FROM Notice WHERE uid = :uid && status = :status');
      $sql->bindParam(':uid', $login);
      $sql->bindParam(':status', $status);
      $login = $_SESSION('login');
      $status = 0;
      $sql->execute();
      if($sql->rowCount() > 0) {
        echo 'У вас есть непрочитанные сообщения';
      }
      else {
        echo 'Нет новых сообщений';
      }

Где-то было мнение, что нельзя передать 2 where.
Товарищи ученые, я изучаю мат часть, но что-то не получается найти рабочий запрос.
Спасибо.

Comments

[DevMan]

day90 подсветка кода? не, не слышал.
в редакторе есть волшебная кнопка с тремя точками.

Answer 1

[DevMan]

д - дебаг и о - включенные ошибки творят чудеса.
вас не смущает биндинг переменных до того как они определены? меня вот смущает.

Comments

[Марк]

Смущает, но исхожу из php.net/PDOStatement.bindParam. До этого, все работало корректно.

[DevMan]

day90: точно, запамятовал.
вам нужно проверять результат, а не кол-во рядов:
if($sql->fetchColumn() > 0)
или переписать запрос на что-то, что возращает результат в зависимости от вашего условия:

SELECT id FROM Notice WHERE status = :status && login = :login

первый вариант и проще и легче, если цель только проверить кол-во сообщений.

[Марк]

Моя цель, выявить есть ли у юзера непрочитанные сообщения(СМС) и если да - вывести цифру или число соответствующее количеству этих СМС, я принял к сведению fetchcolumn и если я правильно понимаю, это не мой вариант, т.к. я решил, что в моем случае именно количество строк содержащее статус 0(не прочитано) в связке с UID($login) полностью удовлетворит все потребности. Вы знаете PHP лучше меня и я не понимаю, как нужно переписать запрос, т.к. в моей вселенной все должно работать, но по прежнему выдается "else", несмотря что в БД есть строки со статусом 0 и сессионным uid.

[Марк]

Спасибо за помощь.

[DevMan]

day90: запрос SELECT COUNT(*) всегда вернет один и только один ряд, даже если ничего не найдено.
запрос SELECT * FROM Notice WHERE uid = :uid && status = :status в совокупности с rowCount() вернет кол-во найденых по условию рядов.

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

$login = $_SESSION('login');
$status = 0;
echo "SELECT * FROM Notice WHERE uid = '$login' && status = '$status'";

полученную строку выполняете в phpmyadmin и смотрите результат.
кстати, вы уверены, что uid и $_SESSION('login') - это одно и то же?

[Марк]

Есть контакт, uid и $_SESSION('login') не бьются, подставил вместо $_SESSION('login') значение из базы например - 418832912388410053. Все есть. Моя проблема не решена, но ответ на поставленный вопрос явно есть. Спасибо.

[DevMan]

day90: пожалуйста.
один момент: $_SESSION('login') - странная запись. может подразумевается $_SESSION['login']?
вы б таки включили вывод или логирование ошибок.

[Марк]

Да, вы правы - $steamprofile['steamid']. Включу.

[FanatPHP]

Марк: зачем помечать как решение бессмысленный и неверный ответ?

[Марк]

Мне тяжело классифицировать ответы DevMan, как бессмысленные и неверные, мне они помогли, в этой связи я пометил их "Решение". Мне очень жаль, если я навредил.

[DevMan]

FanatPHP: чукча не читатель, чукча возмущатель?

Answer 2

[Rsa97]

Поскольку вы запрашиваете COUNT(*), то всегда будет возвращаться одна строка и условие ($sql->rowCount() >= 1) всегда будет истинным.

Comments

[Марк]

Спасибо, исправил, по прежнему выдается else.