[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

[Testosteron]

Nie wiem, czy dobrze Cię zrozumiałem. Mogę stworzyć klasę form, która odpowiada za zwracanie pól formularza. Jej metoda mogłaby przyjmować parametr dotyczący typu pola, czyli - text, password, itp. Inna metoda tej klasy mogłaby ogólnie tworzyć formularz, a więc, np. ustawiałaby metodę get, bądź post i wskazywała na plik do którego mają zostać przesłane dane. Kolejne klasy dziedziczyłyby z tej klasy i odnosiłyby się do konkretnych typów pól formularza. Zawierałyby np. walidację wprowadzonych przez użytkownika danych i zwracałyby true bądź false

Dodatkowo metoda klasy form sprawdzałaby, czy dane zostały wcześniej przesłane. Zwracałaby wartość true bądź false. Poniżej deklaracji wszystkich klas mógłbym stworzyć instrukcję warunkową, która sprawdzałaby, czy dane zostały przesłane. Jeśli nie (a więc odpowiednia metoda zwróci wartość false), tworzy obiekty typu - login, password, mail, itp. W przeciwnym wypadku następuje sprawdzenie poprawności danych i jeśli wszystko jest dobrze, następuje zapisanie ich w bazie danych. W takim układzie tworząc kolejne formularze na stronie (np. komentarze, wyszukiwarka), mógłbym skorzystać z klas stworzonych wcześniej. Oczywiście musiałbym dodać nowe zapytanie, być może nową klasę weryfikującą poprawność danych, ale pracy byłoby o wiele mniej.

Uważasz, że tak byłoby dobrze?

Ten post edytował Testosteron 4.07.2014, 19:23:54