Problemy z walidacją OOP Opcje

[Panicz74]

Witam,

Mam problem z formularzem rejestracji. Walidacja działa jedynie w przypadku pola 'username', w pozostałych przypadkach niestety nie. Zupełnie nie mam pojęcia gdzie może być błąd. Może wy coś doradzicie?

register.php

[PHP] pobierz, plaintext 
<?php
 
require_once 'core/init.php';
 
if(Input::exists())
{
	$validate = new Validate();
	$validation = $validate->check($_POST, array(
		'username' => array(
			'required' => true,
			'min' => 2,
			'max' => 20,
			'unique' => 'users'
		),
		'password' => array(
			'required' => true,
			'min' => 6,
		),
		'password_again' => array(
			'required' => true,
			'matches' => 'password'
		),
		'name' => array(
			'required' => true,
			'min' => 2,
			'max' => 20
		),
	));
 
	if($validation->passed())
	{
		echo "Registered";
	}
	else
	{
		print_r($validation->errors());
	}
}
 
?>
[PHP] pobierz, plaintext 

validate.php

[PHP] pobierz, plaintext 
<?php
class Validate
{
	private $_passed = false;
	private $_errors = array();
	private $_db = null;
 
	public function __construct()
	{
		$this->_db = DB::getInstance();
	}
 
	public function check($source, $items = array())
	{
		foreach($items as $item => $rules)
		{
			foreach($rules as $rule => $rule_value)
			{
				$value = $source[$item];
 
				if($rule === 'required' && empty($value))
				{
					$this->addError("{$item} is required");
				}
				else
				{
					$this->_passed = true;
				}
				return $this;
			}
		}
		if(empty($this->_errors))
		{
			$this->_passed = true;
		}
		//return $this;
	}
 
	private function addError($error)
	{
		$this->_errors[] = $error;
	}
 
	public function errors()
	{
		return $this->_errors;
	}
 
	public function passed()
	{
		return $this->_passed;
	}
}
?>
[PHP] pobierz, plaintext 

[JakubBab]

Pytanie do Twojego kodu :

Linijka:

[PHP] pobierz, plaintext 
						foreach($rules as $rule => $rule_value)
			{
				$value = $source[$item];
 
				if($rule === 'required' && empty($value))
				{
					$this->addError("{$item} is required");
				}
				..............bla bla bla
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
foreach($rules as $rule => $rule_value)
			{
				$value = $source[$item];
[PHP] pobierz, plaintext 

Powieli tablice zgodnie z liczba iteracji (3 lub 4 w tym przypadku). Czy tak mialo byc i czy to jest dobra praktyka? Nie zabiera to (zbednie) pamieci przez iterowanie 3 or 4 razy tego samego?

Ten post edytował JakubBab 20.02.2016, 14:48:43