Oggy
ерунда какая то, давай конкретные параметры, по которым надо проверять

<?php

if(isset($_POST['login'],$_POST['password']))
    {
        $login = $_POST['login'];
        $password = $_POST['password'];
            
        $this->admin_connect_bd();
        $result = mysql_query('SELECT login,password FROM `adm`');
    if ($result)
        {
            while($row = mysql_fetch_array($result)) 
            {
                $logg = $row['login'];
                $paroll = $row['password'];
            }    
    if ($logg == $_POST['login'] and $paroll == $_POST['password'])
        echo 'ok';
          else
        echo 'no';
   
        }
    }
    
?>

передавай попову приветы

johniek_comp
ахах) попов и классы? что то новенькое ты говоришь) если инициализация гет переменных происходит не так - это не значит, что ученик попова

nugle
ТС хочет свою процедурку на какой-то готовый код положить, естественно перед этим просмотрев его видеоуроки художества

1.
$login = $_POST['login'] ? $_POST['login'] : '';
$password = $_POST['password'] ? $_POST['password'] : '' ;

2.
$sql = 'SELECT login,password FROM `adm` WHERE login = "' . mysql_real_escape_string($login) . '" AND password = "' . md5(mysql_real_escape_string($password)) . '"';

Написано очень плохо.

Лена
а если мы выставим error_reporting(ALL); то что? то ошибки появятся в твоем коде

Oggy

if(isset($_POST['login'],$_POST['password'])

- это всегда будет давать true. Лучше сразу проверять на пустоту:

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

Вот этот запрос:

$result = mysql_query('SELECT login,password FROM `adm`');

- выберет вам логины и пароли всех пользователей из таблицы 'adm'. Если хотите проверить пароль какого-то конкретного пользователя, нужно добавить условие:

$result = mysql_query("SELECT login,password FROM `adm` WHERE login = '{$login}'");

Вот эта конструкция в вашем случае вообще смысла не имеет:

while($row = mysql_fetch_array($result)) 
            {
                $logg = $row['login'];
                $paroll = $row['password'];
            }
 

, т.к. значения переменных $logg и $paroll будут каждый раз переписываться заново. Соответсвенно вот здесть вы никогда не получите "ok" (если только не введете данные юзера, находящегося последним в базе данных):

if ($logg == $_POST['login'] and $paroll == $_POST['password'])
        echo 'ok';
 else
        echo 'no';

В общем, настоятельно советую начать читать вот этот курс: http://irbis-team.com/15.

Лена
md5(mysql_real_escape_string($password)) - mysql_real_escape_string здесь не нужен. Строка, закодированная в md5 вреда базе данных не причинит.

johniek_comp
Ты в ударе сегодня как я погляжу...

/*добавляем проверку на длину  ЛОГИНА*/
if (strlen($loginl) < 3 or strlen($loginl) > 15) 
/*---*/{/*---*/ 
	    unset($loginl);
	    echo '"Логин" должен состоять от 3 и до 15 символов  <br >';
/*---*/}/*---*/
/*обрезаем на лишние пробелы по краям*/
$loginl= trim ($login) ;

/*проверка на не соответствие такого-же в базе */
 $result = mysql_query("SELECT `login` FROM `matr_users` WHERE `login` = 
                         '".mysql_real_escape_string($login)."'");
       if(mysql_num_rows($result)>0)

//обработка на невыполнение скриптов
$loginl= htmlspecialchars($login) ;

//и для записи в базу это 
'".mysql_real_escape_string($login)."'

фонтазии вского рода можно добавть

walerus

Согласен, забыл про пустоту.
Хотя в данном случае это не так важно, ведь запрос в БД с пробелом вместо логина или пароля просто вернет false.

<?php 
If(!empty(trim( $_POST['login'] )) && !empty(trim( $_POST['password'] )))
  {
    // Обработка данных
    $login = $_POST['login'];
    $password = $_POST['password']; 
    // И так далее...
  
  }
?>

wplounge +1

и еще свой кусочек добавлю.

session_start();
if(isset($_REQUEST['login'])){
    $error = array(); //массив куда будут складываться ошибки
    //все возможные проверки на пустоту и соответствие

    if(count($array) == 0){
        $sql = "SELECT login,pass FROM `table` WHERE `login` =  '".mysql_real_escape_string($_REQUEST['login'])."' LIMIT 1";
        $result = mysql_query($sql) or die('Неправильный запрос...');
        $row = mysql_fetch_assoc($result);
        if($row['id'] != ''){
            if($row['pass'] == mysql_real_escape_string($_REQUEST['pass'])){
                $_SESSION['login'] = mysql_real_escape_string($_REQUEST['login']);
                header('location:index.php');
            }else{
                $error['pass'] = 'Неверный пароль';
            }
        }
    }else{
        foreach($error as $value)
            echo $value.'<br/>';
    }

}

m4a1fox
ну так с бана вышел, на нервах весь :)

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

if(trim($text) && !empty($text))
    {
        // ваши действия
    }

универсально :)