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

в вашем скрипе сразу два else подряд
что и не верно не соответствует логике
переправте первое else на ifelse

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

Привел к читабельному виду. ifelse думаю не нужен =) каждый else относится к своему if. Или я напутал?

смотри сам что написал два отдельных скрипа
ты просто разорвал один вот и условия не проходят
вот как помогают этот новый стиь сам терь видить должен
у тя новый скрипт начинаеться условия else

Не могу понять, де разорвал, только что все условия проверил. все if и else. всё верно =)

терь правило в одном if должен быть только один else

только один !!

<?php
}

else
{
$pass = trim($_POST['pass']);
$email = trim($_POST['email']);

вот твоё начало скрипта ??? с чего он начинаетьс ???? с чего ????

вот так переделай

<?php session_start(); ?>

тут код формы или ещё какойнить далЬше второй скрипт

<?php
if (!isset($_POST[‘enter’]))
{


else
{
$pass = trim($_POST['pass']);
$email = trim($_POST['email']);

Тогда может условие всё переделать под и будет так?

<?php
session_start();
// Проверка нажатия клавиши "Войти"
?>
 <center>
 
<h4>АВТОРИЗАЦИЯ</h4><br>
<form action="" method="post">
<h4>Имя:</h4>
   <input type="text" name="email" value=""><br><br>
<h4>Password:</h4>
   <input type="password" name="pass"><br><br>
   <input type="submit" name=‘enter’ value="Войти">
</form></center>
<?php
if (isset($_POST[‘enter’]) and ((!empty($email)) or (!empty($pass))))
{
	$pass = trim($_POST['pass']);
	$email = trim($_POST['email']);
	$pass = md5($pass);
	require_once("connect.php");
	// формируем SQL-запрос
	$sql = "SELECT id FROM USERS WHERE email='".$email."' and pass='".$pass."'";

	// делаем запрос
	$result = mysql_query ($sql);
		if (!mysql_num_rows($result))
			{
			die ("Не верный логин или пароль <a href='index.php'> На главную </a>");
			}
			else
			{
			// записываем переменные в сессию
			$line = mysql_fetch_row($result);
			$_SESSION[‘autorized’]=true;
			$_SESSION[‘email’]=$safe_name;
			$_SESSION[‘role’]=$line[2];
 
			// выводим ссылку для входа в админку (можно сделать перенаправление)
			echo "<center><h4>Авторизация прошла успешно</h4><br><a href=index.php>Вернуться на главную</a><br><br><a href=’admin.php’>Войти в админку</a></center>";
			}
}
else 
{
echo "Проверьте Логин и пароль";
}
?>

Но это в конец не верно получается =(

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

if (isset($_POST['enter']) and ((!empty($_POST['email'])) or (!empty($_POST['pass']))))
{
	$pass = trim($_POST['pass']);
	$email = trim($_POST['email']);

$result = mysql_query ($sql);
		if (!mysql_num_rows($result))
			{
			die ("Не верный логин или пароль <a href='index.php'> На главную </a>");
			}
			else
			{
			// записываем переменные в сессию
			$line = mysql_fetch_row($result);
			$_SESSION[‘autorized’]=true;
			$_SESSION[‘email’]=$safe_name;
			$_SESSION[‘role’]=$line[2];
 
			// выводим ссылку для входа в админку (можно сделать перенаправление)
			echo "<center><h4>Авторизация прошла успешно</h4><br><a href=index.php>Вернуться на главную</a><br><br><a href=’admin.php’>Войти в админку</a></center>";
			}

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

die ("Не верный логин или пароль <a href='index.php'> На главную </a>");

die что оно значит ?
может eche

YVSIK
не путай человека - сам вообще набрал бы в гугле что значит die

а так по делу если - не будет этот скрипт никогда работать ибо валидация проходит после того как страница сгенерирована - а это господа только аяксом можно сделать

Последний вариант скрипта вот такой: Ошибку выдаёт что "Логин или пароль не верны, а в базе и логин и пасс есть"

<?php
session_start();
// Проверка нажатия клавиши "Войти"
?>
<center>
<h4>АВТОРИЗАЦИЯ</h4><br>
<form action="" method="post">
<h4>Имя:</h4>
   <input type="text" name="email" value=""><br><br>
<h4>Password:</h4>
   <input type="password" name="pass"><br><br>
   <input type="submit" name=‘enter’ value="Войти">
</form></center>

<?php
if (isset($_POST[‘enter’]))
{
$pass = trim($_POST['pass']);
$email = trim($_POST['email']);

	if ((!empty($email)) or (!empty($pass)))
	{
		$pass = md5($pass);
		require_once("connect.php");
		// формируем SQL-запрос
		$sql = "SELECT email, pass FROM USERS WHERE email='".$email."' and pass='".$pass."'";

		// делаем запрос
		$result = mysql_query ($sql);
		if (!mysql_num_rows($result))
		{
			die ("Не верный логин или пароль <a href='index.php'> На главную </a>");
		}
		else
		{
			// записываем переменные в сессию
			$line = mysql_fetch_row($result);
			$_SESSION[‘autorized’]=true;
			$_SESSION[‘email’]=$safe_name;
			$_SESSION[‘role’]=$line[2];
 
			// выводим ссылку для входа в админку (можно сделать перенаправление)
			echo "<center><h4>Авторизация прошла успешно</h4><br><a href=index.php>Вернуться на главную</a><br><br><a href=’admin.php’>Войти в админку</a></center>";
		}
	}
}		
?>

2 Админ, как тогда сделать норм скрипт? я уже мозг себе вынес =(

ну вот таки стоит эти кавычки поменять!

if (isset($_POST['enter']))

$_SESSION['autorized']=true;
$_SESSION['email']=$safe_name;
$_SESSION['role']=$line[2];

Исправил ковычки, поставил вариант Админа, но отказывается воспринимать из базы данных логин и пароль =(

может моя ошибка, где-то здесь?

$sql = "SELECT email, pass FROM USERS WHERE email='".$email."' and pass='".$pass."'";

		// делаем запрос
		$result = mysql_query ($sql);
		if (!mysql_num_rows($result))
		{
			die ("Не верный логин или пароль <a href='index.php'> На главную </a>");
		}

я бы сделал так(догадываюсь, что делает mysql_fetch_row, но точно не знаю)

if (!mysql_num_rows($result))
		{
			die ("Не верный логин или пароль <a href='index.php'> На главную </a>");
		}
		else
		{
			// записываем переменные в сессию
			$line = mysql_fetch_assoc($result);
			$_SESSION['autorized']=true;
			$_SESSION['email']=$line['email']; //тут поле с БД
			$_SESSION['role']=$line['role'];     //тут тоже
 
			// выводим ссылку для входа в админку (можно сделать перенаправление)
			echo "<center><h4>Авторизация прошла успешно</h4><br><a href=index.php>Вернуться на главную</a><br><br><a href=’admin.php’>Войти в админку</a></center>";
		}

Druids попробуй так
возьми мой скрипт и примени
и ещё любой скрипт начинается с малого
поробуй его минимизировать
чтоб только логика потом исправишь внесешь все необходимое

Сейчас я переписываю своё.

Вот смотрите.
берем с самого начала.
Начинаем с сессии (session_start() далее делаем проверку если нажата кнопка и правилен пароль с мылом, далее делаем сверку с базой а после этого уже авторизуемся?

А в самом конце уже вставляем форму

так щас найду тему свою как я дошел до этого скрипа своего

вотт http://phpforum.ru/index.php?showtopic=40622&st=0

Попробуй так

Ошибка вот в этом кусочке кода. Матерится вот так... Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in W:\home\localhost\www\123\login.php on line 18
Как подправить код?

		/* Проверяем на существование пользователя */
		$result="SELECT email, pass FROM USERS WHERE email='".$email."', pass='".$pass."'";
		if (mysql_num_rows($result))
		{
			// записываем переменные в сессию
			$line = mysql_fetch_row($result);
			$_SESSION['autorized']=true;
			$_SESSION['email']=$email;
			$_SESSION['role']=$line[2];
			// выводим ссылку для входа в админку (можно сделать перенаправление)
			echo "<center><h4>Авторизация прошла успешно</h4><br><a href=index.php>Вернуться на главную</a><br><br><a href=’admin.php’>Войти в админку</a></center>";
		}
		else
		{
		echo 'Логин или пароль не верны!';
		}

Весь код

<?php
session_start();

if(isset($_POST['enter']))
{
	/* Пинимаем данные из формы */
	$pass	=	trim($_POST['pass']);
	$email	=	trim($_POST['email']);
	/* Проверяем на заполнение */
	if(empty($pass) || empty($email))
	{
		echo 'Заполните все поля';
	}
	else
	{
		/* Проверяем на существование пользователя */
		$result="SELECT email, pass FROM USERS WHERE email='".$email."', pass='".$pass."'";
		if (mysql_num_rows($result))
		{
			// записываем переменные в сессию
			$line = mysql_fetch_row($result);
			$_SESSION['autorized']=true;
			$_SESSION['email']=$email;
			$_SESSION['role']=$line[2];
			// выводим ссылку для входа в админку (можно сделать перенаправление)
			echo "<center><h4>Авторизация прошла успешно</h4><br><a href=index.php>Вернуться на главную</a><br><br><a href=’admin.php’>Войти в админку</a></center>";
		}
		else
		{
		echo 'Логин или пароль не верны!';
		}
	}
}
else
{
?>
<center>
	<h4>АВТОРИЗАЦИЯ</h4><br>
	<form action="" method="post">
		<h4>Имя:</h4>
		<input type="text" name="email" value=""><br><br>
		<h4>Password:</h4>
		<input type="password" name="pass"><br><br>
		<input type="submit" name="enter" value="Войти">
	</form>
</center>
<?php
}
?>

$result = mysql_query(
    "SELECT `email`, `pass`
         FROM `USERS`
         WHERE `email` = '".mysql_real_escape_string($email) . "' AND
               `pass` = '" . mysql_real_escape_string($pass) . "'"
) or die(mysql_error());

Теперь вот такая ошибка в тех же строках.

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: Access denied for user 'ODBC'@'localhost' (using password: NO) in W:\home\localhost\www\123\login.php on line 17

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: A link to the server could not be established in W:\home\localhost\www\123\login.php on line 17

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: Access denied for user 'ODBC'@'localhost' (using password: NO) in W:\home\localhost\www\123\login.php on line 17

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: A link to the server could not be established in W:\home\localhost\www\123\login.php on line 17

Warning: mysql_query() [function.mysql-query]: Access denied for user 'ODBC'@'localhost' (using password: NO) in W:\home\localhost\www\123\login.php on line 17

Warning: mysql_query() [function.mysql-query]: A link to the server could not be established in W:\home\localhost\www\123\login.php on line 17
Access denied for user 'ODBC'@'localhost' (using password: NO)

ну дак тебе ж всё написали - нету у тебя с базой соединения

Druids
все верно, где mysql_connect() ? где mysql_select_db() ?

Druids
посмотри еще раз мой пост, я его подредактировал

У вас есть ошибка в синтаксисе SQL; проверить документацию по используемой версии MySQL на предмет корректного синтаксиса около 'проходят = '934d980be49bbe4694aae578cd90cf86''в строке 1
вот полный перевод гуглом

Да английский я и без переводчика знаю =)
Теперь всё таже трабла, что и с самого начала.

<?php
session_start();

if(isset($_POST['enter']))
{
require_once("connect.php");
	/* Пинимаем данные из формы */
	$pass	=	trim($_POST['pass']);
	$email	=	trim($_POST['email']);
	/* Проверяем на заполнение */
	if(empty($pass) || empty($email))
	{
		echo 'Заполните все поля';
	}
	else
	{
	
		/* Проверяем на существование пользователя */
		$pass=md5($pass);
		$result = mysql_query(
		"SELECT `email`, `pass`
         FROM `USERS`
         WHERE `email` = '".mysql_real_escape_string($email) . "' AND
               `pass` = '" . mysql_real_escape_string($pass) . "'"
		) or die(mysql_error());


		if (mysql_num_rows($result))
		{
			// записываем переменные в сессию
			$line = mysql_fetch_row($result);
			$_SESSION['autorized']=true;
			$_SESSION['email']=$email;
			$_SESSION['role']=$line[2];
			// выводим ссылку для входа в админку (можно сделать перенаправление)
			echo "<center><h4>Авторизация прошла успешно</h4><br><a href=index.php>Вернуться на главную</a><br><br><a href=’admin.php’>Войти в админку</a></center>";
		}
		else
		{
		echo 'Логин или пароль не верны!';
		}
	}
}
else
{
?>
<center>
	<h4>АВТОРИЗАЦИЯ</h4><br>
	<form action="" method="post">
		<h4>Имя:</h4>
		<input type="text" name="email" value=""><br><br>
		<h4>Password:</h4>
		<input type="password" name="pass"><br><br>
		<input type="submit" name="enter" value="Войти">
	</form>
</center>
<?php
}
?>

Он выводит теперь постоянно, что логин и пароль не верны.
Но если if (mysql_num_rows($result)) меняем на if (!mysql_num_rows($result)), то авторизуются абсолютно все =)

Как такое правится? ответ руками я уже слышал =)

так когда в базу прописываешь ты пароль также
хешируешь ? $pass=md5($pass);
может там нет такого пароля или он не совпадает ?)))))))
гГг=== согласно скина у тя в базе ведь нет ничего

