запрос с php

[Sigma]

вобщем такой кусок кода
@ $db = mysql_pconnect("localhost");
   if (!$db)
   {
      echo "Не могу подключиться к базе";
      exit;
   }
mysql_select_db("rieltr");
$query = "insert into rieltr.mainrielt (name, action, object, metro, price, email, phone, comment ) values ( ".$name." , ".$action." , ".$object.", ".$metro.", ".$price.", ".$email.", ".$phone.", ".$comm.")";
$result = mysql_query($query);
if ($result)
   echo mysql_affected_rows ()." запись добавлена в таблицу.";


где тут косяк ибо в базу ничё не добавляется ... ? или вообще нужны какие-то доп установки ( сижу в winXP, mysql и apache запущены ).

[Макс]

1. убери собаку @ (ты, судя по коду, еще не такой профи, чтобы писать код с использованием собак)
2. строку: $result = mysql_query($query); замени пока на
$result = mysql_query($query) or die (mysql_error().\'
\'.$query); и сообщи какие ошибки выводит

[Sigma]

заменила, теперь выдаёт
Unknown column \'Вася\' in \'field list\'
и дальше весь запрос как он есть ...

[Меняздесьдавнонет]

значения полей в запросе ОБЯЗАТЕЛЬНО должны быть взяты в кавычки
подробнее http://faq.phpclub.net/slashes

[Sigma]

а у меня разве не кавычках ?

[Меняздесьдавнонет]

нет.
выведи свой запрос на экран.

[Sigma]

ага теперь все работает ... спасибо.
А можно подробнее вот про @ в
@ $db = mysql_pconnect("localhost");

что в ней особенного то ?

[Меняздесьдавнонет]

До чего же ты, Сигма, приятнее одного клиента, с которым я работал только что на другом форуме.
Лепит вообще не понимая смысла ни отдельных функций, ни целых конструкций.
И в голову не приходит хотя бы спросить - для его они нужны.

Я, как раз собирался сам об этом написать, но потом передумал.
уж очень трудно людей уговорить делать правильно.

а эта строка все, сплошь неправильная, не только собака :-)
правильно данная конструкция должна выглядеть так:
mysql_connect() or die mysql_error();

теперь по пунктам:
@ - запрещение вывода сообщений об ошибках.
очень вредный оператор. сообщения об ошибках - это хлеб программиста. Без них он не может и шагу ступить.
А применяют эту собаку совсем не по назначению.
чтобы юзер не видел сообщения об ошибке.
Это то же самое, что лечить головную боль гильотиной.
Не хочешь, чтобы юзер сообщение видел? Запрети ему это, например, директивой display_errors.
Но совсем не подавляй! Самому же понадобится.

дальше. $db=
переменной $db присваивается значение идентификатора соединения.
в 99% случаев эта переменная не нужна, и ее можно опустить. Не присваивать вовсе. У тебя в скрипте, к примеру, она не используется.

mysql_pconnect
эта функция стоит в первой десятке проблем пхп скриптов.
она устанавливает соединение с mysql но не закрывает его. очень быстро лимит соединений исчерпываетя и скрипты перестают работать.
вместо нее надо использовать mysql_connect

"localhost" - это адрес сервера.
если это "localhost", то указывать это не надо, это значение по умолчанию, РНР сам его подставит.

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

[Меняздесьдавнонет]

внимательный читатель может заметить, что die(mysql_error()) выводит сообщение об ошибке пользователю, о вреде какового действия мы говорили ранее.
Такая конструкция приведена исключительно для примера.
На рабочем сервере желательно ошибку выводить не на экран, а в лог-файл, да и работу программы завершать не аварийно, а покрасивее.

[Sigma]

ну просто лучше сразу разобраться, чтобы потом не доставать всех глупыми вопросами спасибо.

[Меняздесьдавнонет]

вопросы твои совсем неглупые.
Уж поверь моему опыту, бывают и глупее.
Так что всегда рады ответить.

Если тебе интересно, я могу и весь приведенный код разобрать

вот эта, к примеру, конструкция,

@ $db = mysql_pconnect("localhost");
if (!$db)
{
echo "Не могу подключиться к базе";
exit;
}

заменяется на одну строчку
mysql_pconnect("localhost") or die("Не могу подключиться к базе");

Ну, про mysql_error и сообщения для юзеров я уже писал.

mysql_select_db имеет столько же шансов не выполниться, сколько и mysql_connect, и поэтому ее надо оформлять так же.

однако забавно, что в твоем примере можно обойтись и без нее совсем, поскольку название БД ты пишешь в запросе.
Чтобы не усложнять код, лучше сделать что-то одно.
традиционно используется функция, а в запросе БД не пишется.

Дальше - запрос.
если в базу вставляются все поля, то перечислять их не нужно, можно использовать краткую запись.
Вообще, про запрос ты можешь почитать в соседней ветке.
Я там объясняю принципы построения запросов и неплохую функцию привел в самом последнем посте.
http://forums.webscript.ru/showthread.php?s=&threadid=15268

далее.
если mysql_connect при неудаче выдает причину, то mysql_query - НЕТ.
И если выводить mysql_error после первой можно по желанию, то после второй - обязательно.
Эта функция тебе выдаст ошибку в запросе.
Ну, тебе уже написал об этом Макс.

mysql_affected_rows () опять же здесь бессмысленно.
если квери прошла нормально, то и так можно сказать, что вставлена одна запись. совсем не нужно зря дергать функцию.

а вот текст после вставки в базу выдавать никакой не надо.
тому есть две взаимосвязанные причины.
1. Если в результате запроса пользователя меняются данные в базе, то такой запрос надо делать метобом ПОСТ.
в частности, это гарантия того, что на кнопку не нажмет робот.
2. Если данные были отправлены методом ПОСТ, то надо после него, не выводя ничего в браузер, перенаправить юзера методом ГЕТ.
об этом подробно написано тут:
http://phpclub.ru/talk/showthread.php?s=&threadid=43521