Формы+PHP

lapwing · July 9, 2009

Здравствуйте! Подскажите пожалуйста в чем ошибка?Код не работает

<?php

if($_POST['submit']=='submit') {

if(!$_POST['p1']||$_POST['p1']==''||!$_POST['p2']||$_POST['p2']==''||!$_POST['p3']||

$_POST['p3']=='') {

$message = 'Вы не заполнили поле';

} else {

$db = mysql_connect ("localhost","buy","buy");

mysql_select_db ("board",$db);

mysql_query("SET NAMES cp1251");

$p1=htmlspecialchars(trim($_POST['p1']));

$p2=htmlspecialchars(trim($_POST['p2']));

$p3=htmlspecialchars(trim($_POST['p3']));

$result=mysql_query("INSRT INTO buy_kv(p1,p2,p3) VALUES('$p1','$p2','$p3')");

if(mysql_affected_rows()==1) {

$message='<p> Данные занесены</p>';

$noform_var=1;

} else {

error_log(mysql_error());

$message='<p> Что-то пошло не так</p>';

}

if(!$noform_var) {

$formfile=$_SERVER['PHP_SELF'];

$message.=<<<EOMSG

<p> Все поля обязательны для заполнения </p>

</FORM>

EOMSG;

}

?>

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

<html>

<head>

</head>

<body>

<?php include("blocks/header.php");?>

</div>

<div><a href="index.php">Главная</a> </div>

<?php include("blocks/left_pan.php");?>

<?php echo $message;?>

</div>

<?php include("blocks/prav_pan.php");?>

<?php include("blocks/copyrigt.php");?>

</div>

</body>

</html>

demous · July 10, 2009

"=" забыл

а что именно не работает?

Edited July 10, 2009 by demous

lapwing · July 10, 2009

Пропустил по невнимательности "="-вставил, все равно не работает, пишет ошибку unexpected $end in C:\xampp\htdocs\Site5\addobj.php on line 65. На линии 64-тег</html>

Int · July 10, 2009

Может в одном из инклудов скобка не закрыта? У меня выдало только одну ошибку: Notice: Undefined index: submit. Замени первую строчку на if (issset($_POST['submit'])).

Edited July 10, 2009 by Int

rus · July 10, 2009

$result=mysql_query("INSRT INTO buy_kv(p1,p2,p3) VALUES('$p1','$p2','$p3')");

$result=mysql_query("INSЕRT INTO buy_kv(p1,p2,p3) VALUES('$p1','$p2','$p3')");

Найдите десять отличий

demous · July 10, 2009

я тоже думаю что проблема с инклудами. у меня только на них ругался. и может лучше лучше использовать конструкцию

$date = date("Y-m-d");

$message.=<<<EOF

<p> Все поля обязательны для заполнения </p>

</FORM>

EOF;

это убирает "> перед кнопкой. У меня показывалась.

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

lapwing · July 10, 2009

Спасибо за поправки-все сделал и submit и INSERT, проверил инклуды - все равно не работает, где еще можно поискать?

Searcher · July 10, 2009

а включаемые файлы не пробовали проверять?!

ищите ошибку в blocks/copyrigt.php

где у вас $end?

demous · July 10, 2009

А можно посмотреть на содержимое файла C:\xampp\htdocs\Site5\addobj.php?

Edited July 10, 2009 by demous

lapwing · July 10, 2009

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

неожиданный $end пишет после тега </html>/ Может нужна какая-то конструкция типа endif?

<?php 
   if(isset($_POST['submit'])) {
	  if(!$_POST['p1']||$_POST['p1']==''||!$_POST['p2']||$_POST['p2']==''||!$_POST['p3']||
	  $_POST['p3']=='') {
		 $message = 'Вы не заполнили поле';
	 } else{
	   $db = mysql_connect ("localhost","buy","buy");
	   mysql_select_db ("board",$db);
	   mysql_query("SET NAMES cp1251");

	   $p1=htmlspecialchars(trim($_POST['p1']));
	   $p2=htmlspecialchars(trim($_POST['p2']));
	   $p3=htmlspecialchars(trim($_POST['p3']));
		 $result=mysql_query("INSERT INTO buy_kv(p1,p2,p3) VALUES('$p1','$p2','$p3')");
		   if(mysql_affected_rows()==1) {
			$message='<p> Данные занесены</p>';
			$noform_var=1;
		  } else {
			 error_log(mysql_error());
			  $message='<p> Что-то пошло не так</p>';
			 }
		}
	if(!$noform_var) {
	 $formfile=$_SERVER['PHP_SELF'];
	 $date=date("Y-m-d");
	 $message.=<<<EOF
	  <p> Все поля обязательны для заполнения </p>
	  <FORM METHOD="post" ACTION="$formfile">
	  <INPUT TYPE="text" NAME="p1"><br>
	  <INPUT TYPE="text" NAME="p2"><br>
	  <INPUT TYPE="text" NAME="p3"><br>
	  <INPUT TYPE="hidden" NAME="added" VALUE="{$date}">
	  <INPUT TYPE="submit" NAME="submit" VALUE"submit">
	  </FORM>
	  EOF;
	}
 }			   	 
?>
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=windows-1251">
<link href="styles.css" rel="stylesheet" type="text/css">
</head>
<body>
 <div class="container">
  <div class="top_pan"></div>
	<div><a href="index.php">Главная</a>	</div>
	   <div class="connent">
		<?php echo $message;?>
	   </div>
  </div>
</body>
</html>

freelancecoder · July 10, 2009

Уважаемый lapwing, в последнем приведенном коде есть синтаксическая ошибка. Обычно они возникают из-за невнимательно расставленных скобок, забытых точек с запятыми, у вас же проблема в лишних пробелах перед закрывающем идентификатором.


      </FORM>
      EOF;
    }

Должно быть так:


      </FORM>
EOF;
    }

