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.

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ć

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?

Skoro wszystko jasne, to czemu nie zastosowales sie do:

?

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

Nie stosując BBCode dla kodu PHP zniechęcasz do jego analizowania, a nawet to uniemożliwiasz - zstanów się jeszcze raz.Niestosowne? Jasne, proste polecenie + uprzejma informacja o skutkach niedostosowania się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.Czyli mamy Ci niby taki przykładowy system napisać?Na podstawie tego wątku mogę stwierdzić, że na zabawę z FW jest dla Ciebie stanowczo za wcześnie.

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


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

edit: otwieram

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.

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