Foreach Statement acting weird?

Question

I am creating a validation class in PHP. I want to validate two items in one array, and for some reason it only validates the first item. I call it like this:

Form::validate(array('user' => 'required', 'pass' => 'required'), 'login');

and the function is

public static function validate($rules, $form) { foreach ($rules as $rule => $val) { if ($val === 'required') { if (empty($_POST[$rule])) { if (isset($_POST[$form])) { self::$_error = Error::set('All fields are required. ' . $rule); echo self::$_error; return false; } } else { return true; } } } }

My question is how I could validate both items in the one array?

Return will do just that, return, exiting the loop

Steve
– Steve

2014-04-15 15:52:22 +00:00
Commented Apr 15, 2014 at 15:52 — Steve
– Steve, Commented Apr 15, 2014 at 15:52

Joni · Accepted Answer · 2014-04-15 15:54:10Z

3

Move return true to the end so that it will run only if everything was valid.

foreach (...) { if (...) { return false; } } return true;

answered Apr 15, 2014 at 15:54

Joni

112k14 gold badges151 silver badges201 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

user1475632 Over a year ago

Thank you that worked. Will accept your answer in 10 minutes.

Collectives™ on Stack Overflow

Foreach Statement acting weird?

1 Answer 1

1 Comment

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Related