rejestracja, sprawdzenie Opcje

[tukan]

Witam,
Mam dla Was do sprawdzenia początek rejestracji. Nie ma tu jeszcze zapisu do bazy danych, ale mam gotową walidację loginu i hasła. Został e-mail, ale to będzie analogicznie to loginu. Podaję kod:

[PHP] pobierz, plaintext 
<?php
function connect() {
	try {
$host = "localhost";
 $dbname = "sfwyf";
$user = "root";
$pass = "vertrigo";
 
		$dbh = new PDO("mysql:host=$host; dbname=$dbname", $user, $pass);
 
		echo "Połączenie nawiązane";
	return $dbh;
} catch ( PDOException $error) {	die("Failed to connect:" . $error->getMessage());
}
 
}
 
function is_availble_login($dbh, $desired_login) // ta funkcja, bo jeszce będę wykorzystywał przy sprawdzaniu ajaxowym, to samo będzie dla meila
{
		$dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
			$sql = "SELECT * FROM users WHERE username =:username)";
		$q = $dbh->prepare($sql);
		try{		$count_row = $q->execute(array(':username'=>$desired_login));}
 
		catch(PDOException $e) { print_r( "checking.... ".$e->getTrace() );}
		if ($count_row->fetchColumn() > 0) {
		return false;
		else return true;
		}
}
class r_User{
public $id;
public $pass;
public $pass2;
public $name;
public $username;
public $email;
public $permission_level;
public $validate = true;
public $registration_error = array();
public $registration_error_id = array();
	function valid_form{
	if(this->$pass==$this->pass2){
 
	$this->pass = trim($this->pass);
	if( !preg_match('^(?:[0-9a-zA-ZąćęłńóśźżĄĆĘŁŃÓŚŹŻ!@#$%^&*()_-=+[]{}|.]{5,15})$', $this->pass) 
	{
		array_push($registration_error, "Podane hasło nie jest poprawne. Musi posiadać od 5 do 15 znaków. Hasło może zawierać następujące znaki: cyfry; litery; oraz następujące znaki: !@#$%^&*()_-=+[]{}|.")
		array_push($regisration_error_id, 0) // żebym wiedział, które pole na czerwono}
		$validate = false;
	}
 
 
	}
	else {array_push($registration_error, "Podane hasła nie zgadzają się");
	array_push($regisration_error_id, 0) // żebym wiedział, które pole na czerwono}
			$validate = false;
 
		}
	//teraz login
 
	$this->username = trim($this->username);
	if( !preg_match('^(?:[0-9a-zA-ZąćęłńóśźżĄĆĘŁŃÓŚŹŻ._-!@#$&]{5,15})$', $this->username)
	{
	array_push($registration_error, "Podany login jest niepoprawnyasło nie jest poprawne. Musi posiadać od 5 do 15 znaków. Hasło może zawierać następujące znaki: cyfry; litery; oraz następujące znaki: ._-!@#$&")		
	array_push($regisration_error_id, 1) // żebym wiedział, które pole na czerwono}
		$validate = false;
	}
	else{ 
	if(is_availble_login($dbh, $this->username))
	{
		//wpisuje do bazy
	}
	else{
	array_push($registration_error, "Ten login jest już zajęty");
	array_push($regisration_error_id, 1) // żebym wiedział, które pole na czerwono}
			$validate = false;
	} 
 
 
	}
 
 
 
	}
	else {array_push($registration_error, "Podane hasła nie zgadzają się");
	array_push($regisration_error_id, 0) // żebym wiedział, które pole na czerwono}
			$validate = false;
 
		}
}
 
 
 
 
}
 
 
 
?>
[PHP] pobierz, plaintext 

Możliwe, że gdzieś "ify" mogą zachodzić na siebie, ale to poprawię. Chodzi o koncepcje walidacji, jak ją poprawić/uoptymalnić?

Ten post edytował tukan 14.07.2012, 22:52:05

[Orzeszekk]

Jeśli zerwiesz z syndromem - "Co nie ja napisałem, to niedobre" to możesz odkryć że ogólnodostępne frameworki PHP takie jak cakePHP lub Symfony 1/2 posiadają bardzo rozbudowane i wygodne walidatory.

Serio, szkoda sie bawic w wynajdowanie koła na nowo. Wbrew pozorom nie nauczysz się podczas tego zbyt dużo.

Ja myślę że walidacja jest paskudnie zrobiona. Wszystkie walidatory masz na stałe wpisane w metode validForm. Gdy przybędzie ci tego trochę, to kod wewnatrz tej metody rozrosnie sie na 30 ifów i case-ów. Trzeba robic rzeczy generycznie, tak by nie robic copy-paste jak bedziesz chcial to uzyc w innym miejscu.

Powinieneś utworzyć przynajmniej 2 klasy oparte na interfejsach podobnych do tych:

interface ValidatorEngine
{
function validateForm(Form $form);
function addValidator(Validator $validator);
}

interface Validator
{
function doValidate(ValidateFieldData $data);
}

w klase ValidateFieldData pakujesz wszystkie dane jakie musisz przeslac z formularza do walidatora, z metody doValidate zwracasz rezultat walidacji, dodajesz walidatory do validatorEngine i wstrzykujesz tam formularz.

Mozesz ten system pocisnac dalej i rozszerzyc interfejs Validator o metode w stylu getJavascript() która zwroci kod javascript potrzebny do przeprowadzenia danej walidacji rowniez po stronie klienta dla responsywnosci, i za pomoca validatorEngine uzbierany w tej sposob JS doklejac do widokow formularzy.

A wtedy ci wyjdzie system walidatorow jaki masz w symfony i wyjdzie na to ze nie warto bylo tego wszystkiego pisac, by sobie zwalidowac pare formularzy.

Ten post edytował Orzeszekk 14.07.2012, 23:37:59