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

Question

[Арман М]

При выборке записей из базы выдает вот такую ошибку:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'desc,date,author,mini_img,view FROM data WHERE cat='1'' at line 1

Где ошибка не понимаю

<?php 
		
echo $myrow["text"];
		
$result = mysql_query("SELECT id,title,desc,date,author,mini_img,view FROM data WHERE cat='$cat'", $db);

if (!$result)
{
echo "<p>Запрос на выборку данных из базы не прошел. Напишите об этом администратору [e-mail]. <br>Код ошибки:</p>";
exit(mysql_error());
}

if (mysql_num_rows($result) > 0)
{
	$myrow = mysql_fetch_array($result);

	do
	{
	printf ("<table border='0'>
            			<tr>
            				<td>
								<p><a href='view_post.php?id=%s'>%s</a></p>
								<p>Дата добавления: %s</p>
								<p>Автор: %s</p>
							</td>
            			</tr>
						
            			<tr>
              				<td>%s<br>Просмотров: %s</td>
            			</tr>
          			</table><br><br>", $myrow['id'], $myrow['title'], $myrow['date'], $myrow['author'], $myrow['desc'], $myrow['view']);
	}
	while ($myrow = mysql_fetch_array($result));
}

else
{
	echo "<p>Информация по запросу не может быть извлечена. В таблице нет записей</p>";	
	exit();
}
		
?>

Answer 1

[Макс]

на desc у Вас ругается. Лучше всего поменяйте на descr(и в таблице, и в запросе), чтобы не парится с экранированием. Ну а совет выкинуть книгу и перестать использовать mysql_ - 100% в яблочко.

Comments

[Арман М]

Спасибо за помощь, попробую. Ну а по советам, я согласен, но аналогов не нашел. Все что не пробовал все сложно для изучения. А попов объясняет самые азы. Только изучив это могу начать изучать более сложные курсы.

[Макс]

Арман Марабян: качните курс от Специалиста. Мне очень понравилось. У них-же отличный курс по MySQL. А к плохому лучше сразу не привыкать.

[Арман М]

так mysql использовать или все таки на PostgreSQL ?

[Макс]

Арман Марабян: БД используйте - какая больше нравится. Если будете использовать MySQL, то перейдите с интерфейса mysql_ на mysqli_. Первый в новых версиях PHP уже просто не работает. погуглите PHP mysql mysqli. Куча статей на русском.

Answer 2

[Melkij]

https://dev.mysql.com/doc/refman/5.6/en/keywords.html

Comments

[Арман М]

Поясни пожалуйста если не сложно, с английским пока плохо все))

[Melkij]

Переводчик в помощь. Заголовок раздела перевести сможет любой переводчик. А для поиска своей ошибки в таблице переводить не надо вовсе.

И выкидывайте своего попова и mysql_* в целом. Первый говнокод, а второй уже не то что deprecated, а удалён из кодовой базы PHP.

[Арман М]

ну а как ещё осваивать самостоятельно php? Я только начинаю.

[Melkij]

Как-как - как и всё остальное: самостоятельно.

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

[Арман М]

я к сожалению не могу научиться по оф мануалам

[Арман М]

что использовать вместо mysql? MySQLi или PostgreSQL ?

[Melkij]

Не путайте расширения PHP для доступа к СУБД с самими СУБД.
Вместо mysql_* использовать PDO или mysqli. Не суть, какое именно расширение. Мне больше PDO по нраву, API удобнее.

Использовать ли mysql или postgresql - познакомьтесь с обеими СУБД.

[Арман М]

Понял, спасибо!

Answer 3

[Руслан Федосеев]

не используйте служебные слова в качестве названий таблиц, полей..
в данном случае desc

Answer 4

[Memorivardo]

Подобные ошибки чаще всего связаны с неверным формированием запроса. Конкретно в Вашем случае, ошибка скорее всего вызвана тем, что в запросе присутствуют слова "desc", "date" и "view", которые являются ключевыми для MySQL. Вы можете заэкранировать их используя косые кавычки: `desc`, `date`,`view` и т.д.

А вобще лучше сразу в сторону какого-нибудь PDO смотрите, там вопросы по экранированию за Вас продумали и с инъекциями в будущем будет проблем чуточку поменьше. Вот здесь вроде бы неплохо расписано всё по нему: phpfaq.ru/pdo

Comments

[Арман М]

спасибо, но я только только начинаю изучать php. А что такое PDO даже незнаю)

Answer 5

[65536]

Напишите об этом администратору [e-mail]