替换嵌套if语句

Question

这与beautiful code中的一章有关。 在那一章中，我读到了嵌套的ifs。

作者在谈论深度嵌套的ifs作为错误的起源和较少可读性。 他正在讨论用案例陈述和决策表替换嵌套ifs。

任何人都可以说明如何删除案例（选择大小写）和决策表的嵌套ifs吗？

Answer 1

好吧，不是直接回答你的问题，因为你具体询问switch/case语句，但是这里有一个类似的问题。

Invert “if” statement to reduce nesting

这谈论更换嵌套如果与后卫语句，即提前返回，而不是逐步解决在返回值之前检查的东西越来越多。

Answer 2

一个例子，我总是尝试做的是顶替重嵌套，如果是这样的（实际上这个不是太坏，但我看过他们深达8或9级在野外）：

if (i == 1) { 
    // action 1 
} else { 
    if (i == 2) { 
     // action 2 
    } else { 
     if (i == 3) { 
      // action 3 
     } else { 
      // action 4 
     } 
    } 
} 

与此：

switch (i) { 
    case 1: 
     // action 1 
     break; 
    case 2: 
     // action 2 
     break; 
    case 3: 
     // action 3 
     break; 
    default: 
     // action 4 
     break; 
} 

我也尽量保持动作尽可能小（函数调用是最适合这个），以保持压缩switch语句（这样你就不必往前走四页到看到它的结尾）。

我相信，决策表只是简单地设置标志，指​​出稍后需要采取什么行动。 “稍后”部分是基于这些标志的动作的简单排序。我可能是错的（它不会是第一次或最后一次:-)。

一个例子是（该标志设置阶段可以如果的是复杂的，因为它的动作是非常简单的）：

switch (i) { 
    case 1: 
     outmsg = "no paper"; 
     genmsg = true; 
     mailmsg = true; 
     phonemsg = false; 
     break; 
    case 2: 
     outmsg = "no ink"; 
     genmsg = true; 
     mailmsg = true; 
     phonemsg = false; 
     break; 
    default: 
     outmsg = "unknown problem"; 
     genmsg = true; 
     mailmsg = true; 
     phonemsg = true; 
     break; 
} 

if (genmsg) 
    // Send message to screen. 
if (mailmsg) 
    // Send message to operators email address. 
if (phonemsg) 
    // Hassle operators mobile phone. 

Answer 3

使病情到布尔值，然后写布尔表达式为每一种情况下。

如果代码是：

if (condition1) 
{ 
    do1 
} 
else 
{ 
    if (condition2) 
    { 
     do2 
    } 
    else (condition3) 
    { 
     do3; 

    } 
} 

一个可以写为：

bool cond1=condition1; 
bool cond2=condition2; 
bool cond3=condition3; 

if (cond1) {do1;} 
if (!cond1 and cond2) {do2;} 
if (!cond1 and cond3) {do2;} 

Answer 4

又如一些语言允许这是

  switch true{ 
      case i==0 
       //action 
      break 

      case j==2 
      //action 
      break 

      case i>j 
      //action 
      break 
      } 

Answer 5

如何链接IFS？

更换

if (condition1) 
{ 
    do1 
} 
else 
{ 
    if (condition2) 
    { 
     do2 
    } 
    else (condition3) 
    { 
     do3; 

    } 
} 

与

if (condition1) { 
    do1; 
} else if (condition2) { 
    do2; 
} else if (condition3) { 
    do3; 
} 

这很像复杂条件switch语句。

Answer 6

对于决策表，请参阅我对this question的回答，或者更好地阅读Code Complete 2中的第18章。

Answer 7

如果和switch语句不纯粹是OO。他们是有条件的程序逻辑，但做得很好！如果您想要删除这些语句以获得更多面向对象方法，请参阅combine the 'State' and 'Descriptor' patterns。

Answer 8

您也可以考虑使用Visitor pattern。

Answer 9

决策表是您在中将条件逻辑存储在数据结构中而不是代码本身中的地方。

所以不是这个（使用@大同的例子）：

if (i == 1) { 
    // action 1 
} else { 
    if (i == 2) { 
     // action 2 
    } else { 
     if (i == 3) { 
      // action 3 
     } else { 
      // action 4 
     } 
    } 
} 

你做这样的事情：

void action1() 
{ 
    // action 1 
} 

void action2() 
{ 
    // action 2 
} 

void action3() 
{ 
    // action 3 
} 

void action4() 
{ 
    // action 4 
} 

#define NUM_ACTIONS 4 

// Create array of function pointers for each allowed value of i 
void (*actions[NUM_ACTIONS])() = { NULL, action1, action2, action3 } 

// And now in the body of a function somewhere... 
if ((i < NUM_ACTIONS) && actions[i]) 
    actions[i](); 
else 
    action4(); 

如果i的可能性不低编号的整数，那么你可以创建一个查找表而不是直接访问actions数组的i th元素。

当您对几十个可能值做出决定时，该技术比嵌套的if s或switch语句更有用。

Answer 10

如果嵌套相当于上述逻辑运算AND

if (condition1) 
{ 
    if (function(2)) 
    { 
     if (condition3) 
     { 
      // do something 
     } 
    } 
} 

等效代码：

if (condition1 && function(2) && condition3) 
{ 
    // do something 
} 

在这两种情况下，当表达式计算假，随后的表达不会被评估。例如，如果condition1为false，则不会调用function（），并且不会评估condition3。

Answer 11

例如，一旦验证的一部分失败，您就可以中断。

function validate(){ 
    if(b=="" || b==null){ 
     alert("Please enter your city"); 
     return false; 
    } 

    if(a=="" || a==null){ 
     alert("Please enter your address"); 
     return false; 
    } 
    return true; 
}