Question

我知道必须有一个更好的方法来做到这一点，但每当我搜索“＆amp;＆amp;”时我的结果不够好......

<?php
if (empty($row[ContactName]) && empty($row[ContactEmail]) && empty($row[ContactPhone]) && empty($row[Website])){
echo "Not Provided";
}
else{
...do stuff...
}
?>

谢谢！

Answer 1

原始代码有什么问题？

<?php
if (empty($row[ContactName]) 
    && empty($row[ContactEmail]) 
    && empty($row[ContactPhone]) 
    && empty($row[Website]))
{
    echo "Not Provided";
} 
else{
 ...do stuff...
}
?>

对我来说看起来很好......

Answer 2

<?php
$i=1;
$ar=array('ContactName','ContactEmail','ContactPhone','Website')

foreach($ar as $a)
  if (empty($row[$a]))
    {
     $i=0;
     break;                      //to make code fast
    }
  if($i)         //do stuff
  else echo 'not provided';
?>

或者如果你真的想让你的代码变得更小，那么在数据库中更改你的列名

From               To
ContactName        Col1
ContactEmail       Col2
ContactPhone       Col3
Website            Col4

然后再做

 <?php
  $i=1;
  for($a=1;$a<5;$a++)
    if (empty($row['Col'.$a]))
    {
      $i=0;
      break;
    }
  if($i)//do stuff
  else echo 'Not Provided';
    ?>

然而，重命名列并不好，因为它会使您的数据库不太容易理解。

Answer 3

如果紧凑性比可读性更重要，那么你可以做到这么短：）

$required = array('ContactName', 'ContactEmail', 'ContactPhone', 'Website');

$ra = array_filter(array_intersect_key($row,array_flip($required)));
if(empty($ra)){
    echo "Not Provided";
}
else{
    //....
}

你不能把array_filter放在empty（）里面，或者你得到“致命错误：在写上下文中不能使用函数返回值”

1）数组翻转将您需要的$值转换为键 2）array_intersect_key抛弃在$ required中找不到的任何$ row键 3）array_filter抛弃所有空值

留下零长度数组，该数组为空。无需担心数组长度或循环

Answer 4

由于php函数存在很多不一致性，因此使用库使其更加一致总是一个好主意。

将此功能放在该库中：

function empty()
{
    foreach(func_get_args() as $arg) if (empty($arg)) return false;
    return true;
}

用法：

if (empty($row[ContactName], $row[ContactEmail], $row[ContactPhone], $row[Website])) {
      echo "Not Provided";
}

Answer 5

<?php
$flag = 1;
if (empty($row[ContactName])){
$flag = 0;
}
if (empty($row[ContactEmail])){
$flag = 0;
}
if (empty($row[ContactPhone])){
$flag = 0;
}
if (empty($row[Website])){
$flag = 0;
}

if($flag == 0){
echo "Not Provided";

}else{
//do stuff
}    


?>

OR

$flag = 1;
$i=0;
$mustarray = array('ContactName','ContactName','ContactName','ContactName'); //or any number of fields that you want as mandatory
foreach($yourresult as $row){
   if(empty($row[$mustarray [$i]])){
    $flag = 0;
} 
$i++;
}
    if($flag == 0){
    echo "Not Provided";

    }else{
    //do stuff
    }

Answer 6

不要只是为了使代码看起来更好而制作数组和东西。正如汤米指出的那样

<?php
if (empty($row[ContactName]) 
    && empty($row[ContactEmail]) 
    && empty($row[ContactPhone]) 
    && empty($row[Website]))
{
    echo "Not Provided";
} 
else{
 ...do stuff...
}
?>

这段代码很好，只需要像他指出的那样正确格式化。制作数组然后启动循环会降低性能

<?php
    $i=1;
    for($a=1;$a<5;$a++)
    if (empty($row['Col'.$a]))
    {
    $i=0;
    break;
    }
    if($i)//do stuff
    ?>

这可能看起来很小，很好并且“专业”，但是有更多操作可以提供与简单if指令相同的结果。不是说它不能更快完成，我不知道php，只记得代码执行速度。

如何清理这个if / else语句？（重构）

6 个答案:

如何清理这个if / else语句？ （重构）

6 个答案:

如何清理这个if / else语句？（重构）