[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

[Pyton_000]

Drobne uwagi co do kodu.
W klasach dziedziczących powinieneś mieć 1 metodę która nazywa się createInput (jego rodzaj definiuje przecież typ obiektu) Nie ma sensu tworzenia oddzielnych metod dla każdej z nich. Dodatkowo setName powinno się odnosić do formularza a nie do input. Nazwę input przekazuj w konstruktorze obiektu i ustawiaj w klasie. Dzięki temu masz każdy obiekt nazwany i nie musisz stosować setName, getName.

Poza tym ja bym zrobił coś na zasadzie:
- Jest sobie obiekt Form. Posiada on kolekcję inputów. coś na zasadzie:

[PHP] pobierz, plaintext 
$form = new Form('FormularzNowościowy');
$form->put(new InputText('nazwa'));
...
if(isset($_POST)) {
   $validateRules = new Validate()->addRule('nazwaInput', array('nazwa_reguly' => 'cos'));
   $form->validate($validateRules);
   ....
}
....
$form->show();
[PHP] pobierz, plaintext 

To tak na szybko.