|
|
|
| форма занесения
<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>
|
| |
|
|
|
|
|
|
|
для: scva
(11.06.2009 в 21:47)
| | вместо выполнения запроса выводим его код на экран.
Код берем в буфер и пытаемся выполнить в phpMyAdmin.
между прочим, что означает if ($result == 'true') ?
про то, где определяются переменные $id и $name я уже молчу. | |
|
|
|
|
|
|
|
для: Trianon
(11.06.2009 в 22:15)
| | а как бы ты написал этот запрос?
if ($result == 'true') указывает выполнилось ли действие | |
|
|
|
|
|
|
|
для: 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>";
?>
|
| |
|
|
|
|
|
|
|
для: Slo_Nik
(11.06.2009 в 23:56)
| | хороший код, а где бы ты прописал глобальные
if (isset($_POST['name'])) {$name = $_POST['name'];} | |
|
|
|
|
|
|
|
для: scva
(12.06.2009 в 00:09)
| | а от этого вооооообще много пользы? скажи мне пож-ста, не легче ли постоянно пользоваться $_POST['name']
кто вас всех этой хрени учит? мало того фигурные скобки можно былобы и неписать… | |
|
|
|
|
|
|
|
для: nikita2206
(12.06.2009 в 00:14)
| | легче конечно использовать $_POST['name'], но есть возможность, что на некоторых хостах может не сработать.
а вот
<?php
if(isset($_POST['name'])) $name = $_POST['name'];
?>
|
сработает всегда.
я с такой фигнёй раз столкнулся. | |
|
|
|
|
|
|
|
для: 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.'); | |
|
|
|
|
|
|
|
для: nikita2206
(12.06.2009 в 00:39)
| | >> if(isset($_POST['name'])) $name = $_POST['name'];
>... проверяем существует ли элемент массива $_POST под ключём name, далее если существует то создаётся переменная $name, но заметте, если не существует, то $name не создаётся, если есть логика, то несложно понять, что это действие не имеет ни какой практической пользы, разве что, переменная теперь с более коротким именем…
>тогда, я думаю, было бы гораздо разумней написать, например, так:
>$name = @$_POST['name'];
Золотая мысль.
Почему-то большинству здесь активно присутствующих в голову не приходит. | |
|
|
|
|
|
|
|
для: Trianon
(12.06.2009 в 10:47)
| | ммм сарказм | |
|
|
|
|
|
|
|
для: nikita2206
(12.06.2009 в 14:20)
| | Да нет вовсе.
Наплыв этого шаблона if(isset($_POST['name'])) $name = $_POST['name']; последнее время участился.
Хотелось ответить именно такими словами.
Правда вместо них лезли другие "Что ж ты делаешь, баран?! Ты же создаешь код, который в половине случаев оставляет переменную не инициализированной! О какой надежности после этого можно говорить?!" . А они здесь не приветствуются.
На всякий случай акцентирую, вышеприведенное - не Слонику персонально. Кодеров с таким подходом здесь десять на дюжину. | |
|
|
|
|
|
|
|
для: Slo_Nik
(12.06.2009 в 00:22)
| | Таким подходом Вы, фактически, низводите методы кодирования на php5 до уровня методов кодирования на php3. Со всеми вытекающими оргвыводами в плане надежности кода.
И только потому, что когда-то напоролись на старый (с лет пять не апгрейдившимся софтом) хостинг. | |
|
|
|
|
|
|
|
для: nikita2206
(12.06.2009 в 00:14)
| | INSERT INTO `order` ( `id` , `name` )VALUES ('$id','$_POST['soname']') так что ли | |
|
|
|
|
|
|
|
для: scva
(12.06.2009 в 00:25)
| | попробуй так сделать и тебе выдаст ошибку | |
|
|
|
|
|
|
|
для: scva
(12.06.2009 в 00:25)
| | нет не так, и еще, используйте по возможности одинарные кавычки вместо двойных и соответственно конкатенацию…
вот так правильно:
<?
mysql('INSERT INTO `order` ( `id` , `name` )VALUES ('.$id.', "'.$_POST['soname'].'")');
|
P.S. хорошо хоть то, что используете наклонные кавычки в mysql) | |
|
|
|
|
|
|
|
для: scva
(12.06.2009 в 00:09)
| | я тебе привёл пример запроса.
определить переменные естественно надо до запроса.
в начале ты используешь короткие теги "<? ?>", а потом "<?php ?>", сделай одинаково | |
|
|
|
|
|
|
|
для: scva
(11.06.2009 в 23:43)
| | >if ($result == 'true') указывает выполнилось ли действие
ничего подобного.
То есть возможно это конструкция и работает, но по чистой случайности. | |
|
|
|