Better way to validate the fields of an php object for empty values

Question

i have a php object i called $product, it has several string properties. Now i want to check the values of its fields to see if they are empty, but i am having to do this with many if statements like below, is there any smarter way?, i don't mind using a library

private function validate(Product $product)
{


    if (isEmpty($product->country)) {

        throw New \Exception("country is empty");
    } elseif (isEmpty($product->getCategory())) {

        throw New \Exception("category is empty");
     } elseif (isEmpty($product->getSubCategory())) {

        throw New \Exception("subcategory is empty");
     } elseif (isEmpty($product->getCoolingType())) {

        throw New \Exception("category is empty");
    } elseif (isEmpty($product->getPackagingType())) {

        throw New \Exception("category is empty");
    } 


}

btw i am using symfony framwork 3.0 if that helps all the fields i am trying to validate are strings

Bing · Accepted Answer · 2018-06-03 23:43:51Z

1

You could loop them with the function names / error to print if returned value is empty as the key/value array.

Would look something like this:

 <?php
 private function validate(Product $product)
 {
     $functions = array("getCategory" => "category",
                         "getSubCategory" => "subcategory"
                        // as many function_name => error messages here as you want
     );

     foreach($functions as $function_name => $error) {
         if(isEmpty($product->$function())) {
           throw New \Exception("$error is empty");
         }
     }
}

Note that your first example, $product->country doesn't fit this, so it would have to use a different case.

This answer explains what I'm doing with the if(isEmpty($product->$function())) { line of code.

edited Jun 3, 2018 at 23:43

answered Jun 3, 2018 at 23:37

Bing

3,1937 gold badges50 silver badges87 bronze badges

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

Collectives™ on Stack Overflow

Better way to validate the fields of an php object for empty values

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related