Почему данные из базы данных не сортируются в PHP PDO?

Question

[alexandreyakuboff]

Делал чат, для этого создал таблицу в MySQL, которая называется messages (имеет 4 столбца: ID, user1, user2 и message) и подключился к ней через pdo query:

$messages = $pdo->query('select * from messages where (user1="'.$_SESSION['user_name'].'" and user2="'.$_POST['user2'].'") or (user2="'.$_SESSION['user_name'].'" and user1="'.$_POST['user2'].'") ORDER by id');
                    while($message = $messages->fetch()){
                        if($message['user1'] == $_SESSION['user_name']){
                            echo "
                            <div id='message'>".$_SESSION['user_name'].": ".$message['message']."</div>
                            ";
                        }
                        if($message['user2'] == $_SESSION['user_name']){
                            echo "
                            <div id='message'>".$_POST['user2'].": ".$message['message']."</div>
                            ";
                        }

Но примерно на 20 сообщении сортировка нарушается. Почему?

Comments

[Akina]

сортировка нарушается.

Как именно? и как именно это определено? Подробности в студию. Например, вместо [censored] "создал таблицу в MySQL, которая называется messages (имеет 4 столбца: ID, user1, user2 и message)" выложите полный вывод SHOW CREATE TABLE messages;

[alexandreyakuboff]

Akina, я делал через PHPMyAdmin, но вот скрин:
Там расставлено правильно, но на самом сайте перемешано примерно на 20 сообщении.

[Rsa97]

Как именно нарушается сортировка? После id 20 идёт id 19? Или у вас какое-то своё, нестандартное понимание сортировки?

Answer 1

[YepBro]

1) У тебя закрывающаяся фигурная скобка пропущена в приведенном коде.

2) Запрос правильный, от старых к новым.

3) Не забывай про защиту от SQL-инъекций, подставляй данные через плейсхолдеры

4) И самое главное внимательно посмотри на свои if. Ты всегда выводишь реплики только одного участника чата.

Comments

[Антон Шаманов]

2) Запрос правильный, от старых к новым.

от новых к старым т.к. там ASC, а в остальном согласен

[YepBro]

Антон Шаманов,

от новых к старым

при всем моем уважение все же ASC (ascending - восходящий) это от меньшего к большему, а в нашем случае это от старых к новым, потому что новые записи будут иметь больший id.

[Антон Шаманов]

YepBro, а! в моей голове это звучало по другому