Как побороть ошибку?

Question

[Jun ior]

Имеется код:

main.php

<?php
if(isset($_POST['getNewId'])){
  loadNewPosts();
}
function loadNewPosts(){
  include 'db-config.php';
  $new = $_POST['getNewId'];
  $newId = mysql_fetch_array(mysql_query("SELECT * FROM posts ORDER BY id DESC"));
  if($newId>$new){
  if($new==0){
    $post1 = mysql_query("SELECT * FROM posts ORDER BY id DESC LIMIT 10");
  }else{
    $post1 = mysql_query("SELECT * FROM posts ORDER BY id DESC LIMIT $new , $newId");
  }
  $post = mysql_fetch_array($post1);
    do{
      print(json_encode(array('post'=>'<div class="area"><div class="pad-5">'.$post['content'].'</div></div>','id'=>$newId)));
    }while($post=mysql_fetch_array($post1));
  }else{
    print(json_encode(array('id'=>$new)));
  }
}

main.js

setInterval(loadNewPosts,5000);
var newId = 0;
function loadNewPosts(){
$.ajax({
  url:SiteURL+'assets/includes/functions/main.php',
  type:'POST',
  data:{'getNewId':newId},
  success:function(text){
    text = JSON.parse(text);
    if(text!=''){
    $('.postContainer').prepend(text.post);
    newId = text.id;
  }else{
    alert(text);
  }
  }
});
}

После выполнения кода воводится ошибка в консоли JavaScript:

Uncaught SyntaxError: Unexpected token < in JSON at position 0
at JSON.parse ()

А вот и json ответ от php:

<!--error--><br />
<b>Warning</b>:  mysql_fetch_array() expects parameter 1 to be resource, boolean given in <b>Z:\home\mind\www\assets\includes\functions\main.php</b> on line <b>17</b><br />
<script language=JavaScript src='/denwer/errors/phperror_js.php'></script>{"post":"<div class=\"area\"><div class=\"pad-5\"><\/div><\/div>","id":{"0":"48","id":"48","1":"8","poster_id":"8","2":"\u041a\u0430\u043a \u0443 \u0432\u0430\u0441 \u0434\u0435\u043b\u0430?","content":"\u041a\u0430\u043a \u0443 \u0432\u0430\u0441 \u0434\u0435\u043b\u0430?"}}<!--error--><br />
<b>Warning</b>:  mysql_fetch_array() expects parameter 1 to be resource, boolean given in <b>Z:\home\mind\www\assets\includes\functions\main.php</b> on line <b>20</b><br />
<script language=JavaScript src='/denwer/errors/phperror_js.php'></script>

Comments

[Дмитрий]

Добрый день.
Исправьте ошибку в sql запросе.

[Дмитрий]

И избавьтесь от цикла do{}while();

[Дмитрий]

Эту строку я бы изменил

json_encode(array('post'=>'<div class="area"><div class="pad-5">'.$post['content'].'</div></div>','id'=>$newId)));

на

json_encode(['post'=> $post['content'],'id'=>$newId]));

И в html помещал бы уже на клиенте.

[Jun ior]

slo_nik, Но как тогда выводить записи из БД

[Дмитрий]

IT Coder, в смысле "как выводить записи из БД"?

[Jun ior]

slo_nik, do{}while() нужен для того чтобы выводить 10 записей из БД

[Дмитрий]

IT Coder, используйте просто while(), do{}while() в подобных ситуациях используется настолько редко, что я даже не вспомню, где подобное видел.
Из документации:

Цикл do-while очень похож на цикл while, с тем отличием, что истинность выражения проверяется в конце итерации, а не в начале. Главное отличие от обычного цикла while в том, что первая итерация цикла do-while гарантированно выполнится (истинность выражения проверяется в конце итерации), тогда как она может не выполниться в обычном цикле while (истинность выражения которого проверяется в начале выполнения каждой итерации, и если изначально имеет значение FALSE, то выполнение цикла будет прервано сразу).

[Дмитрий]

IT Coder, ведь в ошибке же у Вас сказано

mysql_fetch_array() expects parameter 1 to be resource, boolean given in

И ещё, не используйте mysql, используйте хотя бы mysqli.
Опять же из документации:

Внимание
Данное расширение устарело, начиная с версии PHP 5.5.0, и удалено в PHP 7.0.0. Используйте вместо него MySQLi или PDO_MySQL. Смотрите также инструкцию MySQL: выбор API и соответствующий FAQ для получения более подробной информации. Альтернативы для данной функции:
mysqli_fetch_array()
PDOStatement::fetch()

[Jun ior]

slo_nik, у меня на Denwer php 5.3

[Jun ior]

slo_nik , ошибка вышла в консоли пишет :

