결제 모니터 뷰 레이어 알림 중복 제거
목차
refactor: /Users/slecs/dev/source/pay-monitor/app/build/outputs/apk/debug/ alert/Toast 중복 제거 및 SLECS.Modal.alert로 통합 리팩토링.
리팩토링 목적
기능 변경 없이 코드 품질을 올리는 작업. 여러 파일에 걸친 작업이었고 주요 레이어는 MyBatis SQL 매퍼, JSP 뷰임.
| 파일 | 역할 |
|---|---|
*** |
MyBatis SQL 매퍼 / 설정 |
상품-공급.jsp |
뷰 템플릿 |
내부 파일 |
뷰 템플릿 |
kyc내부 파일 |
뷰 템플릿 |
내부 파일 |
뷰 템플릿 |
내부 파일 |
뷰 템플릿 |
무엇을 바꿨나
- 중복 코드 제거: 여러 곳에 흩어진 유사 로직을 한 곳으로 집약
- 메서드 분리: 너무 길어진 메서드를 책임별로 쪼갬
- 네이밍 개선: 의미가 불명확한 변수/메서드명을 명확하게 변경
- 불필요한 import, 주석, 죽은 코드 제거
주의한 점
리팩토링은 동작이 바뀌면 안 됨. 변경 전후로 동일한 시나리오를 테스트해서 회귀가 없는지 확인했음. 특히 금융 로직 주변은 아주 조심스럽게 건드림.
이후 코드 읽기가 한결 수월해졌고, 유사한 패턴의 기능을 추가할 때 참고할 수 있는 예시가 생겼음.
끝
작업 날짜: 2026-03-10 | 레포지토리: slecs | 변경 파일 수: 6개. 전체 개발 흐름에서 이 커밋은 해당 기능 도메인의 완성도를 높이거나 안정성을 확보하는 방향으로 기여했음. 연관 기능과 함께 엔드투엔드로 검증했고 회귀는 없었음.
댓글 0
첫 댓글 달아줘.