Question

有没有更好的方法来编写这个JavaScript？

switch (l) {
    //A
    case '1-1-1':
    case '1-1-2':
    case '1-2-1':
    case '2-1-1':
    case '3-1-1':
        obj.result = 'A';
        break;
    //B
    case '1-2-2':
    case '1-2-3':
    case '2-2-2':
    case '2-2-3':
    case '3-2-2':
    case '3-3-1':
        obj.result = 'B';
        break;
    //C
    case '1-3-2':
    case '1-3-3':
    case '2-3-2':
    case '3-2-3':
        obj.result = 'C';
        break;
    //D
    case '3-3-2':
    case '3-3-3':
        obj.result = 'D';
        break;
    default:
        obj.result = 'AA';
        break;
}

Answer 1

如果你告诉我们你是如何得到这些价值的话，我们或许可以提出一个更简洁的解决方案，但这就是你如何使用查找表（生成的向后）：

var byResult={
    A: ['1-1-1', '1-1-2', /* ... */],
    B: ['1-2-2', '1-2-3', /* ... */],
    /* ... */
};
var byInput={};
for(var output in byResult) {
    if(!Object.prototype.hasOwnProperty.call(byResult, output)) {
        continue;
    }
    var inputs=byResult[output];
    for(var i=0, l=inputs.length; i<l; i++) {
        var input=inputs[i];
        byInput[input]=output;
    }
}
function lookup(value) {
    if(Object.prototype.hasOwnProperty.call(byInput, value)) {
        return byInput[value];
    }else{
        return 'AA';
    }
}

Answer 2

Thilo在评论中提到的查找表：

var lookup =
{
    '1-1-1': 'A',
    '1-1-2': 'A',
    '1-2-1': 'A',
    '2-1-1': 'A',
    '3-1-1': 'A',

    '1-2-2': 'B',
    '1-2-3': 'B',
    '2-2-2': 'B',
    '2-2-3': 'B',
    '3-2-2': 'B',
    '3-3-1': 'B',

    '1-3-2': 'C',
    '1-3-3': 'C',
    '2-3-2': 'C',
    '3-2-3': 'C',
    '3-3-2': 'C',

    '3-3-3': 'D'
};

及其用法：

obj.result = lookup[l] || 'AA';

我不能说这比switch版本更好。

Answer 3

将数字封装到函数中的字母处理中，这样您的主代码就不需要知道翻译是如何完成的。这样称呼：

obj.result = getThing(l);

// or, given your comment that l is formed by concatenating three
// values you could do the concatenation in the function
obj.result = getThing(v1, v2, v3);

然后在getThing()范围内，您可以使用现有的switch语句，或Matt Ball的答案中的查找表，或者您喜欢的任何其他方法。并且您可以随时更改方法，而不会影响调用函数的代码。

function getThing(v1, v2, v3) {
   var l = v1 + "-" + v2 + "-" + v3;
   // use lookup, switch, whatever
   return "somecode";
}

注意：实际上不要将您的功能称为“getThing”;将“Thing”替换为描述这些字母代码的真实内容。

有没有更好的方法来编写这个JavaScript？

3 个答案: