更有效的方式.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

这就是 异常 的用途：

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.
   //
}

当然，这意味着验证器类的方法抛出异常而不是返回布尔值：

class Validator {

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

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

Answer 2

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

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 3

在这种非常具体的情况下，您可以使用 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 的优先级低于 = 赋值运算符。您的 ===true 检查当然是多余的。如果您愿意，您可以将整个条件链重新打包为 if () 谓词。

Answer 4

看起来所有函数调用都返回一个布尔值。这段代码应该可以工作。如果任何调用返回 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

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

像这样的事情：

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');
    }
}

你只需要让你的插入在失败时抛出异常。