[TIL-231121화] 첫 코드리뷰 ..

내가 맡은 파트 백엔드부분이 거의 완료가 되어, 오늘 점심먹고 최종 코드리뷰를 하자고 하셨다.

 

팀장님께서 간단히 코드를 봐주신 적은 있었지만 팀원 4명이 다 같이 리뷰어로 들어온 코드리뷰는 처음이었다.

 

깃랩에 올린 소스를 팀장님과 선배님 두 분이서 클론 받고 실행시켜 보셨다. 조용한 분위기여서.. 떨렸다... ㅜㅜ 

 

한시간 정도 리뷰를 해주셨는데, 

대부분 코딩할 때의 규칙들과 이렇게 하면 더 편하다 등의 노하우를 대부분 알려주셨다.

 

SELECT 쿼리를 API별로 각각 만드는 것이 아니라, 한번에 SELECT하고 WHERE절로 조건문을 달아서 재활용이 가능하도록 작성할 것, 

 

메소드명은 get/mod/reg/del 로 시작하도록 통일할 것, 

 

del_yn 컬럼은 정말 중요하니 INSERT 할 때 빼먹지 않고 모두 'N' 으로 넣어줄 것 , 

 

INSERT 시 필요한 일부 컬럼만 넣지 말고 , 모든 컬럼을 기본 값으로 세팅해서 넣어줄 것 ,

 

DB 툴 (디비버)에는 다른 팀원이 볼 수 있게 컬럼별로 코멘트를 달아둘 것, 

등등을 말씀해주셨다. 

 

모두 피가되고 살이되는 감사한 코멘트였다. ㅎㅎ

아래는 팀장님께서 피드백 해주신 내용을 정리한 것이다. 

팀장님께서는 처음인데 이정도면 잘했다고 해주셨고, 5년차 선배님께서는 너무 완벽하게 잘했다고 해주셨다. ㅠㅠ

 

리뷰를 해주시니까 더 의욕이 생기고 더 열심히 해야겠다는 생각이 들었다! 

최종 리뷰때는 빈틈이 단 하나도 없게 코딩을 해봐야겠다.