[PHP] Skrypt rejestracji (OOP) - prośba o poprawę kodu Opcje

[Testosteron]

Witajcie!

Chcę nauczyć się programowania obiektowego. Niestety, kursy internetowe opierają się na nieco głupich przykładach, w stylu:
- zaszczekaj burku
- hau hau
Nie ma to przełożenia na realne problemy z jakimi stykamy się w programowaniu. W mojej książce, "PHP i MySQL. Vademecum profesjonalisty" także programowanie obiektowe nie zostało dobrze opisane. Postanowiłem więc spróbować napisać coś prostego. W moim przypadku był ro system rejestracji. Skrypt działa w 100%, chociaż nie jest dokończony (nie ma walidacji hasła, a walidacja loginu też nie jest w pełni skończona). Chodzi mi raczej o samą zasadę działania. Od razu mówię, że jest to mój PIERWSZY, obiektowy projekt pisany bez żadnego wzoru. Zależy mi na tym, żeby ktoś poradził, co należy zmienić.

[PHP] pobierz, plaintext 
<?php
/*    class connect{
        function __construct(){
        $polaczenie = @new mysqli('localhost', 'root', 'haslo', 'login');
        if (mysqli_connect_errno() != 0){
            echo '<p>Wystąpił błąd połączenia: ' . mysqli_connect_error() . '</p>';
        } 
    }
}
*/
 
    class formularz{
        function wyswietl(){
            echo"<table>
                    <form action='register.php' method='post'>
                        <tr><td>Login:</td><td><input type='text' name='login'></td></tr>
                        <tr><td>Password:</td><td><input type='password' name='pass'></td></tr>
                        <tr><td><input type='submit' value='zarejestruj'></td></tr></form>
                </table>";
        }
 
    }
 
    class walidacja extends formularz{
 
        //dodawanie usera
        function __construct(){
            $mysqli = new mysqli('localhost', 'root', 'haslo', 'login');
            $login = $mysqli->real_escape_string($_POST['login']);
            $pass = $mysqli->real_escape_string($_POST['pass']);
 
            if (empty($login) || empty($pass) ){
                $this->wyswietl();
            } else {
                if ($this->sprawdz_login($login) ){
                    $query = $mysqli->query("INSERT INTO login VALUE('" . $login . "', '" . $pass . "')");
                    if ($query){
                        echo 'user został dodany';
                    } else {
                        echo 'blad';
                    }
                }
 
            }
        }
 
        //Poprawność loginu
        function sprawdz_login($login){
            $login = $_POST['login'];
            if (strlen($login) < 4){
                echo 'login musi miec przynajmniej 4 znaki';
            } else {
                return true;
            }
        }
    }
 
 
 
    $formularz = new walidacja;
 
?>
[PHP] pobierz, plaintext 

Na początku chciałem napisać klasę connect (obecnie jest zawarta w komentarzu), jednak miałem problem z wykorzystaniem jej, tzn. zależy mi, żeby w jednej klasie zawrzeć wszystkie dane dotyczące logowania do mysql i później żebym mógł w innych klasach wykorzystywać wszystkie metody mysqli. Niestety miałem z tym problem. Jeżeli ktoś wie jak to zrobić, to byłbym także wdzięczny za odpowiedź.

pozdrawiam (IMG:style_emoticons/default/smile.gif)

Ten post edytował Testosteron 3.07.2014, 17:21:48

[SmokAnalog]

Po pierwsze takie małe kosmetyczne sprawy:

• Nazwy klas zwyczajowo powinny zaczynać się od wielkiej litery
• Nazywaj klasy, zmienne i metody po angielsku

Oprócz tego masz logikę dziwną. Klasa walidacja nie rozszerza w logiczny sposób klasy formularz. To powinna być jedna klasa, albo klasa walidacja powinna przyjmować obiekt formularza jako parametr np. w konstruktorze. Ewentualnie na odwrót, ale tak wydaje mi się to bardziej rozsądne, bo klasa formularz ma sens bez walidacji, a walidacja bez formularza już niezbyt.

Fajnie jak pouczysz się wyjątków, bo wyrzucanie błędów w echo to bardzo zły pomysł.