[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

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

[SmokAnalog]

Dziedziczenie jest wspaniałe i daje mnóstwo korzyści. Jedną z moich ulubionych jest tzw. polimorfizm. Jako że nie lubisz przykładów nieinformatycznych (ja np. wolę takie na psach, truskawkach itd. ), to spróbuję podać informatyczny. Ba, podam nawet przykład dla klasy formularzy.

Prędzej czy później będziesz chciał zrobić obiekty pól formularza. To jest doskonałe miejsce dla dziedziczenia, bo one będą miały jakieś części wspólne, na przykład:

[PHP] pobierz, plaintext 
class InputText extends InputField {...}
class InputCheckbox extends InputField {...}
// teraz uważaj:
class InputTextarea extends InputText {...}
[PHP] pobierz, plaintext 

Obiekt generujący textarea mógłby dziedziczyć po obiekcie generującym zwykłe pole tekstowe, bo będą miały prawdopodobnie wiele wspólnego, a jedną z niewielu różnic będzie metoda, która wypluwa ich HTML.

W klasie formularza mógłbyś mieć metodę, która dodaje pole na zasadzie:

[PHP] pobierz, plaintext 
$form = new Form;
$textarea = new InputTextarea;
 
$form->add($textarea);
[PHP] pobierz, plaintext 

A definicja metody add w klasie Form mogłaby wyglądać tak:

[PHP] pobierz, plaintext 
public function add(InputField $field) {...}
[PHP] pobierz, plaintext 

Tutaj działa nasz polimorfizm, bo mówisz metodzie add, że może przyjmować dowolne pole. Każda z naszych powyższych klas InputText, InputCheckbox i InputTextarea to instancje klasy InputField, bo każda po niej dziedziczy. InputText i InputCheckbox bezpośrednio, a InputTextarea dziedziczy po InputText, czyli też jest dzieckiem klasy InputField