Форум MySQL

Есть запрос:

$query = "INSERT INTO price_product SET     id_product = NULL,     stock = '".$stock."',     name = '".$name."',     colvo = ".$colvo.",     description ='".$description."',     vp = ".$vp.",     base_price = ".$base_price.",     base_vozn = ".$base_vozn.",     price_50 = ".$price_50.",     price_42 = ".$price_42.",     price_35 = ".$price_35.",     price_25 = ".$price_25.",     price_100 = ".$price_100.",     pos = 0,     hide = '".$showhide."',     id_catalog = ".$_POST['id_catalog'].",     url = '".$url."'";

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

Если заполняю форму полностью, все ОК, данные добавляются.
Если форма заполнена не полностью (переменным присваиваются значения по умолчанию) - и тут вылетает ошибка:

1064: 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 ' base_vozn = , price_50 = , price_42 = , price_35 = , price_25 = , p' at line 8


Что делать? Где касяк?

Вот часть кода, которая задает значения переменным по умолчанию, при не полностью зполненной форме:

// Устанавливаем значения для не обязательных к заполнению полей if (empty($_POST['colvo'])) $colvo = 1; if (empty($_POST['description'])) $description = "-"; if (empty($_POST['vp'])) $vp = 1; if (empty($_POST['base_price'])) $base_price = 1; if (empty($_POST['base_vozn'])) $base_vozn = 1; if (empty($_POST['price_50'])) $price_50 = 1; if (empty($_POST['price_42'])) $price_42 = 1; if (empty($_POST['price_35'])) $price_35 = 1; if (empty($_POST['price_25'])) $price_25 = 1; if (empty($_POST['url'])) $url = 1;

Подскажите где ошибка.....

а написать echo $query и самому посмотреть не проще?????

>Что делать? Где касяк?

Вам же MySQL сказал

>1064: 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 ' base_vozn = , price_50 = , price_42 = , price_35 = ,
>price_25 = , p' at line 8

Видимо эти поля у вас числового типа, а вы передаете пустоту, и естественно получаете ошибку.

Скорее всего, что данные "проскакивают" вашу проверку и $price_50 (например) остается пустым со всем вытекающим отсюда.

>а написать echo $query и самому посмотреть не проще?????
Кстати очень хороший совет. Я часще всего так ошибки и нахожу.

Про echo $query, я только что узнал. Спасибо!

А про то, что почему-то данные "проскакивают" проверку, действительно какой-то касяк. Почему так, и как это исправить?

Это вот то ты назвал проверкой ->
if(empty($_POST['num'])) $num=1; ........
и так далее?

напиши echo "<PRE>"; print_r($_POST);
Где нибудь в начале скрипта и посмотри что у тебя там...

>Это вот то ты назвал проверкой ->
>if(empty($_POST['num'])) $num=1; ........
>и так далее?
>
>напиши echo "&lt;PRE>"; print_r($_POST);
>Где нибудь в начале скрипта и посмотри что у тебя там...

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

to tAleks: Откуда берутся все эти $base_vozn, $price_50 и т.п. переменные?

Эти пеерменные передаются из формы POST.

Потом, если $_POST['base_vozn'] пустая, т.е. в форме не забили, то $base_vozn присваивается значение.

Приведите пож-та структуру таблицы price_product
будет гораздо проще разобраться.

>Приведите пож-та структуру таблицы price_product
>будет гораздо проще разобраться.

CREATE TABLE 'price_catalog' (
'id_catalog' int(8) NOT NULL auto_increment,
'name' tinytext,
'description' tinytext,
'pos' smallint(3) default '0',
'hide' enum('show','hide') default 'show',
'warning' enum('hide','show') default 'hide',
'url_warning' tinytext,
PRIMARY KEY ('id_catalog')
) ENGINE=MyISAM DEFAULT CHARSET=cp1251 AUTO_INCREMENT=12 ;

вы ошиблись - дали структуру таблицы 'price_catalog' а не price_product :(

>вы ошиблись - дали структуру таблицы 'price_catalog' а не
>price_product :(

Упс.... действительно ошибся.

CREATE TABLE 'price_product' (   'id_product' int(8) NOT NULL auto_increment,   'stock' char(4) default NULL,   'name' tinytext,   'colvo' smallint(6) default NULL,   'description' tinytext,   'vp' decimal(7,2) default NULL,   'base_price' decimal(7,2) default NULL,   'base_vozn' decimal(7,2) default NULL,   'price_50' decimal(7,2) default NULL,   'price_42' decimal(7,2) default NULL,   'price_35' decimal(7,2) default NULL,   'price_25' decimal(7,2) default NULL,   'price_100' decimal(7,2) default NULL,   'pos' smallint(3) default '0',   'hide' enum('show','hide') default 'show',   'id_catalog' int(8) default NULL,   'url' tinytext,   PRIMARY KEY  ('id_product'),   KEY 'id_catalog' ('id_catalog'),   KEY 'stock' ('stock') ) ENGINE=MyISAM DEFAULT CHARSET=cp1251;

ну первое что бросается в глаза - зачем вы делаете средствами PHP то что проще использовать в MySQL параметром DEFAULT

CREATE TABLE 'price_product' (    'id_product' int(8) NOT NULL auto_increment,    'stock' char(4) default NULL,    'name' tinytext,    'colvo' smallint(6) default NULL,    'description' tinytext,    'vp' decimal(7,2) NOT NULL default '1,00',   'base_price' decimal(7,2) NOT NULL default '1,00',    'base_vozn' decimal(7,2) NOT NULL default '1,00',   'price_50' decimal(7,2) NOT NULL default '1,00',    'price_42' decimal(7,2) NOT NULL default '1,00',    'price_35' decimal(7,2)  NOT NULL default '1,00',    'price_25' decimal(7,2)  NOT NULL default '1,00',    'price_100' decimal(7,2)  NOT NULL default '1,00',    'pos' smallint(3) default '0',    'hide' enum('show','hide') default 'show',    'id_catalog' int(8) default NULL,    'url' tinytext,    PRIMARY KEY  ('id_product'),    KEY 'id_catalog' ('id_catalog'),    KEY 'stock' ('stock')  ) ENGINE=MyISAM DEFAULT CHARSET=cp1251;

в этом случае вам не прийдется проверять необязательные для заполнения данные

и второе - необходимо проверить обязательно передается ли значение $_POST['id_catalog'], $stock - они у вас ключевые

А зачем нужны двойные кавычки в одинарных и точки?

stock = '".$stock."',

Это одна из строк вашего запроса к MYSQL

в данном случае у него по видимому одинарные кавычки для записи значений текстовых полей в БД а двойные для входа в PHP - хотя мне кажетя было бы проще писать так

stock = '$stock',

и тот и другой вариант будет работать

А что в этом плохого? В любом случае, этот стиль написания ошибку не вызывает.

Да просто интересно зачем городить огород, когда можно написать проще как это говорит Kolobokk

Городить огород есть смысл.
Хотя бы потому, что корректно написанная строка должна выглядеть так:
"..... stock = '". mysql_escape_string($stock) ."', ....."
Только спрашивать это имело смысл в отдекльной теме, дабы не отвлекать народ от решения исходной проблемы.

А между проверкой и кодом, формирующим SQL-запрос есть что-то ещё? Не могут они подвергаться изменению? Попробуйте выводить

<?php   echo "price_50 - ".$price_50."<br>"; ?>

спускаясь по коду с проверки к SQL-запросу - на каком этапе происходит сбой?

PS Посмотрите также нет ли у вас лишних пробелов в value-значениях HTML-формы.