system logowania - problem Opcje

[areli]

mam kilka pytań... aktualnie średnio sobie radzę w programowaniu obiektowym, więc potrzebuje waszej pomocy...

zrobiłem system logowania z mini panelem i problem jest taki:

1) jak się zaloguje lub wyloguje, to muszę odświeżyć stronę żeby dopiero zobaczyć efekt... co jest nie tak?

2) chciałbym żeby kod był przejrzysty i elegancko napisany, a mam podejrzenia że dużo w tym bałaganu... czy można by napisać to ciekawiej... może krócej i efektowniej?

[PHP] pobierz, plaintext 
<?php
 
class Logowanie
{
    var $login;
    var $pass;
    var $id;
    var $db;
    var $wynik;
    var $ile;
    
    function __construct()
    {
        if (isset($_SESSION['zalogowany']))
        {
            echo "<p>Jesteś zalogowany. Twój login to: ".$this -> login."</p>
            <br />";
            echo " | "."<a href='panel/index.php?menu=4'>Panel</a>"." | "."
            <a href='".$_SERVER['PHP_SELF']."?logout=yes'>Wyloguj</a>"." | ";
        }
        else 
        {
            echo "<form action='".$_SERVER['PHP_SELF']."' method='post'>
                <p>Nie jesteś zalogowany.</p>
                <p>
                    <label>Login: </label><input type='text' size='10' name='konto' />
                    <label>&nbsp;&nbsp;Hasło: </label><input type='password' size='10' name='password' />
                    <input type='submit' value='>>' name='p_log' />
                </p>
            </form>";
            
            $this -> Uwierzyt();
        }
        
        if (isset($_GET['logout']) && ($_GET['logout'] == 'yes'))
        {
            $this -> wyloguj();
        }
    }
    
    public function Uwierzyt()
    {
        if (isset($_POST['konto']) && isset($_POST['password']) )
        {
            $this -> login = trim($_POST['konto']);
            $this -> pass = trim($_POST['password']);
            //    $pass = sha1(md5($pass));
 
            if (empty($this -> login) || empty($this -> pass))
                echo"Błąd: Wypełnij wszystkie pola!";
            else
                $this -> sql_connect();
        }
    }
    
    public function sql_connect()
    {
        @$this -> db = mysqli_connect (DB_HOST, DB_USER, DB_PASS, DB_NAME);
        if (!$this -> db)
            echo "Połączenie z serwerem bazy danych nie powiodło się!";
        else
        {
            $zapytanie = "SELECT id_log FROM user WHERE login='$this->login' and passbaz='$this->pass'";
            $this -> sql_query ($zapytanie);
        }
    }
    
    public function sql_query ($zapytanie)
    {
        if (!$this -> wynik = mysqli_query($this -> db, $zapytanie))
            echo "Połączenie z serwerem bazy danych nie powiodło się!";
        else
            $this -> sql_ile();
    }
    
    public function sql_ile()
    {
        if (!$this -> ile = mysqli_num_rows ($this -> wynik))
            echo "Błędny login lub hasło. Spróbuj jeszcze raz!";
        else
            $this -> sql_fetch_array();
    }
    
    public function sql_fetch_array()
    {
        $tab = mysqli_fetch_assoc ($this -> wynik);
        
            $this -> id = $tab['id_log'];
            $_SESSION['id'] = $this -> id;
            $_SESSION['zalogowany']= $this -> login;
    }
    
    public function wyloguj()
    {http://forum.php.pl/system_logowania_problem_t112090.html#
        $usuw_uzytk = $_SESSION['zalogowany'];
        unset($_SESSION['zalogowany']);
        unset($_SESSION['id']);
        session_destroy();
    }
    
}
?>
[PHP] pobierz, plaintext 

z góry dzięki za pomoc,
pzdr.

Ten post edytował areli 30.01.2009, 11:42:16

[Crozin]

może jestem w błędzie, ale jakoś średnio jestem przekonany do bbcode...

Nie stosując BBCode dla kodu PHP zniechęcasz do jego analizowania, a nawet to uniemożliwiasz - zstanów się jeszcze raz.

ponadto taki tekst moderatora wydaje mi się nie na miejscu

Popraw bbcode na [php] w przeciwnym wypadku temat zostanie zamkiety

Niestosowne? Jasne, proste polecenie + uprzejma informacja o skutkach niedostosowania się

odpowiedź Crozin’a to sugestie i własne przemyślenia... niewykluczone że ma rację, ale ja nie jestem zobowiązany stosować się do wszystkiego w 100%

Własne przemyślenia (mocno nietrafiony zwrot) to co najwyżej punkt 4-ty. Pierwszy to sprawy "techniczne", dwa następne to poprostu błędy, 4-ty - tutaj wspomniane własne "przemyślenia", a na końcu znowu błąd.

najlepiej kogoś kto w sposób praktyczny potrafiłby pokazać jak powinien wyglądać mój system logowania...

Czyli mamy Ci niby taki przykładowy system napisać?

s. czy do oddzielenia kodu użyć jakiegoś framework’a?

Na podstawie tego wątku mogę stwierdzić, że na zabawę z FW jest dla Ciebie stanowczo za wcześnie.