[Life As Game] DTO of 메서드 남발에 대한 문제 인식

 

public final class AdminPlayerTitleResponse {

    private AdminPlayerTitleResponse() {
    }

    public record Granted(
            Long playerId,
            Long titleId,
            String code,
            String name,
            String category,
            Instant acquiredAt
    ) {
        public static Granted of(
                Long playerId,
                Long titleId,
                String code,
                String name,
                String category,
                Instant acquiredAt
        ) {
            return new Granted(
                    playerId,
                    titleId,
                    code,
                    name,
                    category,
                    acquiredAt
            );
        }
    }
}

문제 인식

• to, of, from 메서드를 쓰다보니 어느 순간 습관적으로 쓰고 있음을 인지하게 되었음
• 역할을 고려하지 않은 사용으로 불필요한 코드가 많아짐을 생각하게 되었음
• 처음 해당 메서드들을 사용했던 이유를 다시 생각해보게 됌

 

활용 이유

• 첫 활용 때는 of 메서드로 title을 null 체킹하여 default 값을 넣어주는 역할이 있었음
• from 메서드를 통해 필요한 속성만 변환해주는 역할을 했음
• 현재는 역할을 고려하지 않고 습관적으로 유사 컨벤션처럼 사용함

 

결론

• 무분별한 역할을 고려하지 않은 메서드의 활용을 중단하고, 확실한 목적을 갖을 때만 사용하도록 리팩터링함