Система комментариев PHP с ответом 1 уровня. Проблема с отображением запроса

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

Первый вариант – сохранить ваш запрос таким, какой он есть, но всякий раз, когда вы печатаете строку, вы смотрите на сообщение, которое вы печатаете. Вы сохранили бы этот последний отпечатанный идентификатор сообщения, и если текущий идентификатор совпадает, вы просто не печатаете сообщение, а только комментарий. Для этого, очевидно, потребуется разделить разметку для сообщения и комментария (что в любом случае является хорошей идеей). В основном это было бы так:

 $lastPost = -1; while ($query_row = mysql_fetch_assoc($query_run)) { // ... if ($pid != $lastPost) { echo "<code for the post>"; $lastPost = $pid; } echo "<code for the comment>"; } 

Другая идея заключалась бы в том, чтобы разделить ваш запрос, загрузив все сообщения, а затем для каждого опубликованного вами сообщения загрузить комментарии и распечатать их все. Это, очевидно, требует n+1 SQL-запросов для n сообщений, но это уменьшает объем данных, поступающих из вашей базы данных (поскольку у вас больше нет дубликатов сообщения, следующего за каждым комментарием). Если у вас есть страница для отображения только одного сообщения, это позволит вам снова использовать код. Обычно это выглядит так:

 $query_run = // query only for posts, ignoring comments while ($query_row = mysql_fetch_assoc($query_run)) { // ... echo "<code for the post>"; $comment_query = // query only for comments with post_id = $pid while ($comment_row = mysql_fetch_assoc($comment_query)) { // .. echo "<code for the comment>"; } } 

Обновить

У вас есть несколько способов включить информацию в форму, которая затем передается на целевую страницу. Можно было бы включить их как параметры GET в URL-адрес, как вы сами пробовали. Другой будет включать скрытые элементы ввода, которые просто удерживают ваши статические данные. Независимо от того, что вы используете, вам нужно будет получить эти значения из $_GET или $_POST на целевой странице. После некоторой проверки ввода (которую вы должны сделать) вы можете использовать эти значения для вставки комментария так же, как обычно, с одной формой, за исключением того, что идентификатор сообщения также поступает из переменной запроса. Если вы не знаете, как правильно обращаться с вводом пользователя и создать комментарий, я предлагаю вам начать немного медленнее и просто создать одну форму комментариев, чтобы узнать, как это работает.

Ошибка «неопределенного индекса» означает, что вы пытаетесь получить доступ к индексу в массиве (возможно, $query_row ), которого не существует. Вы можете получить доступ только к тем значениям, которые вы действительно запросили в своем sql-запросе. Поэтому проверьте, действительно ли переменная, которую вы пытаетесь получить, в запросе.

Это то, что я ожидал бы, потому что ваш SQL-запрос является SELECT из двух таблиц, которые являются внутренним соединением, поэтому вы получите дублирование строк.

Попробуйте выполнить запрос непосредственно в окне запроса mysql чтобы узнать, что выйдет.

Я бы рекомендовал хранить значение текущего сообщения и выводить комментарии только тогда, когда сообщение не изменилось; что-то вроде следующего:

Также вы не записываете идентификатор сообщения в форме (либо в виде скрытой переменной, либо в URL-адрес), поэтому я добавил скрытую переменную, поскольку не подозреваю, что post_id станет вашей следующей ошибкой.

  $last_post_id = null; while ($query_row = mysql_fetch_assoc($query_run)){ $post_id = $query_row['post_id']; if ($last_post_id == $post_id) { ?> <ul class=\"comment\"> $commenter_user_id <li> $comment </li> </ul> <?php } else { if ($last_post_id !== null) { ?> <form name=\"message\" method=\"post\" action=\"sendmessage.php\"> <textarea name=\"message\"></textarea> <br> <input type=\"hidden\" name="\post_id\" value=\"$last_post_id\" /> </form> <input type=\"submit\" value=\"Send\" /></form> <?php $last_post_id = $post_id; } ?> <div id=\"post\"> Post $pid $post_id Poster: $poster_id Mentions: $user_id <br><br> $message <br><br> <ul class=\"comment\"> $commenter_user_id <li> $comment </li> </ul> </div> <?php } ?> 

Обновленный ответ

в sendmessage.php (используя мою форму выше с типом ввода = скрытый), см. ниже. Вы правы, что вам нужно будет использовать сеансы для получения текущего идентификатора пользователя для целей публикации.

Я бы рекомендовал использовать метод POST для отправки любой формы, которая приводит к изменению данных.

 <?php if (isset($_POST['post_id'])) { $post_id = mysql_real_escape_string($_POST['post_id']); $user_id = $_SESSION['current_user_id']; $sql = "INSERT INTO `comments`SET `post_id` = $post_id 'commenter_user_id = $user_id"; if (!mysql_query($sql)) echo "MySQL error during insert ". mysql_error(); } else { echo "Invalid form posting"; } 

Как насчет рекурсии? Соответствует ли это вашим потребностям? Если да, то все, что вам нужно, это идентификатор родительского сообщения для каждого сообщения (или NULL, если это не ответ). Затем вы загружаете все сообщения и вызываете эту функцию только с помощью сообщений NULL, также предоставляя весь массив сообщений.