Если вы не начинаете закрывающий идентификатор в первом столбце строки, то парсер думает что это строка и нижеследующие - продолжение heredoc-строки. Для избежания подобных вещей советую приобрести какой-либо редактор с подсветкой синтаксиса php - увидев неправильную расцветку вы быстрее обнаружите источник проблемы.

Подробности про синтаксис heredoc можно прочитать здесь:

http://www.php.net/manual/ru/language.type....syntax.heredoc

demous · July 10, 2009

   if(isset($_POST['submit'])) {
	  ...
	if(!$noform_var) {
	 ...
	}
}

Вот я немного не понял. тут получается только ссылка на главную отображается. В связи с тем, что POST запрос не выполнен, форма не показывается, т.к. условие показывать форму только если на кнопку нажали. ))

ну и EOF; надо в самом начале строки ниписать.

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

Edited July 10, 2009 by demous

lapwing · July 10, 2009

Убрал пробелы перед EOF - спасибо за подсказку,сам бы долго рылся, теперь выводится сама страница, а форма почему-то нет. Я так понял, что форма должна выводиться во всех случаях, кроме тех, когда передача завершается успешно, для этого и существует переменная $noform_var? которая принимает значение в случае успешного выполнения команды INSERT , а до этого переменной не существует и по условию if(!$noform_var) форма должна выводиться, а не выводится. Что я упускаю или не понимаю? Вот demous мысль подкинул. Может первый запрос должен выглядеть как-то иначе?

demous · July 11, 2009

if(isset($_POST['submit'])) { // сначала проверяешь если поступили данные
	  ...
}
if(!$noform_var) { // потом проверяешь нужно ли показывать форму
	 ...
}

2 разных условия получается. ))

и кстати если action у формы не указать, то она на эту же страницу ссылаться будет. так что action можно убрать и переменная $formfile тоже не нужна

и у кнопки value тоже "=" пропустил

Edited July 11, 2009 by demous

lapwing · July 11, 2009

Похоже я запутался в трех соснах. Я рассуждал так: сначала условие if(isset($_POST['submit'])), поскольку кнопка не нажата, переменной $_POST['submit'] не существует, значит идем дальше - -подключаемся к БД, пытаемся выполнить INSERT, вставлять нечего, переменная $noform_var не определена и по условию if(!noform_var) должна выводиться форма. Может надо ввести вообще какую-либо другую переменную ( не submit) , а как пока не соображу.

demous · July 11, 2009

я же написал решение.

