모범 사례 -

Question

다른 내가 다가 내 코드가 잘 있는지 알고 싶습니다, 내가하고 싶은에 : 그것을 만드는 존재하고 할당 할 경우 객체가

존재

1. 확인하는 경우 이 속성
2. 이미 단지 특성

을 할당 제가 지금 가지고있는 것은 다음과 코드,하지만 내가 쓰고 싶은 존재하지 않는 경우 두 번 같은 줄

에게

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); 
} 

Answer 1

이 작업을 수행하는 일반적인 방법은 중복 검사입니다 :

someobject = someobject || { 
    project:"some value", 
    status:[] 
}; 

Answer 2

단순화 된 코드

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); 
    } 

Answer 3

이 경우 은 객체이거나 정의되지 않은이지만 문자열, 숫자 등이 아니므로 진리 값이 있는지 확인할 수 있습니다. A (truthy) 값이 존재하지 않는 경우에만 변수에 값을 할당하므로

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); 
}