Почему выводит массив?

Question

[Икотий Айтишевич]

Внутри return есть переменная, в ней выводит array, почему? Там же count...

function userMessageNotification() {
       for($i=1; $i>0; $i++) {
          $userNotificationId = $_SESSION["profile"]["id"];
          $messageStatus = getOne("SELECT count(*) FROM uni_chat_messages WHERE chat_messages_status=1 AND chat_messages_id_user=' $userNotificationId ';");
          if( $messageStatus > 0 ) {
             return '
               <li>
                  <i class="las la-envelope"></i>
                  <a href="'. _link( "chat" ) .'">Новых сообщений получено (<span>'. $messageStatus .'</span>)</a>
               </li>
             ';
          }else {
             return '';
          }
       }
    }

Comments

[Valentyn]

Интересно спросить, зачем это обернуто в цикл for?

Answer 1

[vdem]

Вообще-то следовало бы добавить код функции getOne(). Нам что, догадываться что в ней? Впрочем, это несложно. Она возвращает в виде массива одну запись из набора данных. И этот массив содержит единственный элемент - значение count.

Comments

[Икотий Айтишевич]

$messageStatus = $mysqli->query("SELECT count(*) FROM 'uni_chat_messages' WHERE 'chat_messages_status'='1' AND 'chat_messages_id_user'='$userId'");
      if( $messageStatus > 0 ) {
         return '
         <li>
            <i class="las la-envelope"></i>
            <a href="'. _link( "chat" ) .'">Новых сообщений получено (<span>'. $messageStatus .'</span>)</a>
         </li>
         ';
      }else {
         return '';
      }
   }

Вывел таким способом, а переменная пустая...

[vdem]

iko.it, Она не может быть пустая. Попытайтесь хоть документацию к этой функции прочесть.

Возвращает false в случае неудачи. В случае успешного выполнения запросов SELECT, SHOW, DESCRIBE или EXPLAIN mysqli_query() вернёт объект mysqli_result. Для остальных успешных запросов mysqli_query() вернёт true.

Ссылка

[vdem]

iko.it, А вообще, будете в таком стиле писать, обязательно нарветесь на SQL injection.

[Икотий Айтишевич]

vdem, спасибо, нашёл ошибку. Учту.

[vdem]

iko.it,

$pdo = new PDO('mysql:host=localhost;dbname=my_db', 'my_user', 'my_password');
$query = $pdo->prepare('SELECT count(*) FROM `uni_chat_messages` WHERE `chat_messages_status`=:status AND `chat_messages_id_user`=:userId');
$query->execute([':status' => 1, ':userId' => $userId]);
$messageStatus = $query->fetch()[0];

[Икотий Айтишевич]

vdem, Да, спасибо. Только не пойму в чём прикол. Добавляю строку в запрос: AND `chat_messages_id_user`=:userId', выводит: 0. Убираю её выводит нормальное число...

[vdem]

Ноль это тоже вполне нормальное число. Если выводит 0, значит у такого юзера нет записей в uni_chat_messages со статусом 1.

[Икотий Айтишевич]

vdem, Да, согласен. Только вот у такого юзера есть записи со статусом 1.

[vdem]

Пробовали этот запрос запустить в MySQL вручную, минуя PHP? С тем же статусом и userId?

[Икотий Айтишевич]

vdem, Нет в MySQL выдаёт нужный результат. Но сейчас уже нашёл ошибку. Спасибо Вам за помощь.

Answer 2

[toxa82]

Кому-то лень сделать var_dump переменной.

Answer 3

[anna_makeenko]

Внутри то, что возвращает getOne, а не count