Показать сообщение отдельно
  #8 (permalink)  
Старый 26.02.2018, 16:01
Профессор
Отправить личное сообщение для laimas Посмотреть профиль Найти все сообщения от laimas
 
Регистрация: 14.01.2015
Сообщений: 12,990

Сообщение от SIGTTD
А в цикле просто записи выцепляются по очереди, криво сделано?
И криво, и просто ужасно. Во-первых $i >= 0 просто не может быть, если id, это уникальные значения формируемые автоматически полем с автоинкрементом - никогда не будет сформирован ID со значением 0.

Во-вторых это полнейшая глупость, ибо записи могут удаляться, а будучи уникальным значением они повторно не будут использованы, то есть запрос id=45 может ничего и не вернуть.

В третьих, это насилование базы, а не выборка из нее. Если необходимо получить 300 записей, значит:

mysql_query("SELECT * FROM ".$userName." ORDER BY id LIMIT 300")

и уже результат выборки в цикле и помещать в нужное.

И or die(mysql_error()); - это тоже плохо. Ошибки нужно выводить по условию - режим отладки? вывод, иначе что-то, но не mysql_error().

<form id='editForm' - такого быть не должно, вам уже говорилось, что id должно быть уникальным. Да и не нужно столько форм, форма должна быть одна.

Последний раз редактировалось laimas, 27.02.2018 в 07:16.
Ответить с цитированием