팀원 코드리뷰#2

오늘은 내 코드와 팀원들의 코드를 보면서 중복처리와 관련된 이슈사항을 발견했다.

우리 프로젝트는 아래의 화면과 같이 각 화면에 따라 각기 다른 입력 값들을 사용자가 직접 입력할 수 있는

형태이다.

 

이처럼 각화면에 따라 입력 속성이 다르기 때문에 중복처리할 속성 또한 다를 수밖에 없었다.

하지만 중복처리 방식에 있어서는 어느정도 통일성이 필요하다고 생각했기 때문에 리뷰를 진행했다.

 

django views.py의 중복처리 부분 코드.

위 코드는 사업장 화면에서 중복처리를 구현한 코드이다. django의 ORM을 기반으로 하여

DB에 해당 속성에 같은 값이 존재하면 에러 flag와 에러 메시지를 context에 담아서 return 시켜버린다.

같은 방식으로 4개의 모든 입력 필드에 대하여 중복처리가 구현되어 있었는데

모든 입력 필드에 대하여 단하나의 중복도 허용하지 않게 구현되어 있었다.

코드 상 단하나의 중복 값이 존재하면 return 하여 사용자의 Create 요청이 실패하기 때문에 

어떠한 속성은 중복을 허용하고 어떤 건 허용 안할지 명확하게 설정할 필요성이 있었다.

 

 

따라서 먼저 내 담당 화면인 법인정보 부분을 기획자분들에게 컨펌 요청하여

위와 같이 법인 코드만 중복 불가능하게 설정하였고 팀원들에게도 각자 담당화면에 대하여

기획자분들과 협의하여 중복 허용 여부를 명확하게 설정해줄 것을 요청하였다.