Lightt
Сначала проверку на нажатие кнопки, а внутри неё уже проверку, введены ли данные, и если не введены - выводи текст о том, что не введены они были.

Логику своего скрипта читай:
- Введены ли данные? - НЕТ!, значит мы что делаем? Правильно, выводим текст, который ты написал выводить в случае отсутствия данных.
А логика такая должна быть:

if(Нажата кнопка?) 
{
    if(Введены данные?)
   
}
А тут форму вывели.

в этом скрипте, чтобы показывало сообщение "Поля не заполнены", как минимум, надо написать вывод этого сообщения!


Кстати, где форма-то?

Что-то типа этого )

<form action='guestbook_msg_sender.php' method='POST'>
  				<td width="150">Сообщение </td>
				<td><textarea name="message" id="contentbox"></textarea></td>
				<td><input type='submit' name='ok' value='ОК' /><br />
<div style="color:red;"><?php echo $editError; ?></div>	</td>
</form>

если честно, то впервые вижу такую Индию )

inpost Дааа спасибо =) действительно тупанул что то...
Но теперь не выводится сообщение

//добавление нового сообщения
if (isset($_POST['ok']))
{
//Проверка на пустоту
if (!isset($_POST['login']) || $_POST['login'] == '')
	{
	$editError = 'Не введен логин';
	}

else
$result = mysql_query('insert into guestbook (login, mail, message, date)
VALUES ("'.$_POST['login'].'", 
		"'.$_POST['mail'].'",
		"'.$_POST['message'].'",
		"'.$dat.'")');
header('Location: ' . $_SERVER['PHP_SELF']);
exit;
}	 

CyberOrcX Да это само собой разумеется, есть там форма, это не полный код. А именно часть которая отвечает за вывод сообщения

Lightt

if (!isset($_POST['login']) || $_POST['login'] == '')

Лучше так

if (!empty($_POST['login']))

Lightt
Ай! Короче! Вот так

//добавление нового сообщения
if (isset($_POST['ok']))
{
//Проверка на пустоту
if (!empty($_POST['login']))
	{
	$editError = 'Не введен логин';
	}else{
$result = mysql_query("INSERT INTO `guestbook` SET
              `login` = '".mysql_real_escape_strings($_POST['login'])."',
              `mail` = '".mysql_real_escape_strings($_POST['mail'])."',
              `message` = '".mysql_real_escape_strings($_POST['message'])."',
              `date` = '".htmlspesialchars($_POST['date'])."'");
header('Location: ' . $_SERVER['PHP_SELF']);
exit;
}
}	

m4a1fox
Warning, не существует переменной

if (empty($_POST['login']))	
{	
    echo $editError = 'Не введен логин';	
}

Игорь_Vasinsky
Читай выше

inpost
Это как??????

m4a1fox
Тогда уж так -

if (empty($_POST['login']))	

Но все равно не выводится сообщение.

кокой переменной нет... не вижу...
есть ошибки в SQL (пропущены ' - несколько раз)

Lightt
Ну да - пусто пусто! Переборщил я с !-знаком!...

Lightt
для вывода ошибки используй echo - эти 4 буквы творят чудеса с пустой страницей. я выше написал как.

Игорь_Vasinsky

:D наоборот ковычки

'" . $var. "'

Игорь_Vasinsky

Так сообщение тоже не выводится =)
И + мне надо вывести его в конкретном месте...

Игорь_Vasinsky
Ага! Точно модмечено Холмс!!

Lightt

зачем ошибки в бд заносить? историю ошибок юзера чтоли делаете?


если есть ошибка - то есть переменная $editError, счтоб её вывести нужно проверить есть ли она.

if($editError)
    echo $editError;

в нужном месте.

Игорь_Vasinsky
Про кавычки никто не объяснял)) делаю как умею, так правильней?

$dat = date("Y.m.j."); //дата

//----------------------------------------------------------
include_once "mysql.php"; //подключение хоста
//------	
mysql_query("SET NAMES cp1251"); //отоброжения русского текста
mysql_select_db("links");


//добавление нового сообщения
if (isset($_POST["ok"]))
{
//Проверка на пустоту
if (empty($_POST["login"]))	
{	
echo $editError = "Не введен логин";	
}



else
$result = mysql_query("insert into guestbook (login, mail, message, date)
VALUES ('".$_POST["login"]."', 
		'".$_POST["mail"]."',
		'".$_POST["message"]."',
		'".$dat."')");
header("Location: " . $_SERVER["PHP_SELF"]);
exit;
}	 
//вывод всех сообщений
$result = mysql_query("select * from guestbook ");				 
while ($all_guestbook[] = mysql_fetch_assoc($result)) {}

 echo "Вот тут должна появиться ошибка".$editError;

Слова появляются, переменная нет.

Lightt
Черт! Ну вот как так? Если вы пишите

//Проверка на пустоту
if (empty($_POST["login"]))	
{	
echo $editError = "Не введен логин";	
}

то почему else без скобок?

//Проверка на пустоту
if (empty($_POST["login"]))	
{	
echo $editError = "Не введен логин";	
}else{
$result = mysql_query("insert into guestbook (login, mail, message, date)
VALUES ('".$_POST["login"]."', 
		'".$_POST["mail"]."',
		'".$_POST["message"]."',
		'".$dat."')");
header("Location: " . $_SERVER["PHP_SELF"]);
exit;
}

Про оформление SQL запросов написано у меня в подписи - в частности про конкатенацию.

Если одно действие в условии - скобки можно опустить.
я бы вообще сделал так:

if($_POST['ok'])
{
   $login = !empty($_POST['login']) ? htmlspecialchars($_POST['login']) : null;
   //Не стоит принебрегать безопастностью, например htmlspecialchars()
   // И вообще бы не дурно было бы проверить на соответствие регуляркой
 ...
 ...
 ...
 //Создали массив для ошибок
  $editErrors = array();

  if($login)
      $editErrors[] = "Не заполнено поле Логин!";
   ...
   ...
   ...
   
  // Проверили на наличие элементов в массиве - если они(он) есть - значи и есть ошибки которые надо вывести.
  if($editErrors)
  {
      foreach($editErrors as $editError)
      {
          echo $editError."<br/>";
      }
  } 

}

m4a1fox

Блиин, внатуре о_О ошибку не выдавал, а сам про них забыл =) спасибо. Заработало))

Игорь_Vasinsky
Ок, счас почитаю, переварю, и попробую сделать правильно =)

Про тернарный оператор - у меня в журнале или на ирбисе.

Lightt
Не забывай про них! А если опускаешь то вот так

if(!isset($_POST['log']))
    echo 'Ups! Shit Error! ';
else
    echo 'This is OK!';

ИМХО! Я так делаю!

if($editError)
echo $editError;
- это уж тем более ошибка.

Пиши так:
if(!isset() || empty())

inpost
так $editError - если нет ошибок - то она не определена.

поэтому и if($editError)

Игорь_Vasinsky
Если она не определена, то
if($editError) - будет равен WARNING.

варнингов не наблюдал вроде...

ну дописать isset()