zacząłem uczyć się programowania obiektowego, napisałem prostą klasę do walidacji formularzy

[PHP] pobierz, plaintext 
class form_valid {
 
		public function valid_email($value) {
			if (!preg_match('/^([^@s]+)@((?:[-a-z0-9]+.)+[a-z]{2,})$/i', $value)) {
				$this->error = 'Podany email jest nieprawidłowy.';
			}
		}
 
		public function show_error() {
			return $this->error;
		}
 
		public function is_valid() {
			return (count($this->error) > 0) ? FALSE : TRUE;
		}
 
	}
[PHP] pobierz, plaintext 

tak z niej korzystam:

[PHP] pobierz, plaintext 
if (isset($_POST['valid'])) {
		$email = $_POST['email'];
		$form_valid = new form_valid;
		$form_valid->valid_email($email);
 
		if ($form_valid->is_valid()) {
			echo 'Email jest prawidłowy.';
		}
		else {
			$form_valid->show_error();
		}
 
	}
 
 
?>
 
<form action="" method="post">
	email: <input type="text" name="email" />
	<input type="submit" value="sprawdz" name="valid" />
</form>
[PHP] pobierz, plaintext 

tylko nie wiem dlaczego jak wpiszę zły adres email to nie pokazuje błędu.. jak wpisuję poprawny to jest ok. co jest nie tak?

[PHP] pobierz, plaintext 
if ($form_valid->is_valid()) {
echo 'Email jest prawidłowy.';
}
else {
echo  $form_valid->show_error();   // <- brakowało tego echo   
} 
[PHP] pobierz, plaintext 

Dobrze byłoby, byś metody, które jedynie zwracają jakąś wartość (tzw. gettery) nazywał np. tak: getCosTam - w Twoim przypadku zamiast show_error() mogłoby być getError().

Ale to detal

zapomniałem o echo, mam kilka pytań

[PHP] pobierz, plaintext 
<?php
 
	class form_valid {
 
		public function validate($value, $require = false, $min_length = false, $max_length = false) {
			if ($require = true) {
				if (empty($value)) {
					$this->error = 'Pole wymagane';
				}
			}	
			if (strlen($value) < $min_length) {
				$this->error = 'Musisz podać minimum '.$min_length.' znaków.';
			}
			if (strlen($value) > $max_length) {
				$this->error = 'Maksymalna ilosć znaków to '.$max_length.'.';
			}
		}
 
		public function valid_email($value) {
			if (!preg_match('/^([^@s]+)@((?:[-a-z0-9]+.)+[a-z]{2,})$/i', $value)) {
				$this->error = 'Podany email jest nieprawidłowy.';
			}
		}
 
		public function get_error() {
			return $this->error;
		}
 
		public function is_valid() {
			return (count($this->error) > 0) ? FALSE : TRUE;
		}
 
	}
 
	if (isset($_POST['valid'])) {
		$email = $_POST['email'];
		$name = $_POST['name'];
		$form_valid = new form_valid;
		$form_valid->validate($email, false, 3, 5);
		$form_valid->validate($name, true);
 
		if ($form_valid->is_valid()) {
			echo 'Email jest prawidłowy.';
		}
		else {
			echo $form_valid->get_error();
		}
 
	}
 
 
?>
 
<form action="" method="post">
	email: <input type="text" name="email" /><br />
	imie: <input type="text" name="name" /><br />
	<input type="submit" value="sprawdz" name="valid" />
</form>
[PHP] pobierz, plaintext 

1. jak zrobić, żebym mógł sprawdzać email czy jest emailem i dodatkowo czy ma min. x znaków i max. x znaków? próbowałem tak:

[PHP] pobierz, plaintext 
$form_valid->validate($form_valid->valid_email($email), false, 3, 5);
[PHP] pobierz, plaintext 

ale za każdym razem wyrzuca mi Musisz podać minimum 3 znaków. tak jakby nie przekazywało $value do metody

2. dlaczego jak w nawiasach przy metodzie dam np. $min_length = false to skrypt tego nie pomija? robię np. tak:

[PHP] pobierz, plaintext 
$form_valid->validate($name, true);
[PHP] pobierz, plaintext 

wtedy jak nic nie wpiszę to jest ok, pokazuje błąd, ale jak wpiszę byle co to wyskakuje mi: Maksymalna ilosć znaków to .

3. mam formularz z np. 3 polami i każde chce inaczej walidować np. mam pole:

imię (pole wymagane, min. znaków, max. znaków),
nazwisko (pole wymagane),
email (poprawność emaila, czyli valid_email)


jak mam to zrobić?

1. $form_valid->valid_email($email) nie zwraca żadnej wartości więc jak chcesz walidować coś co nie istnieje?

2.

[PHP] pobierz, plaintext 
 
public function aaa($len = null) {
 
     if(!is_null($len)) {
 
          if($len > 10) {
...
[PHP] pobierz, plaintext 

3. Stwórz klasę bazową np. FormValid i dla każdego rodzaju elementu formularza utwórz klasy pochodne np. ValidInput, ValidCheckbox itd.

[PHP] pobierz, plaintext 
 
$form = new Form();
//if post 
$valid = new FormValid($form);
$valid -> addValidation(new ValidCheckbox('my_checkbox'), false, 1)
         -> addValidation(new ValidText('my_text'), false, 12, 150, '[a-zA-Z]');
 
if($valid -> isValid()) {
[PHP] pobierz, plaintext 

Dla każdego rodzaju elementu formularza mam robić osobną klasę? przecież to bez sensu jest. W większości kursach pisali, że 1 plik = 1 klasa, to żebym mógł walidować wszystkie rodzaje formularza musiałbym mieć z ~ 8 klas. O to chodzi?

Metoda valid_email zwraca błąd jeśli email jest nieprawidłowy, tzn. "przekazuje" ten błąd do metody get_error, która go wyświetla, więc chyba powinno zajarzyć..