Форум: Форум PHPФорум ApacheФорум Регулярные ВыраженияФорум MySQLHTML+CSS+JavaScriptФорум FlashРазное
Новые темы: 0000000
PHP 5/6. В подлиннике. Авторы: Кузнецов М.В., Симдянов И.В. PHP. Практика создания Web-сайтов (второе издание). Авторы: Кузнецов М.В., Симдянов И.В. PHP 5. На примерах. Авторы: Кузнецов М.В., Симдянов И.В., Голышев С.В. Программирование. Ступени успешной карьеры. Авторы: Кузнецов М.В., Симдянов И.В. Самоучитель PHP 5 / 6 (3 издание). Авторы: Кузнецов М.В., Симдянов И.В.
ВСЕ НАШИ КНИГИ
Консультационный центр SoftTime

Форум MySQL

Выбрать другой форум

 

Здравствуйте, Посетитель!

вид форума:
Линейный форум Структурный форум

тема: Не заносит в базу
 
 автор: scva   (11.06.2009 в 21:47)   письмо автору
 
 

форма занесения

<form action="order1.php" method="post" name="form">
<p> имя
<br>

<input name="name" type="text"size="50">
</p>

<p>
<input name="submit" type="submit" value="Добавить">

</form>




odrer1.php

<?
require_once("base.php");
if (isset(
$_POST['name']))                {$name $_POST['name'];}




?>
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<html>


<body>


<?php 


if (isset($name))
  {
$result mysql_query ("INSERT INTO order (id,name) VALUES ('$id','$name')");
if (
$result == 'true') {echo "<h3>добавлена</h3><br> <a href='manager.php'>На администраторскую </a>";
}
else {echo 
"<h3>не добавлена</h3>";}}
else { echo 
"<h3> Вы ввели не все данные";}
    
?> 

</body>
</html>

  Ответить  
 
 автор: Trianon   (11.06.2009 в 22:15)   письмо автору
 
   для: scva   (11.06.2009 в 21:47)
 

вместо выполнения запроса выводим его код на экран.
Код берем в буфер и пытаемся выполнить в phpMyAdmin.

между прочим, что означает if ($result == 'true') ?

про то, где определяются переменные $id и $name я уже молчу.

  Ответить  
 
 автор: scva   (11.06.2009 в 23:43)   письмо автору
 
   для: Trianon   (11.06.2009 в 22:15)
 

а как бы ты написал этот запрос?

if ($result == 'true') указывает выполнилось ли действие

  Ответить  
 
 автор: Slo_Nik   (11.06.2009 в 23:56)   письмо автору
 
   для: scva   (11.06.2009 в 23:43)
 

а не проще ли написать

<?php 
$query 
"INSERT INTO order (id,name) VALUES ('$id','$name')";
if(!
mysql_query($query)) echo "Ошибка".mysql_error();
else echo 
"<h3>добавлена</h3><br> <a href='manager.php'>На администраторскую </a>";
?>

  Ответить  
 
 автор: scva   (12.06.2009 в 00:09)   письмо автору
 
   для: Slo_Nik   (11.06.2009 в 23:56)
 

хороший код, а где бы ты прописал глобальные
if (isset($_POST['name'])) {$name = $_POST['name'];}

  Ответить  
 
 автор: nikita2206   (12.06.2009 в 00:14)   письмо автору
 
   для: scva   (12.06.2009 в 00:09)
 

а от этого вооооообще много пользы? скажи мне пож-ста, не легче ли постоянно пользоваться $_POST['name']

кто вас всех этой хрени учит? мало того фигурные скобки можно былобы и неписать…

  Ответить  
 
 автор: Slo_Nik   (12.06.2009 в 00:22)   письмо автору
 
   для: nikita2206   (12.06.2009 в 00:14)
 

легче конечно использовать $_POST['name'], но есть возможность, что на некоторых хостах может не сработать.
а вот

<?php 
if(isset($_POST['name'])) $name $_POST['name'];
?>

сработает всегда.
я с такой фигнёй раз столкнулся.

  Ответить  
 
 автор: nikita2206   (12.06.2009 в 00:39)   письмо автору
 
   для: Slo_Nik   (12.06.2009 в 00:22)
 

