javascript - 通过重复 if/else 部分简化函数

Question

我正在尝试简化此功能，因为数据对象可以有多种类型，并且每种类型都有男性和女性版本。对象中元素的数量和名称始终相同。

如您所见，大部分代码都是重复的...

function calculate(type, j, value, s) {
    for (var i = j; i > 4; i--) { 
        if (type == 'weight') {
            if (s == 'f') { 
                if (weightFemale.hasOwnProperty(i)) {
                    var m = weightFemale[i][0],
                        l = weightFemale[i][1],
                        s = weightFemale[i][2];     
                    return getcalc( m,l,s );
                }
            }
            else {
                if (weightMale.hasOwnProperty(i)) {
                    var m = weightMale[i][0],
                        l = weightMale[i][1],
                        s = weightMale[i][2];       
                    return getcalc( m,l,s );
                }               
            }
        }
        else if (type == 'length') {
            if (s == 'f') { 
                if (lengthFemale.hasOwnProperty(i)) {
                    var m = lengthFemale[i][0],
                        l = lengthFemale[i][1],
                        s = lengthFemale[i][2],
                    return getcalc( m,l,s );
                }
            }
            else {
                if (lengthMale.hasOwnProperty(i)) {
                    var m = lengthMale[i][0],
                        l = lengthMale[i][1],
                        s = lengthMale[i][2],
                    return getcalc( m,l,s );
                }               
            }
        }
    }
    return false;
}

如何简化类型和性别的 if/else 部分？

Answer 1

由于您对每个对象执行相同的操作，因此只需让您的条件定义单个对象引用并仅调用计算一次。

类似于:

var obj;

if (type == 'weight') {
    obj = s == 'f' ? weightFemale : weightMale;
} else if (type == 'length') {
    obj = s == 'f' ? lengthFemale : lengthMale;
}

if (obj.hasOwnProperty(i)) {
    var m = obj[i][0],
        l = obj[i][1],
        s = obj[i][2];
    return getcalc(m, l, s);
}