PHP / MySQL Insert [дубликаты]

Question

PHP / MySQL Insert [дубликаты]

0

Хорошо, я использовал PHP + MySQL некоторое время, поэтому считаю себя опытным. Я сделал свою долю синтаксических ошибок в прошлом, но это честно меня раздражает:

http://img251.imageshack.us/img251/3760/fubar.png

Если кто-нибудь скажет мне, почему это простое выражение не работает, я был бы очень благодарен.

Glenn Nelson 18 сен. 2010, в 22:29

Источник

0

на первый взгляд, в этом нет ничего плохого ... какую ошибку вы на самом деле получаете? Вы используете переменные вместо статического значения?
Crayon Violent 18 сен. 2010, в 20:32
0

Определить «не работает». Вы использовали mysql_error ()? Что это говорит?
halfdan 18 сен. 2010, в 20:32
0

Ничего не произошло. Там нет ошибок. Ничего не добавлено в базу данных.
user372743 18 сен. 2010, в 20:34
0

что mysql_query() ?
Jake N 18 сен. 2010, в 20:36

Показать ещё 2 комментария

Теги:

php

mysql

insert

3 ответа

3

Используйте обратные ссылки для опции.

INSERT INTO poll (`Option`) VALUES ('Stuff')

vassilis 18 сен. 2010, в 17:36

0

Или просто сделайте INSERT INTO poll VALUES ('Stuff') . Нет необходимости указывать имя столбца.
Mark Snidovich 18 сен. 2010, в 20:39
0

@ Марк Снидович - это верно только в том случае, если вы планируете указывать значения для каждого столбца (и в правильном порядке столбцов). В общем, это не очень хорошая практика, потому что если вы решите добавить еще один столбец в таблицу позже, для какого-то другого сценария ... это нарушит это утверждение.
Crayon Violent 18 сен. 2010, в 20:43
0

@Crayon это правда, но в этом случае таблица имеет только один столбец. Обычно я так и делаю - задаю значение для каждого столбца, и если я изменю таблицу, лучше быть осторожным. Хорошие моменты, хотя, я думаю, это зависит от того, хотите ли вы обеспечить отказоустойчивость, или вы будете вынуждены быть точными в указанных значениях.
Mark Snidovich 19 сен. 2010, в 02:18

Показать ещё 1 комментарий

2

Глядя на код, который вы пытаетесь вставить в $_POST ['survey'], ваша вставка должна выглядеть так:

$vote = $_POST['survey'];

// connect to db

mysql_query(sprintf(
    "INSERT INTO poll (`Option`) VALUES ('%s')", 
    mysql_real_escape_string($vote)
);

Также обратите внимание, что "опция" является зарезервированным ключевым словом и должна находиться внутри обратных ссылок.

halfdan 18 сен. 2010, в 18:54

0

Вы забыли sprintf :)
Fanis 18 сен. 2010, в 20:42
0

@ Фанис: Боже, какой провал :-( Исправлено.
halfdan 18 сен. 2010, в 21:08

Ещё вопросы

на первый взгляд, в этом нет ничего плохого ... какую ошибку вы на самом деле получаете? Вы используете переменные вместо статического значения?
Определить «не работает». Вы использовали mysql_error ()? Что это говорит?
Ничего не произошло. Там нет ошибок. Ничего не добавлено в базу данных.
Или просто сделайте INSERT INTO poll VALUES ('Stuff') . Нет необходимости указывать имя столбца.
@ Марк Снидович - это верно только в том случае, если вы планируете указывать значения для каждого столбца (и в правильном порядке столбцов). В общем, это не очень хорошая практика, потому что если вы решите добавить еще один столбец в таблицу позже, для какого-то другого сценария ... это нарушит это утверждение.
@Crayon это правда, но в этом случае таблица имеет только один столбец. Обычно я так и делаю - задаю значение для каждого столбца, и если я изменю таблицу, лучше быть осторожным. Хорошие моменты, хотя, я думаю, это зависит от того, хотите ли вы обеспечить отказоустойчивость, или вы будете вынуждены быть точными в указанных значениях.
@ Фанис: Боже, какой провал :-( Исправлено.

Crayon Violent · Accepted Answer · 2010-09-18T18-04-00.000Z

На самом деле я вижу 1 ошибку... "Вариант" - зарезервированное слово. оберните его в backtics: `Option` или еще лучше, измените имя столбца на то, что не зарезервированное слово.