레거시 코드를 클린코드로, 누구나 쉽게 리팩토링 - 1

final static 필드를 모아 놓아서 뚱뚱해진 클래스 개선하기
AKENDEV's avatar
Mar 27, 2024
레거시 코드를 클린코드로, 누구나 쉽게 리팩토링 - 1

현상

  • 상수를 하나의 클래스에서 모아서 관리함

문제점

  • 상수의 통합 관리에 초점을 맞추려다 보니, 상수 클래스에 종속되어 의존성이 높아짐
  • 클래스에 상수가 많으면 이들 값을 구별하기 힘들고 유지보수가 어려워짐

개선방향

  • 의도된 상수를 사용하도록 상수 관리 클래스에 정의된 상수들을 목적에 맞게 분류하고 필요에 따라서는 상수를 해당 객체(책임 클래스)로 이동시켜야 함.
  • 시스템 상수와 일반 로직 상수를 구분하고 분리해야 함
  • 로직관련 상수는 되도록 로직과 가까운 곳에 있어야 함

개선과정

  1. 상수를 목적에 맞게 내부 클래스로 범위를 다시 지정함
      • 상수의 추가, 수정, 검색이 쉬움
  1. 내부 클래스를 목적에 맞는 클래스로 이동
Share article

akendev's Archive