优化一大组 IF 语句

Question

我有一大组嵌套 IF 语句，我想知道是否有人对如何优化速度、大小和可读性有任何建议。

下面是一个 if 语句及其嵌套语句的示例。文件中大约有 25-30 个这样的内容。

if( $row["inlet_moisture"] > $row["inlet_moisture_high_warning"] ) {
    if( $row["inlet_moisture"] > $row["inlet_moisture_high_critical"] ) {
        if( $row["inlet_high_critical"] == 0 ) {
            if( $row["email_notification"] == 1 ) {

            }
            if( $row["mobile_notification"] == 1 ) {

            }
        }
    } else {
        if( $row["inlet_high_warning"] == 0 ) {
            if( $row["email_notification"] == 1 ) {

            }
            if( $row["mobile_notification"] == 1 ) {

            }
        }
    }
} else if( $row["inlet_moisture"] < $row["inlet_moisture_low_warning"] ) {
    if( $row["inlet_moisture"] < $row["inlet_moisture_low_critical"] ) {
        if( $row["inlet_low_critical"] == 0 ) {
            if( $row["email_notification"] == 1 ) {

            }
            if( $row["mobile_notification"] == 1 ) {

            }
        }
    } else {
        if( $row["inlet_low_warning"] == 0 ) {
            if( $row["email_notification"] == 1 ) {

            }
            if( $row["mobile_notification"] == 1 ) {

            }
        }
    }
}

这个想法是；我有一个读数（温度/速度/湿度），我需要检查它是否达到任何限制（高警告/高临界/低警告/低临界），如果是，我首先需要检查我是否已经为此发出警报。如果没有发送警报，我需要检查用户是否请求了警报通知（手机/电子邮件/两者），

目前这是有效的。我只是不喜欢它有多重？我可以在这方面改进吗？

谢谢。

Answer 1

在我看来，这似乎更清楚，尽管你可以组合嵌套的 if，但我更喜欢这个

if( $row["inlet_moisture"] > $row["inlet_moisture_high_critical"] ) {
  if( $row["inlet_high_critical"] == 0 ) {
   $message = 'the pertinent message';
  }
}
else if( $row["inlet_moisture"] > $row["inlet_moisture_high_warning"] ) {
  if( $row["inlet_high_warning"] == 0 ) {
   $message = 'the pertinent message';
  }
}
else if( $row["inlet_moisture"] < $row["inlet_moisture_low_critical"] ) {
  if( $row["inlet_low_critical"] == 0 ) {
   $message = 'the pertinent message';
  }
}
else if( $row["inlet_moisture"] < $row["inlet_moisture_low_warning"] ) {
  if( $row["inlet_low_warning"] == 0 ) {
   $message = 'the pertinent message';
  }
}


if( $row["email_notification"] == 1 ) {
  sendMessage($message, $email);
}
if( $row["mobile_notification"] == 1 ) {
  sendMessage($message, $mobile);    
}

Answer 2

过早优化是万恶之源 - 对于我们在这里处理的内容，无论您做什么这样做不会对性能产生太大/任何明显的影响。

话虽如此，大量 if 语句通常可以替换为一个或多个 switch 结构，尽管这是否会提高性能或提高可读性尚不清楚有争议的。您还可以为重复的代码创建一些函数，尽管这实际上可能会对性能产生负面影响。

根据您上面的评论...创建具有更好名称的变量对性能的影响几乎为零。如果会稍微增加您的内存使用量，但处理时间的影响将接近于零。而且，如果您将值评估为布尔值，则无需将它们显式转换为布尔值，因为 1 仍评估为 TRUE，0 仍评估为 FALSE。但是，如果您确实想要这样做

$email_notification = $row["email_notification"] == 1 ? true : false;

...不必要的冗长，您可以执行以下操作：

$email_notification = $row["email_notification"] == 1;

...或...

$email_notification = (bool) $row["email_notification"];

...并且它将具有相同的效果。