foreach op lege velden geeft onverwacht resultaat

Door thomas de vries op 19-05-2016 18:56

2.245 views

Goedenavond,

ik heb een register script.
4x een input veld

deze wil ik door middel van foreach laten controleren of ze leeg zijn.


<?php
if(isset($_POST['register']))
	{
		$fields = array('regist_username','regist_password','repeat_regist_password','email');
		if(!$Func->is_empty($fields) == FALSE) {
			echo 'OK';
		}
	}
?>

Function


<?php
public function is_empty($val) {
		//$fields = array('regist_username','regist_password','repeat_regist_password','email');
		foreach($val AS $fieldname ) {
			if(empty($_POST[$fieldname])) {
				return $this->Message('Niet alle velden zijn ingevuld!<br />',0);
			} else {
				return TRUE;
			}
		}
	}
?>

Echter als ik veld 1 niet leeg houd dan geeft die een OK dus met andere woorden hij controleert de overige velden niet.

Mis ik iets?

Met vriendelijke groet,

Frank Nietbelangrijk

19-05-2016 19:52 gewijzigd op 19-05-2016 19:55

zodra je een return gebruikt wordt de functie verlaten/afgebroken, ook als je in een (foreach) lus zit. Die else moet dus gewoon weg en die return TRUE; moet na de foreach lus komen.

<?php
public function is_empty($val) {
//$fields = array('regist_username','regist_password','repeat_regist_password','email');
foreach($val AS $fieldname ) {
if(empty($_POST[$fieldname])) {
return $this->Message('Niet alle velden zijn ingevuld!<br />',0);
}
}
return TRUE;
}
?>

Jan de Laet

19-05-2016 19:58

Moet regel 5 niet zijn: $Func->is_empty($fields) != FALSE

Verder doe je binnen is_empty ook al een return op regel 6 en 8. Moet je dat niet doen nadat alle velden zijn gecheckt? Dus na de foreach. En geeft de return op regel 6 een FALSE of TRUE als waarde?

Verder vind ik het verwarrend dat de functie is_empty TRUE als resultaat heeft als de variabele niet empty is.

Frank Nietbelangrijk

19-05-2016 20:00 gewijzigd op 19-05-2016 20:12

Wees voorzichtig met empty(). Als een gebruiker een '0' ingeeft in een veld dan zegt empty() nog steeds dat het veld leeg is!

Ik vermijd empty() om deze reden liever en gebruik dan strlen().

<?php
if(!strlen($_POST[$fieldname])) {
// niets ingevuld!
}
?>

Toevoeging op 19/05/2016 20:02:54:

Jan de Laet op 19/05/2016 19:58:03

Verder vind ik het verwarrend dat de functie is_empty TRUE als resultaat heeft als de variabele niet empty is.

Ben het met je eens Jan maar ik zou al helemaal geen functienaam is_emtpy gebruiken. Ik zou een functienaam gebruiken als validate of validateForm of iets dergelijks.

thomas de vries

20-05-2016 07:28

Heel erg bedankt voor jullie reacties.
Ik ga ermee bezig ook met de functienaam.

Zal idd iets moeten zijn als validateForm zodat er meerder forms op die manier gecontroleerd kunnen worden.
Nu heb ik is_empty en andere validaties dat kan wel in 1.

Reageren

Inloggen om te reageren