[멘토링] 코드 리뷰에 대해서 질문드리고 싶습니다. (답변희망멘토 : gray.great)

[Y0unse0]

🙋질문자

판교 3기 / 3조(깔3) - sei.jang(장윤서)/BE

🙋답변 희망 멘토

gray.great(박준형)/ @kakao-gray-great

✏️ 궁금한 내용

프로젝트를 진행하면서 코드리뷰를 하게 되는데 현업에서는 코드리뷰를 보통 어떤기준으로 진행하는지 궁금합니다.

[추가 이해를 위한 디테일한 질문]

• 신입이나 주니어 코드에서 가장 코멘트가 많이 달리는 부분은 어디인가요?
• 리뷰를 하시면서 좋은 코드라고 느끼셨던 경험이 있다면 어떤 점이 좋으셨나요?

👀 시도한 방식

⏭️ 이제 어떻게 하려고 하는지?

현업에서 중요하게 보는 코드 리뷰 기준을 참고해 코드 리뷰를 진행해보고 유의미한 경험으로 가져가고 싶습니다.

[graygreat]

현업 코드 리뷰의 핵심 기준은 가독성, 로직의 정확성, 유지보수성인 것 같아요.
주니어분들의 코드에서 주로 나오는 코멘트는 아래와 같아요.

• 네이밍: 변수, 메소드 이름만으로 의도 파악하기 어려울 때
• 예외 처리: 엣지 케이스 놓쳤을 때
• 관심사 분리: 하나의 함수가 많은 일을 하고 있거나 중복된 로직 있을 때
• 로직 정확성: 도메인 지식의 부족으로 인한 로직의 오류

제가 생각하는 좋은 코드는 의도가 명확해서 읽기 쉬운 코드입니다.
화려한 로직보다는 주석 없이 어떤 역할을 하는지 바로 이해되는 코드를 볼 때 마음이 편하더라고요.

취준생 분들이 많이 오해하는 것 중 하나가 코드 리뷰를 '지적' 처럼 하려고 하는 경향이 있는데, 더 나은 코드를 만들기 위한 협업으로 받아들여 보세요.
코멘트를 주고 받으면서 많은 성장을 할 수 있을 것 같아요.