질문이 있습니다.

function doorHandler(e) {
    const targetElem = e.target;
    

    if ( !targetElem.classList.contains('stage') && currentItem ) {
        let tgid = parseInt(targetElem.parentNode.dataset.id);
        if ( tgid ) {
            inactivate(currentItem);
        }
    } 
    if ( targetElem.classList.contains('door-body') ) {
        activate(targetElem.parentNode);
    }
};

/*
     문이 닫히지 않는 조건을 2개 적용했습니다.
          1.  stage 클릭했을 때
          2. 열려있는 문을 클릭했을 때

     1번의 경우,  !targetElem.classList.contains('stage')
     2번의 경우,
          let tgid = parseInt(targetElem.parentNode.dataset.id);
        if ( tgid ) {
            inactivate(currentItem);
        }
      
      위 1,2번으로 조건을 적용하였는데 이렇게 적용하면 맞는 걸까요?
      기능상으로는 제가 원하는 대로 작동은 합니다만
      이 방법이 맞는 건지, 또는 좀 더 나은 방법이 있는 건지
      궁금해서 여쭤봅니다.
*/

function doorHandler(e) {
				const targetElem = e.target;

				if (targetElem.classList.contains('door-body')) {
					if (currentItem) {
						inactivate(currentItem);
					}
					activate(targetElem.parentNode);
				}
			}