✍️ 코드 리뷰에서 흔히 보이는 세 가지 실수

많은 개발자에게 코드 리뷰는 매일 하는 일 중 하나인데요, 코드 리뷰를 잘 하려면 어떻게 해야 할까요? 시니어 엔지니어 Jordan Cutler는 아래 글에서 자신이 했던 실수 세 가지를 공유하며 코드 리뷰를 할 때 이것만은 피하라고 권합니다. 실수 1️⃣ 내가 옳다고 생각하는 방식만을 강요한다. 회사마다, 개발 팀마다 코드 스타일이 조금씩 다를 수 있다는 걸 간과하고 자신만의 스타일을 강요하는 것은 팀에 도움이 되지 않습니다. 자신이 기존 팀 코드를 모두 뜯어 고칠 게 아니라면 원래 스타일에 따라 가는 게 맞겠죠. 실수 2️⃣ 제안 대신 지시하는 말투를 사용한다. 코드 리뷰에서는 같은 내용이라도 말투의 차이로 인해 다른 결과를 초래하는데요, ‘지시’하는 말투를 사용하면 상대방이 방어적인 태도를 취하기 쉽지만, ‘제안’하는 말투를 사용하면 상대방이 의견을 좀 더 쉽게 수용하게 되고 의미 있는 토론으로 이어지기도 합니다. [예제] 지시: 이 코드는 너무 기니까 다른 파일로 나누세요. 제안: 이 코드가 길어지는 것 같은데 다른 파일로 쪼개는 건 어떻게 생각하세요? 실수 3️⃣ 모든 코멘트를 같은 중요도로 취급한다. 코드에 대한 코멘트를 달 때 어떤 부분은 꼭 변경해야 하는 사항이고, 어떤 부분은 작은 개선 사항인 경우가 많습니다. 모든 코멘트를 다 꼭 필요한 변경 사항인 것처럼 취급하기보다는 중요도를 구분해서 코멘트를 다는 것이 더 효과적인데요, 코멘트 맨 앞에 중요도의 정도를 적어주는 것만으로도 도움이 됩니다. 📔 함께 읽어보면 좋은 글 코드 리뷰 시 중점적으로 봐야 할 다섯 가지: https://careerly.co.kr/comments/87254