Имеет ли право на жизнь такая авторизация?

<?php
//check.php
    require_once("config.php");
    $login = $_POST['login'];
    $password = $_POST['password'];
    $result = mysql_query ("SELECT ID FROM users WHERE login=$login and password=$password");
    $id = mysql_fetch_assoc($result);
    session_start();
    $_SESSION['ID'] = $id;
    if($_POST['Submit']){
        if(($_POST['login']==$login)&&($_POST['password']==$password)){
            $_SESSION['logged'] = $_POST['login'];
            header("Location: index.php");
            exit;
        }
    }
?>

<?php
 //index.php
    unset($_SESSION['logged']);
    session_start();
    if(!isset($_SESSION['logged'])){
        require_once("config.php");
        $_SESSION['ID'] = $id;
        $result = mysql_query ("SELECT role FROM users WHERE ID=$id");
        $role = mysql_fetch_assoc($result);
        for ($role = "Пользователь") echo "Этот текст видит только пользователь";
        for ($role = "Бухгалтер") echo "Этот текст видит только бухгалтер";
        for ($role = "Техник") echo "Этот текст видит только техник";
        for ($role = "Администратор") echo "Этот текст видит только администратор";
    }
?>

$login = $_POST['login'];
// ...
if ($_POST['login']==$login)

for ($role = "Пользователь") echo "Этот текст видит только пользователь";
        for ($role = "Бухгалтер") echo "Этот текст видит только бухгалтер";
        for ($role = "Техник") echo "Этот текст видит только техник";
        for ($role = "Администратор") echo "Этот текст видит только администратор";