在以下代码段中,

var paintMode = data.hasOwnProperty('regions') ?
        'regions' : data.hasOwnProperty('vpcs') ?
        'vpcs' : data.hasOwnProperty('zones') ?
        'zones' : data.hasOwnProperty('subnets') ?
        'subnets' : data.hasOwnProperty('clusters') ? 'clusters' : null;


我使用了一个嵌套很深的(condition) ? true : false-这可以接受吗?优化了吗?如果不好,还有其他选择吗?

它在执行一些SVG操作的递归函数中使用,如果您好奇的话,这里是函数片段。

function paintGroups(data) {
    var paintMode = data.hasOwnProperty('regions') ? 'regions' : data.hasOwnProperty('vpcs') ? 'vpcs' : data.hasOwnProperty('zones') ? 'zones' : data.hasOwnProperty('subnets') ? 'subnets' : data.hasOwnProperty('clusters') ? 'clusters' : null,
        depth = data[paintMode].length,
        i,
        Shape; //= raphealObj.rect();

    // Register stacking order
    // Paint a shape with styles based on paintMode
    // Store its id & Label for other ops.

    if(paintMode) {
        for(i = 0; i < depth; i++) {
            paintGroups(data[paintMode][i]);
        }
    }

    // to reverse the order of paint - place your statements here
}

最佳答案

一般来说,由于分支预测,缓存,yada yada yada,使用二进制操作应该比带有分支的条件语句更快。但是您的代码很好。我喜欢&& ||但这只是一个偏好,并不基于任何经验。

var data = {a: 1, b: 2, c: 3};
var result = data.x ||
    data.y && "y" ||
    data.z && "z" ||
    data.w && "w" ||
    null;


我不想键入.hasOwnProperty。

编辑实际查看您的代码后

var paintMode = data.regions || data.vpcs || data.zones || data.subnets || data.clusters || null;
if(paintNode) {
    for(var i=0; i<paintMode.length; i++)
        paintGroups(paintMode[i]);
}

10-04 15:34