🧩 코드리뷰, 정말 꼭 해야 할까?

✍️ 아직 코드리뷰 문화가 없는 팀이라면?

스타트업, 사이드 프로젝트, 혹은 초기 단계의 조직에서는
코드리뷰를 하지 않아도 당장 눈에 띄는 문제는 없어 보입니다.

• “리팩터링은 나중에 하지 뭐”
• “어차피 혼자 짠 건데 뭐하러 리뷰해”
• “시간 없는데 코드 보느라 시간을 낭비할 순 없어”

하지만 코드는 '현재'만을 위한 것이 아닙니다.
2주 후, 2달 후, 또는 다른 사람이 봤을 때
잘 읽히고, 안전하고, 유지보수 가능한 코드여야 합니다.

코드리뷰는 바로 그 연결고리 역할을 합니다.

---

✅ 코드리뷰가 가져다주는 진짜 효과

1. ❗ 버그를 미리 발견할 수 있습니다

코드리뷰는 실시간 테스트 도구입니다.
의도하지 않은 side effect나 누락된 예외 처리를 빠르게 포착할 수 있습니다.

2. 📘 코드 스타일과 구조가 통일됩니다

비슷한 기능을 전혀 다른 방식으로 구현하면 기술 부채로 이어집니다.
리뷰를 통해 컨벤션을 지키고 통일성을 유지할 수 있습니다.

3. 🧠 비즈니스 로직 이해도와 팀 전파력이 올라갑니다

“이 기능 왜 이렇게 만들었지?”를 코드 안에서 유추하는 것이 아니라,
리뷰 과정을 통해 명시적이고 문맥적으로 전달할 수 있습니다.

4. 🙌 학습의 기회이자 성장의 루틴이 됩니다

리뷰는 단순 피드백이 아니라 지식 교류입니다.
주니어는 시니어의 관점을, 시니어는 주니어의 시도를 배울 수 있습니다.

은근히 많이 듣는 이야기:
“리뷰하다가 내가 더 배웠어요.”

5. 🚀 리뷰는 곧 제품 품질입니다

성공적인 코드리뷰 문화는 곧 서비스 안정성으로 이어집니다.

---

❌ 그렇다면, 코드리뷰의 어려움은?

• 리뷰하는 데 시간이 걸립니다
• 코드 문맥을 모르면 피드백이 어렵습니다
• 감정 섞인 표현은 의도치 않게 상처가 될 수 있습니다

하지만 이건 도구와 문화로 극복할 수 있습니다:

• 리뷰 기준을 명확히 하고
• PR 템플릿을 도입하고
• 피드백의 목적이 '사람'이 아니라 '코드'라는 점을 잊지 않는 것

---

🤝 리뷰를 ‘지적’이 아닌 ‘제안’으로 바꾸는 법

리뷰어의 말 한 마디가 조직의 분위기를 바꿉니다.

❌ “이건 잘못됐어요.”
✅ “혹시 이런 방식은 어떨까요?”

❌ “이거 왜 이렇게 짰나요?”
✅ “이렇게 구현하신 이유가 궁금합니다!”

❌ “이 함수는 너무 복잡해요.”
✅ “한 번 분리해보면 가독성이 더 좋아질 수 있을 것 같아요!”

---

😌 리뷰를 받는 입장에서 기분 나쁘지 않게 하려면?

• 🙏 “질문드릴게요” → 상대에 대한 존중 표현
• 🙇‍♂️ “제가 잘 몰라서 여쭤봅니다” → 대화의 시작을 부드럽게
• 🤝 “이런 방향도 고려해보면 좋을 것 같아요” → 피드백보다는 협업 제안

---

📚 코드리뷰를 '배움'으로 바꾸는 팀 문화 만들기

• 코멘트가 마음에 안 들어도 일단 ‘왜 그렇게 말했을까?’ 생각하기
• 리뷰어도 반대로 리뷰받아보기 → ‘양방향 문화’ 조성
• 리뷰 요청 시: “특히 어떤 부분이 궁금한지” 적어두기
• 리뷰 후: 칭찬도 같이 남기기! (예: 네이밍 좋네요, 리팩터링 방향 좋습니다 등)

---

🏷️ Pn 룰 가이드

**Pn 룰(Priority Notation)**은 리뷰 코멘트를 우선순위 기반으로 분류하여, 리뷰어와 개발자 간의 커뮤니케이션을 더 명확하게 해주는 방법입니다.

등급	의미	행동 지침
P1	꼭 반영해주세요	🔴 Request changes — 서비스에 중대한 영향을 주는 오류 가능성. 반드시 반영하거나 리뷰어를 설득해야 합니다.
P2	적극적으로 고려해주세요	🔴 Request changes — 필수는 아니지만 중요. 토론과 합의가 권장됩니다.
P3	웬만하면 반영해주세요	🟡 Comment — 수용 or 반영하지 못하는 이유를 설명하거나 추후 반영 계획을 명시하세요.
P4	반영해도 좋고 넘어가도 괜찮습니다	🟢 Approve — 무시해도 괜찮지만, 고민해볼 가치는 있습니다.
P5	그냥 사소한 의견입니다	🟢 Approve — 반영해도 좋고 무시해도 괜찮습니다.

💬 예시 코멘트

• P1: useEffect 안에 비동기 함수가 직접 들어가 있어서 에러가 발생할 수 있어요. 반드시 수정해주세요. (P1)
• P2: 이 API 호출은 debounce 처리하는 게 UX 측면에서 좋을 것 같아요. (P2)
• P3: 변수명을 조금 더 명확하게 해도 좋을 것 같아요. 예: result → userScore (P3)
• P4: 코드 스타일은 팀 내 규칙에 따라 정리하셔도 괜찮습니다. (P4)
• P5: 이 부분 개인적으로 너무 좋네요. 👍 (P5)

---

✍️ 코드리뷰를 요청하는 사람은 어떻게 해야 할까?

✅ 커밋을 논리적으로 잘게 나누기

• 기능 단위로 나누기: feat: 로그인 폼 레이아웃 추가, feat: 로그인 요청 API 연동
• 수정 단위로 나누기: fix: 로그인 상태관리 오류 수정

✅ 커밋 메시지를 명확하게 작성하기

• ❌ 불명확한 메시지: fix, update
• ✅ 명확한 메시지: fix: 로그인 API 요청 시 중복 호출 방지

✅ PR 설명을 꼼꼼하게 작성하기

## 작업 개요
- 로그인 기능 중 상태 관리 구조를 개선했습니다.

## 주요 변경 사항
- Zustand → Context API 마이그레이션
- API 요청 로직을 커스텀 훅으로 분리

## 리뷰 받고 싶은 부분
- 상태 구조가 적절한지
- 요청 훅 분리 방식에 개선 여지가 있는지

---

🧪 링크 드라퍼, 지금 베타 테스트 중입니다

링크 드라퍼는 링크를 저장하고,
정리하고,
다시 꺼내보게 만들어주는 서비스입니다.

• 📎 폴더별 링크 정리
• ✨ OG 이미지 자동 추출
• 🧠 공유 URL 생성 및 게시 기능
• ⚡ 크롬 익스텐션 원클릭 저장

👉 링크 드라퍼 베타 체험하러 가기
👉 크롬 웹스토어에서 설치하기

---

💬 카카오톡 채널 추가하고 소식 받기

• 서비스 업데이트
• 기능 꿀팁

카카오톡 채널을 통해 빠르게 받아보세요!

👉 카카오톡 채널 추가하기