php - 试图避免 "spaghetti code"，为什么多个 if-else 不好？

Question

我刚刚在读 top 100 signs of spaghetti code我遇到了数字 4，它简单地说明:

if ($status == "awake"){ 
    $actitivity = "Writing spaghetti code"; 
} else if ($healthstatus == "OK"){ 
    $activity = "Sleep"; 
} else { 
    print "CALL 911 IMMEDIATELY!"; 
}

我在 other spaghetti discussions 中看到过这种多重 if-else 模式.我有点困惑为什么会这样，即使它适用于这个例子。

上面的例子不好是因为

• 第一个变量是activity，它表示编码器需要一些 sleep ，所以这是个玩笑，或者

• View 不应在逻辑期间输出，或者

• 关于太多 if/else 的事情

---

EDIT 不要在意第二部分，它很糟糕，因为嵌套条件和多次返回

在other spaghetti discussions链接，不好是因为

- 逻辑中有 return，这会中断流程，或者

- 有太多的 if/else 堆积在彼此之上...？

Answer 1

If/else 语句经常中断 Open-closed principle . (Java 示例但在 PHP 中也有效)

解决方案 => 支持多态性。

此外，多次分配一个临时变量确实很容易出错并且会降低可读性。特别是在 PHP 中，因为它不是一种静态类型的语言。事实上，如果有人先分配 $actitivity = "Writing spaghettivity code"; 然后分配 $actitivity = 1; 会怎样？ ...在同一个容器中混合苹果和橙子。看看这个:http://sourcemaking.com/refactoring/split-temporary-variable

此外，仅当其中一个条件得到验证时，逻辑才允许产生副作用(print)=> 方法不具有内聚性，因此违反了 SRP。