목록 보기
코드 악취를 맡는 후각 훈련의 시간
아키텍처

코드 악취를 맡는 후각 훈련의 시간

마켓컬리
마켓컬리
2020년 6월 8일

두줄요약

한 달간 기준 수립·TODO 리뷰·테스트 검증을 기반으로 단계적 리팩토링을 진행했습니다. 예외 처리와 공통 로직을 개선하고, 코드 품질·테스트 커버리지·팀 이해도를 높였습니다.

문제 상황

  • 기능·성능 이슈는 없지만 누적된 복잡도와 낮은 가독성으로 유지보수 부담 증가
  • 제한된 한 달 동안 3명의 개발자가 리팩토링 범위와 기준을 정해야 하는 상황

해결 방법

  • 메인 기능과 코드 유사성 기준으로 8개 파트 분류, 3차 진행과 단계별 QA·배포 계획 수립
  • 30라인 초과, 깊은 조건문, 과다 파라미터, 불필요 코드·로그, 모호한 이름 등의 점검 기준 합의
  • 대상 코드와 변경 방향을 TODO로 기록하고 팀 리뷰로 리팩토링 방향 정렬
  • 서비스 레벨 mock 테스트 작성 후 컴파일·로컬·통합 테스트로 기존 기능 검증

성능/운영 포인트

  • 오류 메시지 enum 기반 ExceptionType 도입으로 문자열 상수와 예외 검증 방식 정리
  • 조건문이 누적된 공통 로직을 검수·검품 처리별로 재분리해 가독성 개선
  • 코드 라인 감소와 서비스 레벨 테스트 커버리지 증가

적용해볼 점

  • 담당 외 기능의 리팩토링 배정으로 프로젝트 이해도 상향 평준화
  • 반복적인 테스트 코드 작성과 리뷰를 통한 테스트 거부감 완화 및 코드 품질 관점 확장

다음 읽기

#refactoring 주제를 이어서 읽기

코드 악취를 맡는 후각 훈련의 시간

RMS를 기능·유사도 기준으로 나누고 TODO, 리뷰, mock 테스트 중심의 단계적 리팩토링을 진행했습니다. 예외 처리와 공통 로직을 개선하고 코드 가독성·테스트 커버리지·팀 이해도를 높였습니다.

마켓컬리
마켓컬리
아키텍처

댓글 0개

댓글을 작성하려면 로그인이 필요합니다.

댓글을 불러오는 중...