я думаю вы заблуждаетесь и не нашли реальной причины проблемы…
просто смотрите что сдесь делается: сначала проверяем существует ли элемент массива $_POST под ключём name, далее если существует то создаётся переменная $name, но заметте, если не существует, то $name не создаётся, если есть логика, то несложно понять, что это действие не имеет ни какой практической пользы, разве что, переменная теперь с более коротким именем…
тогда, я думаю, было бы гораздо разумней написать, например, так:
$name = @$_POST['name'];
или, если уж заняться нечем, то так:
$name = array_key_exist($_POST, 'name') ? $_POST['name'] : exit('Some variables was not submited to this script.');

  Ответить  
 
 автор: Trianon   (12.06.2009 в 10:47)   письмо автору
 
   для: nikita2206   (12.06.2009 в 00:39)
 

>> if(isset($_POST['name'])) $name = $_POST['name'];
>... проверяем существует ли элемент массива $_POST под ключём name, далее если существует то создаётся переменная $name, но заметте, если не существует, то $name не создаётся, если есть логика, то несложно понять, что это действие не имеет ни какой практической пользы, разве что, переменная теперь с более коротким именем…
>тогда, я думаю, было бы гораздо разумней написать, например, так:
>$name = @$_POST['name'];

Золотая мысль.
Почему-то большинству здесь активно присутствующих в голову не приходит.

  Ответить  
 
 автор: nikita2206   (12.06.2009 в 14:20)   письмо автору
 
   для: Trianon   (12.06.2009 в 10:47)
 

ммм сарказм

  Ответить  
 
 автор: Trianon   (12.06.2009 в 14:57)   письмо автору
 
   для: nikita2206   (12.06.2009 в 14:20)
 

Да нет вовсе.
Наплыв этого шаблона if(isset($_POST['name'])) $name = $_POST['name']; последнее время участился.
Хотелось ответить именно такими словами.
Правда вместо них лезли другие "Что ж ты делаешь, баран?! Ты же создаешь код, который в половине случаев оставляет переменную не инициализированной! О какой надежности после этого можно говорить?!" . А они здесь не приветствуются.

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

  Ответить  
 
 автор: Trianon   (12.06.2009 в 10:50)   письмо автору
 
   для: Slo_Nik   (12.06.2009 в 00:22)
 

Таким подходом Вы, фактически, низводите методы кодирования на php5 до уровня методов кодирования на php3. Со всеми вытекающими оргвыводами в плане надежности кода.

И только потому, что когда-то напоролись на старый (с лет пять не апгрейдившимся софтом) хостинг.

  Ответить  
 
 автор: scva   (12.06.2009 в 00:25)   письмо автору
 
   для: nikita2206   (12.06.2009 в 00:14)
 

INSERT INTO `order` ( `id` , `name` )VALUES ('$id','$_POST['soname']') так что ли

  Ответить  
 
 автор: Slo_Nik   (12.06.2009 в 00:35)   письмо автору
 
   для: scva   (12.06.2009 в 00:25)
 

попробуй так сделать и тебе выдаст ошибку

  Ответить  
 
 автор: nikita2206   (12.06.2009 в 00:42)   письмо автору
 
   для: scva   (12.06.2009 в 00:25)
 

нет не так, и еще, используйте по возможности одинарные кавычки вместо двойных и соответственно конкатенацию…
вот так правильно:
<?
mysql
('INSERT INTO `order` ( `id` , `name` )VALUES ('.$id.', "'.$_POST['soname'].'")');

P.S. хорошо хоть то, что используете наклонные кавычки в mysql)

  Ответить  
 
 автор: Slo_Nik   (12.06.2009 в 00:18)   письмо автору
 
   для: scva   (12.06.2009 в 00:09)
 

я тебе привёл пример запроса.
определить переменные естественно надо до запроса.
в начале ты используешь короткие теги "<? ?>", а потом "<?php ?>", сделай одинаково

  Ответить  
 
 автор: Trianon   (12.06.2009 в 00:02)   письмо автору
 
   для: scva   (11.06.2009 в 23:43)
 

>if ($result == 'true') указывает выполнилось ли действие

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

  Ответить  
Rambler's Top100
вверх

Rambler's Top100 Яндекс.Метрика Яндекс цитирования