更有效的方式.php

Question

我正在使用此代码，但我知道这不是很有效。 还有另外一种方法吗？ 更高效 ？

  if ($val-> check($form) === true) {
        {$data['livre'] = $val-> validate_age($form);}

        if ($val->validate_age($form) === true) {
            {$data['livre'] = $val->insertData($db, $form, $id);}

            if ($val->insertData($db, $form, $id) === true) {
                {$data['livre'] = $val->insertLanguages($db, $form, $id);}

                if ($val->insertLanguages($db, $form, $id) === true) {
                    {$data['livre'] = $val->val($form);}

                    if ($val->val($form) === true) {
                        {$data['livre'] = $val->valexp($form);}

                        if ($val->valexp($form) === true ) {
                            {$data['livre'] = $val->insertWorker($db, $form, $id);}

                            if ($val->insertWorker($db, $form, $id) === true) {
                                {$data['livre'] = $val->univAndCourse($form);}
...

谢谢

Answer 1

你可以提前退出..我不知道你的代码在发生故障时到底发生了什么，但如果这是在一个函数中......你可以做而不是这样：

if(condition1) {

  if (condition2) {

     return true;

  }

}

return false;

你可以这样做：

if (!condition1) {
  return false;
}

if (!condition2) {
  return false;
}
return true;

所以你基本上先处理'else'案例..

除此之外..这也可能有效：

if (condition1 && condition2 && condition3 && condition4) {
    return true;
}

要么：

if (
     condition1 && 
     condition2 && 
     condition3 && 
     condition4
  ) {
    return true;
} else {
    return false;
}

Answer 2

在这个非常特殊的情况下，您可以使用and链接将其压缩为表达式：

$val-> check($form)
AND
    $data['livre'] = $val-> validate_age($form)
AND
    $data['livre'] = $val->insertData($db, $form, $id)
AND
    $data['livre'] = $val->insertLanguages($db, $form, $id)
AND
    $data['livre'] = $val->val($form)
AND
    $data['livre'] = $val->valexp($form)
AND        
    $data['livre'] = $val->insertWorker($db, $form, $id);

这似乎非常合适，因为你真的是双重配额， if检查。

这是因为and比= assigment运算符具有更低的优先级。 你的===true检查是多余的。 如果你想要你可以重新打包整个条件链，就像if ()谓词一样。

Answer 3

这就是例外情况：

try {
   $data['livre'] = $val->validate_age($form);
   $data['livre'] = $val->insertData($db, $form, $id);
   $data['livre'] = $val->insertLanguages($db, $form, $id);
   $data['livre'] = $val->val($form);
   $data['livre'] = $val->valexp($form);
   $data['livre'] = $val->insertWorker($db, $form, $id);
   $data['livre'] = $val->univAndCourse($form);
} catch (Exception $e) {
   // 
   // Do what ever necessary to process the interrupted logic.
   //
}

当然，这意味着验证器类的方法抛出exceptions而不是返回booleans ：

class Validator {

   function validate_age($form) {
      if (!is_numeric($form['age'])) throw new Exception('Invalid age.');
   }

   // 
   // .. etc ..
   //
}

Answer 4

看起来你的所有函数调用都返回一个bool。 这段代码应该有效。 如果任何调用返回false，则$ data ['livre']将为false。

$data['livre'] = $val->check($form) &&
                 $val->validate_age($form) &&
                 $val->insertData($db, $form, $id) && 
                 $val->insertLanguages($db, $form, $id) && 
                 $val->val($form) &&
                 $val->valexp($form) && 
                 $val->insertWorker($db, $form, $id) && 
                 $val->univAndCourse($form);

Answer 5

如果结果是否定的，将其包裹在函数中，将所有检查都转为负数并从函数返回。 此外，您可以在$ val方法中使用异常，这样您就可以在出现错误时中断执行，而无需检查每个操作。

Answer 6

您可以在一个外部条件中检查所有验证，在其中打开一个数据库事务。 将所有插入放在catch中的try和事务回滚中。

这样的事情：

if ($val-> check($form) === true && $val->validate_age($form) === true && $val->val($form) === true && $val->valexp($form) === true) {
    //begin your transaction here.  Depending on your framework it could be different.
    mysql_query('start transaction');
    try {
        $val->insertData($db, $form, $id);
        $val->insertLanguages($db, $form, $id);
        $val->insertWorker($db, $form, $id);

        //... blah blah blah  more inserts here

        //commit your transaction here
        mysql_query('commit');
    } catch (Exception $e) {
        //Roll back the transaction here
        mysql_query('rollback');
    }
}

如果它们失败，您只需要让插入抛出异常。