<?php
if(isset($_POST['submit'])) {
  if(!$_POST['p1']||$_POST['p1']==''||!$_POST['p2']||$_POST['p2']==''||!$_POST['p3']||$_POST['p3']=='') {
	 $message = 'Вы не заполнили поле';
  } else{
	 $db = mysql_connect ("localhost","buy","buy");
	 mysql_select_db ("board",$db);
	 mysql_query("SET NAMES cp1251");

	 $p1=htmlspecialchars(trim($_POST['p1']));
	 $p2=htmlspecialchars(trim($_POST['p2']));
	 $p3=htmlspecialchars(trim($_POST['p3']));
	 $result=mysql_query("INSERT INTO buy_kv(p1,p2,p3) VALUES('$p1','$p2','$p3')");
	 if(mysql_affected_rows()==1) {
		$message='<p> Данные занесены</p>';
		$noform_var=1;
	 } else {
		error_log(mysql_error());
		$message='<p> Что-то пошло не так</p>';
	 }
  }
}
if(!$noform_var) {
  $date=date("Y-m-d");
  $message.=<<<EOF
	  <p> Все поля обязательны для заполнения </p>
	  <FORM METHOD="post">
	  <INPUT TYPE="text" NAME="p1"><br>
	  <INPUT TYPE="text" NAME="p2"><br>
	  <INPUT TYPE="text" NAME="p3"><br>
	  <INPUT TYPE="hidden" NAME="added" VALUE="{$date}">
	  <INPUT TYPE="submit" NAME="submit" VALUE="submit">
	  </FORM>
EOF;
}
?>
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=windows-1251">
<link href="styles.css" rel="stylesheet" type="text/css">
</head>
<body>
<div class="container">
  <div class="top_pan"></div>
	<div><a href="index.php">Главная</a>	</div>
	   <div class="connent">
		<?php echo $message;?>
	   </div>
  </div>
</body>
</html>

вот так должно работать. если не понятно почему, лучше спроси сразу, чтоб вопросов не осталось ))

Edited July 11, 2009 by demous

lapwing · July 11, 2009

Наконец-то получилось!Разобрался с if(isset($_POST['submit'], остался только вопрос по ACTION. Действительно, в htmlbook.ru написано, что если параметр action отсутствует, то текущая страница перезагружается, а элементы формы возвращаются к значениям по умолчанию, но в то же время указано, что данный параметр обязателен - вроде бы получается противоречие? В каких случаях тогда используется конструкция :

$formfile=$_SERVER['PHP_SELF'];
..............
<FORM ACTION="$formfile">
// или вот так я еще видел
<FORM ACTION="<?php echo $formfile;?>"

Может, это потом понадобится? Допустим, если я захочу доработать форму в таком направлении: если пользователь все поля ввеп нормально, а какое-то одно не заполнил или заполнил как-то неправильно то, чтобы возвращалась страница с заполненной пользователем формой и стояла пометка "такое-то поле не заполнено"

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

Сейчас все работает.Прыгаю до потолка.

Int · July 11, 2009

чтобы вывести страницу с заполненной формой, нужно делать что-то типа

demous · July 12, 2009

чтобы вывести страницу с заполненной формой, нужно делать что-то типа
<input type="text" name="name" value="<? if (isset($_POST['name'])) echo $_POST['name'];?>">

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

<input type="text" name="name" value="<?=$_POST['name'];?>"> ))

А по поводу action, ну может оно и обязательное поле с точки зрения w3c, и если уж так нужно его использовать, то можно сделать анологично

action="<?=$_SERVER['PHP_SELF'];?>".

P.S. <? echo $aaa;?> == <?=$aaa?> ))

Int · July 12, 2009

Если проверку не делать, php завалит нотисами, тогда уж value="<?=@$_POST['name']?>", с подавлением ошибок, но это как-то негуманно.

Edited July 12, 2009 by Int

demous · July 12, 2009

Если проверку не делать, php завалит нотисами, тогда уж value="<?=@$_POST['name']?>", с подавлением ошибок, но это как-то негуманно.

Странно, у меня никогда ни на хостинге, ни локально ошибок не выводил. Это же массив просто, не функция. Возможно дело в настройке PHP. Ну и я не говорил что это не правильно, просто на мой взгляд это более просто.

lapwing · July 12, 2009

Большое спасибо всем ответившим. Буду двигаться дальше.

Sign In

Формы+PHP

Question

Link to comment

Share on other sites

21 answers to this question

Recommended Posts

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Join the conversation

Обсуждения

Important Information