코드리뷰에서 동료에게 기분 안 상하게 피드백 주는 법
버그나 설계 문제를 지적하면서도 관계를 유지하는 코드리뷰 코멘트.
건설적 코드리뷰 코멘트
비난이 아닌 제안으로
이 로직 접근 방식 좋네요! 👍 한 가지 제안인데, 여기서 early return 패턴을 쓰면 중첩 if문을 줄일 수 있을 것 같아요. 예:
if (!user) return null;
가독성도 올라가고 유지보수도 편해질 것 같습니다. 어떻게 생각하세요?
💡 긍정적인 코멘트를 먼저 하고 개선 제안을 하면 수용도가 높아집니다.
Nit: 이 변수명이 data인데, 좀 더 구체적으로 userProfileData 같이 바꾸면 나중에 다른 개발자가 봤을 때 바로 이해할 수 있을 것 같아요. 사소한 건데 참고만 해주세요! 😊
💡 'Nit'이라고 표시하면 중요한 이슈가 아닌 스타일 제안임을 명확히 합니다.
💡 사용 팁
- 코드리뷰는 코드에 대한 피드백이지, 사람에 대한 평가가 아닙니다.
- '왜 이렇게 했어?'보다 '이렇게 하면 어떨까요?'가 건설적입니다.
⚠️ 주의사항
- '이거 왜 이렇게 짰어요?'는 공격으로 느껴집니다. 의도를 물어보세요.