Запрос на обновление PHP/MYSQL не работает

Question

Может ли кто-нибудь сказать, почему этот запрос обновления не работает?

if ($_GET['update']) { 
include 'config.php'; 
//Connect to MYSQL Database server 
$connect = mysql_connect(DB_HOST, DB_USER, DB_PASS) or die("Could not connect to MYSQL Database."); 
$result = mysql_select_db(DB_NAME, $connect) or die("Could not connect to MYSQL table."); 

mysql_query("UPDATE contact SET read = 1 WHERE id = '$_GET[update]'")or die("Query failed."); 
echo "Update works!"; 
} else { 
echo "Update does not work...ughh."; 
} 

Заранее спасибо.

Редактировать: Я получил запрос на работу. Для тех, кто беспокоился о безопасности, я использовал этот скрипт в качестве теста, чтобы узнать, хочу ли я его использовать. Я просто добавил безопасность теперь, когда скрипт работает. Спасибо всем за помощь и советы.

Answer 1

Что такое колонка, прочитанная?

mysql_query("UPDATE contact SET read = 1 WHERE id = '$_GET[update]'") 

Судя по отсутствию капитализации читать, я подозреваю, что вы используете зарезервированное слово в MySQL для этого столбца.

См:

Reserved Words in MySQL

Чтобы обойти эту проблему, просто поставить апостроф вокруг чтения. И.Е.

mysql_query("UPDATE contact SET 'read' = 1 WHERE id = '$_GET[update]'") 

Или лучше на j.bruni:

mysql_query("UPDATE contact SET `read` = 1 WHERE id = '$_GET[update]'") 

Answer 2

Вы можете проверить так

mysql_query("UPDATE contact SET read = 1 WHERE id = '".(int)$_GET['update']."'")or die("Query failed."); 

если не эта проблема конкретных

Answer 3

Я считаю, что вам нужно бежать строку, чтобы иметь $ _GET [ «обновление»], чтобы добавить его значение в строка. Но вы действительно должны использовать подготовленные инструкции, в которых вы атакованы злонамеренными пользователями.

Подготовленные отчеты: http://php.net/manual/en/pdo.prepared-statements.php

Answer 4

Попробуйте для линии запроса:

mysql_query("UPDATE contact SET read = 1 WHERE id = '".$_GET[update]."'")or die("Query failed: " . mysql_error()); 

Обратите внимание на изменение die() заявления для лучшей обработки ошибок:

die("Query failed: " . mysql_error()); 

* Кроме того, просто FYI, вы должны действительно избегать пользовательских переменных (например, переменных GET), чтобы избежать инъекций SQL:

mysql_query("UPDATE contact SET read = 1 WHERE id = '".mysql_real_escape_string($_GET[update])."'")or die("Query failed: " . mysql_error()); 

Пожалуйста, сообщите результат.

Answer 5

READ - зарезервированное слово. Вы должны поместить его в backticks или переименовать свое поле.

Посмотрите на эту ссылку:

http://dev.mysql.com/doc/refman/5.5/en/reserved-words.html

Answer 6

mysql_query("UPDATE contact SET read = 1 WHERE id = '.$_GET[update].'")or die("Query failed."); 
echo "Update works! 

Пожалуйста, попробуйте не использовать mysql_query. Он старый, и он не эффективен. почему бы не попытаться узнать о PDO и подготовить заявления ..?