PHP Object Validation

Question

I'm currently working on an OO PHP application. I have a class called validation which I would like to use to check all of the data submitted is valid, however I obviously need somewhere to define the rules for each property to be checked. At the moment, I'm using arrays during the construction of a new object. eg:

$this->name = array(
'maxlength' => 10,
'minlength' => 2,
'required' => true,
'value' => $namefromparameter
)

One array for each property.

I would then call a static method from the validation class which would carry out various checks depending on the values defined in each array.

Is there a more efficient way of doing this? Any advice appreciated. Thanks.

Answer 1

I know the associative array is used commonly to configure things in PHP (it's called magic container pattern and is considered bad practice, btw), but why don't you create multiple validator classes instead, each of which able to handle one rule? Something like this:

interface IValidator {
    public function validate($value);
}

$validators[] = new StringLengthValidator(2, 10);
$validators[] = new NotNollValidator();
$validators[] = new UsernameDoesNotExistValidator();

This has multiple advantages over the implementation using arrays:

• You can document them (very important), phpdoc cannot parse comments for array keys.
• Your code becomes typo-safe (array('reqiured' => true))
• It is fully OO and does not introduce new concepts
• It is more readable (although much more verbose)
• The implementation of each constraint can be found intuitively (it's not in a 400-line function, but in the proper class)

EDIT: Here is a link to an answer I gave to a different question, but that is mostly applicable to this one as well.