Это лишнее

`pass` = '" .$pass. "'"

Попробуй вообще без mysql_real_escape_string написать в запросе

---

Спустя 1 минута, 6 секунд PHPprogramer написал(а):

---

И пропиши вот так на всякий случай

echo $pass	=	md5(trim($_POST['pass']));
echo 	$email	=	trim($_POST['email']);

И сравни с тем, что в базе

PHPprogramer
тоже самое. проверил что выводит.
Тоже самое что и в базе.

$pass=md5($pass);
		$result = mysql_query(
		"SELECT `email`, `pass`
         FROM `USERS`
         WHERE `email` = '".mysql_real_escape_string($email) . "' AND
               `pass` = '" . mysql_real_escape_string($pass) . "'"
		) or die(mysql_error());

тут все дело в кавычках вы их перепутали задом на перед
щас подравлю

теперь всё

$pass=md5($pass);
		$result = mysql_query(
		"SELECT `email`, `pass`
         FROM `USERS`
         WHERE `email` =" '.mysql_real_escape_string($email) . '" AND
               `pass` = "' . mysql_real_escape_string($pass) . '"
		) or die(mysql_error());

надо лишь найти отличия

Может у вас включены магические кавычки ?

---

Спустя 42 секунды PHPprogramer написал(а):

---

