[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]

Chodziło mi o aplikacje na urządzenia mobilne. Dużo stron posiada takie aplikacje, które ułatwiają korzystanie z nich, np. FB, nk, Kwejk, Demotywatory, itp.

Natrafiłem na pewien problem w kodzie. Otóż - muszę stworzyć klasę, która sprawdza, czy dane zostały przesłane z formularza (następuje ich weryfikacja) czy nie (wyświetlenie formularza). W poprzednim kodzie sprawdzałem czy którakolwiek ze zmiennych globalnych nie została wysłana. W nowym kodzie mogę stworzyć dowolną ilość kontrolek formularza o dowolnych nazwach. Jak więc zrobić taką klasę? Może po prostu zrobić to już poza klasami?

Faktycznie, ściągnąłem starszą wersję. W sumie za bardzo nie wczytałem się na ich stronie, ale na pierwszy rzut oka widzę, ze jest trochę zabawy z instalacją Symphony 2. Trzeba zainstalować Compozera, itp. W Zend pewnie też są w miarę porządnie napisane klasy walidacji danych. Jutro ściągnę Zenda albo ę Symphony 2, bo dzisiaj jestem już zmęczony.

Trochę poprawiłem kod, ale na razie zrobiłem tylko szkielet.

[PHP] pobierz, plaintext 
<?php
	class form{
		private $name, $value;
 
		public function setName($name){
			$this->name=$name;
		}
		public function getName(){
			return $this->name;
		}
		public function setValue($name){
			$this->value=$value;
		}
		public function getValue(){
			return $this->value;
		}
 
		public function newForm($method, $file){
			echo '<form action="' . $file . '" method="' . $method . '" />';
		}
 
		public function finish(){
			echo '</form>';
		}
	}
 
	class inputText extends form{
		public function createText(){
			return '<input type="text" name="' . $this->getName() . '" />';
		}
	}
 
	class inputPass extends form{
		public function createPass(){
			return '<input type="password" name="' . $this->getName() . '" />';
		}
	}
 
	class inputSubmit extends form{
		public function createSubmit(){
			return '<input type="submit" value="' .$this->getValue(). '" />';
		}
	}
	class validation{
		private $variable, $type;
 
		public function setValidation($variable, $type){
			$this->variable = $variable;
			$this->type = $type;
		}
		public function validateForm(){
			htmlspecialchars($this->variable); //na razie nie łączę się z bazą danych, więc powinno mi to wystarczyć
			switch ($this->type){
				case 1: //nr 1 oznacza login
					{
						if (strlen($this->variable) < 3){
							return false;
						} else {
							return true;
						}
					}
				//case 2:
				//etc
			}
 
		}
	}
 
	class register{
		private $login, $pass;
 
		public function setRegister($login, $pass){
			$this->login = $login;
			$this->pass = $pass;
		}
 
		public function finalRegister(){
			return 'Zarestrowałem konto o nicku ' . $this->login . '. Jego hasło to: ' . $this->pass;
		}
	}
 
if (empty($_POST['login']) || empty($_POST['pass']) ){
	$form = new form;
		$form->newForm('POST', '8.php');
		$login = new inputText;
		$login->setName('login');
		echo $login->createText();
		$pass = new inputPass;
		$pass->setName('pass');
		echo $pass->createPass();
		$submit = new inputSubmit;
		$submit->setValue('Wyslij');
		echo $submit->createSubmit();
	$form->finish();
} else {
	$valid = new validation;
	$valid->setValidation($_POST['login'], 1);
	if ($valid->validateForm() == false){
		echo "Nie moge zarejestrowac tego usera";
	} else {
		$reg = new register;
		$reg->setRegister($_POST['login'], $_POST['pass']);
		echo $reg->finalRegister();
	}
}
?>	
[PHP] pobierz, plaintext 

Nikt już się nie wypowie?

Ten post edytował Testosteron 6.07.2014, 19:10:17