如何减少这里使用的“if”语句的数量？

Question

已回答

感谢您的帮助！

我正在为我的AP计算机科学类的一些代码工作，我想知道是否有更有效的方法来做到这一点。该方法非常长且冗余。解释旁边我可以做出的改进将不胜感激！

public void setColor(){ 

    if(sequenceNum == 0){ 
     myColor = Color.RED; 
    } 
    if(sequenceNum == 1){ 
     myColor = Color.ORANGE; 
    } 
    if(sequenceNum == 2){ 
     myColor = Color.YELLOW; 
    } 
    if(sequenceNum == 3){ 
     myColor = Color.GREEN; 
    } 
    if(sequenceNum == 4){ 
     myColor = Color.BLUE; 
    } 
    if(sequenceNum == 5){ 
     myColor = Color.MAGENTA; 
    } 

} 

Answer 1

Java的switch语句是你在找什么：

https://docs.oracle.com/javase/tutorial/java/nutsandbolts/switch.html

switch (sequenceNum) { 
    case 0: 
    myColor = Color.RED; 
    break; 
    case 1: 
    myColor = Color.ORANGE; 
    break; 
....SNIP... 
} 

Answer 2

与switch

switch (sequenceNum) { 

case 0: 
    // Do something 
    break; 

case 1: 
    // Do something else 
    break; 

default: 
    // Do a default something 
    break; 
} 

Answer 3

你可以做

Color[] colors = {Color.RED, Color.ORANGE, Color.YELLOW...}; 
myColor = colors[sequenceNum]; 

Answer 4

public void setColor(){ 
    switch(sequenceNum){ 
    case 0: 
     myColor = Color.RED; 
     break; 
    case 1: 
     myColor = Color.ORANGE; 
     break; 
    case 2: 
     myColor = Color.YELLOW; 
     break; 
    case 3: 
     myColor = Color.GREEN; 
     break; 
    case 4: 
     myColor = Color.BLUE; 
     break; 
    case 5: 
     myColor = Color.MAGENTA; 
     break; 
    } 
} 

但是，这也是丑陋由Reimeus提供的答案是方式更优雅的只是确保你检查0 < = sequenceNum < 5，避免受到arrayoutofboundsexception。我猜你正在学习，所以你应该检查维基有用的资源：https://stackoverflow.com/tags/java/info。和Javadoc来控制流的更多信息：https://docs.oracle.com/javase/tutorial/java/nutsandbolts/flow.html

好运

Answer 5

switch语句是一个想法，但如果序列号是密集的，我会用一个查找表：

final Color[] myColors = {Color.RED, Color.ORANGE, Color.YELLOW, ...}; 

myColor = myColors[sequenceNum]; 

当然，在经过范围检查sequenceNum之后。

请注意，您的原始代码在每个if之前应该有else，第一个除外。在比赛结束后重新测试没有意义。

Answer 6

public static Color[] MAP_SEQNUM_TO_COLOR = { 
    Color.RED, 
    Color.ORANGE, 
    Color.YELLOW, 
    Color.GREEN, 
    Color.BLUE, 
    Color.MAGENTA 
}; 

public void setColor(){ 
    if((0 <= sequenceNum) && (sequenceNum <= 5)) { 
    myColor = MAP_SEQNUM_TO_COLOR[sequenceNum]; 
    } 
} 

免责声明：没有测试，更好地把它当作伪代码。尽管如此，我在C＃领域比Java更积极，但这个想法应该很清楚。

Answer 7

如果通过“减少”如果“语句”的含义是字面意思，那么您可以使用开关，但这相当于代码运行时执行的类似测试序列。另一方面，如果你真的想要减少测试的数量，一个更清洁的解决方案就是使用一个数组（如其他人已经建议的那样），特别是因为你的示例代码暗示着每种颜色都与一个唯一的整数。