YVSIK

покажи connect.php - а то я смотрю в базе кодировка ср1251

да каккие магические кавычки то
вы местами поменяли эти "c '
сам попутал сдаюсь

<?php
$sqlhost="localhost";
$sqluser="root";
$sqlpass="";
$db="WorldVote";

//Подключение к MySQL
mysql_connect ($sqlhost, $sqluser, $sqlpass) or die ("MySQL не доустпен ".mysql_error());
//Подключение к БД
mysql_select_db ($db) or die ("Нет соединения с БД ".mysql_error());
?>

Коннект рабочий, проверял на скрипте регистрации

=" '.mysql_real_escape_string($email) . '" AND
               `pass` = "' . mysql_real_escape_string($pass) . '"

ржу ни магу

Возможно база в одной кодировке, а скрипт отдает в другой

<?php
$sqlhost="localhost";
$sqluser="root";
$sqlpass="";
$db="WorldVote";

//Подключение к MySQL
mysql_connect ($sqlhost, $sqluser, $sqlpass) or die ("MySQL не доустпен ".mysql_error());
//Подключение к БД
mysql_select_db ($db) or die ("Нет соединения с БД ".mysql_error());
mysql_query('SET NAMES utf8');
?>

Я дописал кодировке в конце кода, поменяйте utf8 на ту которая у вас, если что

