javascript - 避免嵌套条件中的重复

Question

对于类似于以下的嵌套条件，您如何优化分支以增强性能和/或可读性？由于许多潜在案例中存在循环，因此将重复条目合并为更实用的函数会更有效。

//a is defined either 1, 2, or 3
//b is defined either true or false

for(var i=0; i<hugeNumber; i++){
switch(a){
    case 1:
        if(b){
            for(objects in longlist){
                objects.color = object.c;
                objects.position = object.x
            }
        }else{
            for(objects in longlist){
                objects.color = object.c;
                objects.position = object.y
            }
    case 2:
        if(b){
            for(objects in longlist){
                objects.color = object.b;
                objects.position = object.x;
            }
        }else{
            for(objects in longlist){
                objects.color = object.b;
                objects.position = object.y;
            }
    case 3:
        if(b){
            for(objects in longlist){
                objects.color = blackColor;
                objects.position = object.x;
            }
        }else{
            for(objects in longlist){
                objects.color = blackColor;
                objects.position = object.y;
            }
}
}

将条件语句放在一个包罗万象的 for 循环中似乎同样不合理。

理想情况下，目标变量可以在开始时定义，当条件已知时 - 因为条件 a 始终为 0 生成颜色 c，为 1 生成颜色 b，为 2 生成 blackColor，而条件 b 始终为 true 生成位置 x位置 y 为 false。

我已经看到这个问题针对 PHP 和 Ruby 的变体，但不太确定如何将这些解决方案应用于 JavaScript。我能想到一些可行的方法，但到目前为止我还无法使代码在语法上起作用。

更新/解决方案:一个答案促使我发现这可以通过 eval() 有效地完成:

var targetColor;
var targetPosition;

switch(a){
    case 1: targetColor = "objects.c"; break;
    case 2: targetColor = "objects.b"; break;
    case 3: targetColor = "blackColor"; break;
}
if(b){
    targetPosition = "objects.x";
}else{
    targetPosition = "objects.y";
}

for(var i=0; i<hugeNumber; i++){
    for(objects in longlist){
        objects.color = eval(targetColor);
        objects.position = eval(targetPosition);
    }
}

如果有比这更好的方法，我绝对愿意接受其他建议 - 我知道 eval 有时可能很危险。

Answer 1

略微紧凑但可读性强，没有不必要的复查:

var coord = b ? 'x' : 'y';
var col = '';
switch(a){
    case 1: col = 'c';
    case 2: col = 'b';
    // may add more cases...
}
for(objects in longlist) {
    object.color = object[col] || 'blackColor';
    object.position = object[coord];
}