我刚刚在读 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
堆积在彼此之上...?
最佳答案
If/else
语句经常中断 Open-closed principle . (Java 示例但在 PHP 中也有效)
解决方案 => 支持多态性。
此外,多次分配一个临时变量确实很容易出错并且会降低可读性。特别是在 PHP 中,因为它不是一种静态类型的语言。
事实上,如果有人先分配 $actitivity = "Writing spaghettivity code";
然后分配 $actitivity = 1;
会怎样? ...在同一个容器中混合苹果和橙子。看看这个:http://sourcemaking.com/refactoring/split-temporary-variable
此外,仅当其中一个条件得到验证时,逻辑才允许产生副作用(print
)=> 方法不具有内聚性,因此违反了 SRP。
关于php - 试图避免 "spaghetti code",为什么多个 if-else 不好?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/18528647/