Не могу найти ошибку в этой строке

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

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

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

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

<?php 
$Host = "database.agava.ru"; 
$User = "xxx"; 
$Password = "xxx"; 
$DBName = "xxx"; 
$TableName = "Feedback"; 
mysql_connect($Host, $User, $Password) or die (mysql_error()); 
mysql_select_db($DBName) or die (mysql_error());
$Query="INSERT INTO $TableName (".implode(",",array_keys($Array)).") values (\'".implode("\',\'",$Array)."\')";
if (mysql_query($Query)) {
  header("Location: sql_feedback_view.php");
  exit;
} else { 
  echo "Не удается обновить таблицу $TableName.
\\n";
  echo "Ошибка: ".mysql_error()."
\\n";
  echo "Запрос: $Query
\\n";
} 
?>

Код я не проверял, если есть опечатки - постарайся исправить их сам.
Если ты действительно хочешь научиться правильно писать на РНР.
Если нет - только скажи - я закрою эту тему и мы расстанемся друзьями.

[Matold]

и тут же берешь код, в котором столбцы не перечислены

Так как я уменьшил таблицу, поэтому не стал перечислять столбцы.

Неужели так трудно подумать и спросить, что непонятно?
человеку дают ГОТОВЫЙ правильный пример

Вот мне и не понятно, если ты объясняешь что пример правильный и готовый, то почему результат кода
<?php 
$Host = "database.agava.ru"; 
$User = "xxx"; 
$Password = "xxx"; 
$DBName = "xxx"; 
$TableName = "Feedback"; 
$Link = mysql_connect($Host, $User, $Password); 
//$Query = "INSERT into $TableName values (\\"0\\", \\"$Array[FirstName]\\", \\"$Array[LastName]\\", \\"$Array[Email]\\", \\"$Array[Comments]\\")"; 
$query="INSERT INTO $TableName values (0,\'".implode("\',\'",$Array)."\')"; 
if (mysql_db_query ($DBName, $Query, $Link)) { 
echo "обновлены данные таблицы $TableName.
\\n"; 
echo "\\n <meta http-equiv=\\"refresh\\" content=\\"1;URL=sql_feedback_view.php\\" >"; 
} else { 
echo "Не удается обновить таблицу $TableName.
\\n"; 
} 
mysql_close ($Link); 
?> 

при выполнении выдает "Не удается обновить таблицу" Я написал абсолютно как ты сказал:

ну, или если без перечисления полей, то так:

$query="INSERT INTO $TableName values (0,\'".implode("\',\'",$Array)."\');

Так ты убиваешь кучу зайцев: получается универсальный запрос не зависящий от состава и количества полей.
получается синтаксически правильный запрос.
Избегаешь возможных ошибок при ручном вводе.

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

Тут я тебя не понимаю... //$Query=... Это то, что у меня работало, но чтобы не стирать я ее поместил в комментарий.
То что ты сказал - $query я вставил в программу. Ну так объясни, ПОЧЕМУ НЕ РАБОТАЕТ ? Причем здесь "копировать не метод, не умение, не знание, а тупо, как обезьяна, просто передирать код, не понимая его смысла" Я понимаю смысл своего кода, и он действует, но если вставить твой код, то не база не обновляется

[Matold]

Я думаю, нужно все таки почитать что такое . и зачем она нужна... как только ты пймешь, что это такое... я думаю ты поймешь конструкции \\" в твоем коде - гупость.

Если не секрет, где найти про . ? (Вернее как найти? В руководстве php поиск точки ничего не дает) Насколько я знаю она складывает 2 строки, но не понимаю как относиться к данному примеру

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

Matold:
Вот мне и не понятно, если ты объясняешь что пример правильный и готовый,

ты смеешься что ли надо мной?
по-моему, ты издеваешься.
Правильный код, который я тебе дал, выводит ПРИЧИНУ, почему  "Не удается обновить таблицу" , а не только это тупое сообщение.

Matold:
То что ты сказал - $query я вставил в программу

ты вставил $qery, а дальше у тебя, в следующей строчке запрашивается $Qery
Об этом я тебе толкую, но до тебя не доходит!

Matold:
ПОЧЕМУ НЕ РАБОТАЕТ

объяснение одно - опечатки.
Которые надо найти и исправить.
У тебя есть другое мнение?
или ты считаешь, что я тебе эту опечатку телепатическим путем вычислю?


Давненько я не встречал такого мутного клиента.

[Matold]

$qery, а дальше у тебя, в следующей строчке запрашивается $Qery

Ах черт! Все спасибо, я не знал что к регистру чувствителен PHP, я это просто для удобства называл с большой
ОГРОМНОЕ СПАСИБО НУ Я И ИДИОТ. Все, тему можно закрыть

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

точно, все работает?
ты уж на меня не обизайся за резкость.
устал я что-то...

[FreeSpace]

Я бы добавил сюда addslashes:

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

А я бы не добавил.
зачем?

[FreeSpace]

Я не совсем понял. Ты спрашиваешь у меня, зачем делать addslashes перед добавлением данных в БД?

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

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

[FreeSpace]

Спасибо, твой фак я уже давно читал. Но на всякий случай перечитал ещё раз - может там что-то новенькое появилось
Не мог бы ты нормально ответить, чем тебе не понравилось моё дополнение?
Если речь идет о magic_quotes_gpc, то лично я считаю значение off нормальным для этой директивы. Либо в начале скрипта делать проверку и при необходимости чистить соответствующие массивы от слешей. Из этих предпосылок я и исходил. Или ты с этим не согласен?

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

я не согласен с тем, как ты это выдал здесь - безапелляционно и безкондиционно втулить аддслашес.
9:1 за то, что у него слеши добавляются автоматом.
админ его хостинга, увы, не спросил тебя, что ты считаешь нормальным.
А жалобы ламеров "ай меня поломали!!!" ни один админ слушать не хочет.
И если сделать, как ты написал, то добавятся лишние.

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

Matold, если ты еще слушаешь

учитывая замечание FreeSpace, можно уже сделать функцию, которая будет за нас выполнять всю рутинную работу.
с такой функцией скрипт станет компактнее и более удобочитаем.
Ее можно поместить в файл, который инклюдится во все скрипты, и пользоваться в любом месте.

function sqlInsert($TableName,$Array) {
  if (get_magic_quotes_gpc()) foreach ($Array as $k=> $v) $Array[$k]=mysql_escape_string($v);
  $Query="INSERT INTO $TableName (".implode(",",array_keys($Array)).") values (\'".implode("\',\'",$Array)."\')"; 
  if (!mysql_query($Query)) return FALSE;
  return mysql_insert_id();
}

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

последняя строчка получает id и возвращает его.

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

Matold:
я не знал что к регистру чувствителен PHP

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

имена функций не чувствительны