3

我想知道我的代码接近是否良好,我想:

  1. 检查对象是否存在
  2. 如果不存在,则创建它并为其分配属性
  3. 如果已经存在,只需分配属性

我现在拥有的是以下代码,但我不喜欢在同一行写两次

function doSomething(_whatever){
    if(typeof someobject === "undefined"){
        someobject = { //dont exist
            profile : "some value",
            status : []
        }

        someobject.status.push(_whatever);
    }else{
        someobject.status.push(_whatever); //because already exist
    }
}

编写此代码段的更好方法是什么?或者做得更好,更少重复?

提前致谢

------ 原创功能

function addPerson(_person){
    var people = Iee.dashboard.analytics.data.people.data;      
    if(typeof people[_person.Id_Emp] === "undefined"){
        people[_person.Id_Emp] = {
            profile : _person,
            status : []
        }

        people[_person.Id_Emp].status.push({Id_Emp : _person.Id_Emp, status : _person.Estatus1, estatusby : _person.Centro_de_trabajo});
    }else{
        people[_person.Id_Emp].status.push({Id_Emp : _person.Id_Emp, status : _person.Estatus1, estatusby : _person.Centro_de_trabajo});
    }

    addBlackList(_person);
}
4

3 回答 3

4

执行此操作的常用方法是冗余检查:

someobject = someobject || {
    project:"some value",
    status:[]
};
于 2013-04-11T20:10:11.133 回答
4

简化代码

function addPerson(_person){
        var people = Iee.dashboard.analytics.data.people.data;      
        people[_person.Id_Emp] = people[_person.Id_Emp] || {
                profile : _person,
                status : []
            };

            people[_person.Id_Emp].status.push({Id_Emp : _person.Id_Emp, status : _person.Estatus1, estatusby : _person.Centro_de_trabajo});

        addBlackList(_person);
    }
于 2013-04-11T20:13:48.023 回答
1

在这种情况下,您希望它是一个对象或 undefined,但不是字符串、数字等,因此您可以检查它是否具有真值。

function addPerson(_person) {
    var people = Iee.dashboard.analytics.data.people.data,
        person = people[_person.Id_Emp];

    if (!person) person = {
        profile: _person,
        status: []
    };
    person.status.push({
        Id_Emp: _person.Id_Emp, 
        status: _person.Estatus1, 
        estatusby: _person.Centro_de_trabajo
    });

    addBlackList(_person);
}

这应该比其他答案中提到的冗余检查稍好一些,因为如果(真实)值不存在,它只会为变量分配一个值。

只是为了好玩,这是一个超浓缩版本:

function addPerson(_person) {
    var people = Iee.dashboard.analytics.data.people.data, id = _person.Id_Emp;

    (people[id] || { profile: _person, status: [] }).status.push({
        Id_Emp: id, status: _person.Estatus1, estatusby: _person.Centro_de_trabajo
    });
    addBlackList(_person);
}
于 2013-04-11T20:22:09.370 回答