$db="WorldVote";

а база как зоветься к которой вы цепляетесь ???

 FROM `USERS`

пароль в запросе где хешируется? что-то не понятно....
покажи скрин содержимого из phpmyadmin и дамп запроса сделай.

$pass=md5($pass); // вот этого в коде я не увидел
$q = "SELECT `email`, `pass`
         FROM `USERS`
         WHERE `email` = '".mysql_real_escape_string($email) . "' AND
               `pass` = '" . $pass . "'";
echo '<pre>'; print_r($q); echo '</pre>'; // смотрим, какой запрос получился и сравниваем со скриншотом таблицы
$result = mysql_query($q) or die(mysql_error());

Кодировка сайта, какая у тебя, и базы ?

FROM `USERS`
а зоветься она как ???
в запросе ты к другой цепляешся к другой

База у тебя в windows-1251 ?

mysql_query('SET NAMES cp1251');

не ну сделай так в самом начале

include 'connect.php';
$res = mysql_query ("SELECT * FROM `USERS`");
while = ($row = mysql_fetch_assoc ($res))
{
echo $row ['pass'].'<br />';
}

нужно прописать SET NAMES, как написал PHPProgramer, а также сам файл со скриптом желательно чтоб cp1251 тоже был
кстати, после смены кодировки БД, при том, что она наполенена данными в другой кодировке бывают бока, насколько я слышал

