상세 컨텐츠

본문 제목

[TIL] 2024.02.02 product-resume

[TIL]

by 재호링 2024. 2. 5. 11:25

본문

문제점 

  • 기존에 enum으로 값을 나눠놔서 상태 값을 입력 받을 시 알아서 예외처리 될 거라 생각했음
    근데 다른 값이 들어오면 서버가 멈추고 새로 시작해야되는 문제점 발견
    즉, 추가적인 검증을해야함. 

해결

    if(!['APPLY','DROP','PASS', 'INTERVIEW1', 'INTERVIEW2', 'FINAL_PASS'].includes(resumeStatus)){
      return res.status(400).json({message: '올바르지 않은 상태 값 입니다.'})
    }
  • .includes메소드를 활용해 특정 요소를 받아와 확인해줌

PS

  • 근데 굳이 .includes메소드를 사용하면 'enum'의 이점은 사라지는거 아닌가라는 생각을 하게 됨.
    사실 default를 APPLY로 주는 건 이력서가 생성될 때 명시해주면 되는 것이고,,

  • 찾아보니 코드의 가독성과 유지보수성을 생각했을 때 정의해두고 사용하는 방법이
    좋다고 하는데 내가 작성한 코드에서는 효율적인 방법은 아닌 것 같다.

관련글 더보기