VM347:1 Uncaught SyntaxError: Unexpected token { in JSON at position 328
at JSON.parse ()

А JSON :

{"post":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?<\/div><\/div>","id":{"0":"49","id":"49","1":"8","poster_id":"8","2":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?","content":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?"}}{"post":"\u041a\u0430\u043a \u0443 \u0432\u0430\u0441 \u0434\u0435\u043b\u0430?<\/div><\/div>","id":{"0":"49","id":"49","1":"8","poster_id":"8","2":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?","content":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?"}}{"post":"\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\r\n\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430\u0430<\/div><\/div>","id":{"0":"49","id":"49","1":"8","poster_id":"8","2":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?","content":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?"}}{"post":"\u044b\u044b\u044b\u044b\u044b<\/div><\/div>","id":{"0":"49","id":"49","1":"8","poster_id":"8","2":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?","content":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?"}}{"post":"\u044b\u044b\u044b\u044b\u044b<\/div><\/div>","id":{"0":"49","id":"49","1":"8","poster_id":"8","2":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?","content":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?"}}{"post":"\u044b\u044b\u044b\u044b<\/div><\/div>","id":{"0":"49","id":"49","1":"8","poster_id":"8","2":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?","content":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?"}}{"post":"\u0446\u0446\u0446\u0446<\/div><\/div>","id":{"0":"49","id":"49","1":"8","poster_id":"8","2":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?","content":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?"}}{"post":"\u0432\u0432\u044b\u044b\u044b\u044b\u044b\u044b\u044b\u044b\u044b<\/div><\/div>","id":{"0":"49","id":"49","1":"8","poster_id":"8","2":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?","content":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?"}}{"post":"\u0432\u0432\u044b\u044b\u044b\u044b\u044b<\/div><\/div>","id":{"0":"49","id":"49","1":"8","poster_id":"8","2":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?","content":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?"}}{"post":"\u0432\u0432<\/div><\/div>","id":{"0":"49","id":"49","1":"8","poster_id":"8","2":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?","content":"\u041d\u043e\u0440\u043c, \u0441\u0430\u043c \u043a\u0430\u043a?"}}

[Дмитрий]

у меня на Denwer php 5.3

IT Coder, давно пора забыть как страшный сон. Если хотите заниматься разработкой - ставьте нормальные окружение.

[Дмитрий]

IT Coder, если есть ошибка - исправляйте.

[Jun ior]

slo_nik, Но как обновить версию php?
И я уже 2 дня пытаюсь решить проблему но ошибку найти не могу

[Дмитрий]

IT Coder, выведите в консоль ответ сервера и посмотрите что и как.
Возможно не надо использовать JSON.parse()

[Jun ior]

slo_nik, Но я же передаю переменную id и контент поста

[Дмитрий]

IT Coder, удалите "Denwer" полностью. Ставьте другой сервер, например xamp. Можно всё по отдельности поставить, потом разобраться с настройками... Но "Denwer" забыть однозначно!!!

[Jun ior]

slo_nik , Когда меняю вот это:

if($new==0){
    $post1 = mysql_query("SELECT * FROM posts ORDER BY id DESC LIMIT 10");
  }else{
    $post1 = mysql_query("SELECT * FROM posts ORDER BY id DESC LIMIT $new , $newId");
  }

На это:

if($new==0){
    $post1 = mysql_query("SELECT * FROM posts ORDER BY id DESC LIMIT 1");
  }else{
    $post1 = mysql_query("SELECT * FROM posts ORDER BY id DESC LIMIT 1");
  }

Ошибка ищезает и загружаются новые новости но при загрузке на странице выводится только одна новость

[Jun ior]

slo_nik, Хорошо

[Дмитрий]

Но я же передаю переменную id и контент поста

IT Coder, ну и что? Возможно достаточно будет циклом пройтись и всё разобрать.

[Jun ior]

slo_nik, Эта переменная нужна для того чтобы понять какая запись уже загружена, а какая нет

[Дмитрий]

Ошибка ищезает и загружаются новые новости но при загрузке на странице выводится только одна новость

IT Coder, если иСЧЕЗАЕТ, то это хорошо, а одна новость потому что в запросе указали через LIMIT, что надо выводить всего одну новость.

[Jun ior]

slo_nik , Когда меняю вот это:

if($new==0){
    $post1 = mysql_query("SELECT * FROM posts ORDER BY id DESC LIMIT 10");
  }else{
    $post1 = mysql_query("SELECT * FROM posts ORDER BY id DESC LIMIT $new , $newId");
  }

На это:

if($new==0){
    $post1 = mysql_query("SELECT * FROM posts ORDER BY id DESC LIMIT 1");
  }else{
    $post1 = mysql_query("SELECT * FROM posts ORDER BY id DESC LIMIT 1");
  }

Ошибка ищезает и загружаются новые новости но при загрузке на странице выводится только одна новость

[Дмитрий]

Эта переменная нужна для того чтобы понять какая запись уже загружена, а какая нет

IT Coder, да пусть будет. Я же не говорю, что надо удалить эту переменную.
Просто разбирать при помощи JSON.parse может не понадобиться.

Answer 1

[Андрей Юмашев]

1. Не используйте устаревшие функции mysql_*.

php.net/mysqli_fetch_array

2. У вас нет коннекта к базе - в функцию передаётся не ресурс, а false. Предварительно проверьте - есть ли у вас соединение с базой. Вернулся ли корректный ответ после mysqli_query.

3. Не забывайте экранировать приходящие данные - хотя бы mysqli_real_escape_string.

Comments

[Jun ior]

Спасибо, но проблема уже решена.
И подключение к бд имеется.