Invis1ble
PHPprogramer
Кодировку указал, как говорите, но проблема остаётся =(

include 'connect.php';
$res = mysql_query ("SELECT * FROM `USERS`");
while = ($row = mysql_fetch_assoc ($res))
{
echo $row ['pass'].'<br />';
}

Ругается на 4ю строку - while = ($row = mysql_fetch_assoc ($res))
Parse error: syntax error, unexpected '=', expecting '(' in W:\home\localhost\www\123\login.php on line 4

= лишнее, очепятка

while ($row = mysql_fetch_assoc ($res)) знак равно убери - блин тут редактор неудобный

934d980be49bbe46
934d980be49bbe46

Warning: session_start() [function.session-start]: Cannot send session cache limiter - headers already sent (output started at W:\home\localhost\www\123\login.php:6) in W:\home\localhost\www\123\login.php on line 8

и покажи все таки 2 скрина, 1 - со структурой, 2-ой - с данными

тогда я вообще ничего не понимаю

http://clip2net.com/clip/m56917/1310565704-clip-45kb.png - структура
http://clip2net.com/clip/m56917/1310565733-clip-43kb.png - данные

хеш пароля в БД 16-битный, видимо обрезанный, либо зашифрованный по другому алгоритму
в этом вся проблема

Скопируйте хеш пароля с базы и сравните его в скрипте

if($pass == 'a2d5f6.....')
    echo 'OK';
else
    echo 'bad';

PHPprogramer
да что там сравнивать, и так видно

ну ёлки палки - такая банальщина

тогда вопрос, как её поправить? =-)

короче это md5-хеш обрезанный пополам

$q = "SELECT `email`, `pass`
         FROM `USERS`
         WHERE `email` = '" . mysql_real_escape_string($email) . "' AND
               `pass` = '" . substr(md5($pass), 0, 16) . "'";
echo '<pre>'; print_r($q); echo '</pre>'; // смотрим, какой запрос получился и сравниваем со скриншотом таблицы
$result = mysql_query($q) or die(mysql_error());

длину у поля pass сделай 32

Во разфлудились мы здесь, аж форум ложится

нет, он же писал, что выводит скрипт Админа
и поле на скрине varchar(16)

Такс. подтер в базе записи. Начал регить их заново, слетела регистрация, не регит, 5 минут назад всё было ок. щас найду где косяк, и попробую с пасом разобраться

И поставь для поля pass varchar(32)

УРА, теперь всё работает. с меня каждому + в карму;)