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]

1) Kod wstaw w BBCode [ php ]
2) Nie stosuj elementów PHP4
3) Nie zrobiłeś klasy do logowania. Zrobiłeś klasę boga, która robi ileś tam rzeczy zamiast jeden
4) Stosuj wyłącznie język angielski w nazwach zmiennych, metod itd.
5) Co to za dziwny pomysł, żeby klasa odpowiedzialna za logowanie/wylogowanie wyświetlała jakiś kod HTML strony? Ona w ogóle ani grama HTML, ani jeden istrukcji echo nie powinna mieć

[areli]

no ok. wszystko jasne, ale nadal nie wiem co z tym odświeżaniem strony i jak w praktyce miałbym zastosować się do pt.2 i 3... może jakieś konkretne rozwiązanie?

[nospor]

no ok. wszystko jasne

Skoro wszystko jasne, to czemu nie zastosowales sie do:

Kod wstaw w BBCode [ php ]

?

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

[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.

[nospor]

ale ja nie jestem zobowiązany stosować się do wszystkiego w 100%

Rejestrując sie na forum zatwierdziłeś jego regulamin. Skoro nie chcesz sie do niego stosować to nie. temat zamykam

ponadto taki tekst moderatora wydaje mi się nie na miejscu

hihi...
Jak ci policjant wlepi mandat za szybką jazdę, to też pewnie uznasz to nie na miejscu....

edit: otwieram

[Kildyt]

Pozwolę się odnieść do problemu autora. W programowaniu obiektowym nie jestem najlepszy, ale spróbuję.

To co już napisał Crozin.
W programowaniu obiektowym właśnie chodzi o to, żeby działać na obiektach. Obiektem nie jest (jako sam) komputer, a podzespoły (klawiatura, mysz itd.).
Stwórz kilka klas. Pierwszą odpowiedzialną za wyświetlanie tekstu i wyglądu, drugą odpowiedzialną za połączenie z bazą i operacje na niej, trzecią właśnie z systemem logowania. Czy tak nie łatwiej? Masz wszystko podane jak "na tacy" i uporządkowane.

Jeżeli chodzi o to, że stronę musisz odświeżyć to powód jest prostu: za to, czy ktoś jest zalogowany jest odpowiedzialna wartość w zmiennej _SESSION.

Co do nie stosowania kodu PHP 4 to wytłumaczenie jest bardzo proste. Popatrz: mamy już wersję PHP 5, a za niedługo wyjdzie PHP 6. Jak wyobrażasz sobie używanie przestarzałych funkcji? Może i kod teraz zadziała, ale co z nim za rok? Czytaj dokumentację. Tam pisze, która funkcja straci wsparcie w nowszych wersjach PHP.

Nie wszystkie zmienne w klasie to $this ->. Jeżeli masz zmienną, którą używasz tylko w obrębie funkcji nie bajeruj.


Mam nadzieję, że chociaż troszkę pomogłem.

Ten post edytował Kildyt 15.02.2009, 20:47:53

[megawebmaster]

Ja sobie pozwolę odpowiedzieć tylko na brak przekierowania: Header()