在`if`条件中检查嵌套对象的更好方法是什么?

时间:2013-02-12 05:45:25

标签: javascript coding-style conditional-statements

考虑我有这样一个对象:

Obj.Boo.Foo.Zoo

好的,如果我想确保zoo有一个值而不是空,我应该这样做:

if(Obj != null && Obj.Boo != null && Obj.Boo.Foo != null && Obj.Boo.Foo.Zoo != null) {
    //blah blah
}

如果我在Zoo之后有更多嵌套对象,我应该扩展if并且代码变得有点格式错误。

有没有更好的方法来做这种情况?

3 个答案:

答案 0 :(得分:3)

我保留了一个小功能,我打电话给chainGet(obj, chain, default),就像这样的情况:

function chainGet(obj, chain, dflt) {
    if (typeof chain === "string")
        chain = chain.split(".");
    var result = obj;
    for (var i = 0; i < chain.length; i += 1) {
        if (result === undefined)
            break;
        result = result[chain[i]];
    }
    return result === undefined? dflt : result;
}

例如:

> chainGet({ foo: 42 }, "foo")
42
> chainGet({ foo: 42 }, "bar", "x")
'x'
> chainGet({ foo: { bar: 42 }}), "foo.bar")
42

这甚至适用于数组:

> chainGet([{ foo: [42] }], [0, "foo", 0])
42

当然,虚线访问的长链是a code smell,因此在使用它们时应该谨慎行事......但有时你只需要这样做;)

答案 1 :(得分:0)

如果你真的不需要处理这种情况,一个不好的做法是尝试在try catch语句中访问Object

try{
    // access Obj.Boo.Foo.Zoo
}
catch{
    // catch a NullReferenceException
}

更好的方法是修改getter和setter,以便在尝试访问子对象及其null时,父对象抛出一个合适的异常或处理它。在这种情况下,检查第一个对象就足够了。

答案 2 :(得分:0)

可以在不使用功能的情况下实现。 快捷方法可以是 -

if(((((Obj || {}).Boo || {}).Foo || {}).Zoo || {}) != {}) {
    // Zoo has a value and not empty
}

或ES6方式 -

const checkNull = (keys, Object) =>
  keys.reduce((Obj, o) =>
    (Obj && Obj[o]) ? Obj[o] : null, Object)

console.log(checkNull(['Boo', 'Foo', Zoo], Obj))
// [Value of Zoo]
console.log(checkNull(['Boo', 'Foo1', Zoo], Obj))
// null