코드 리팩토링 [1]

📕 Spring Framework/Spring Project

코드 리팩토링 [1]

GroovyArea 2022. 8. 3. 23:41

지난 1주일간 기존 프로젝트의 리팩터링을 위해 새로운 저장소를 생성했다.

아키텍처 부분 설계를 거의 2, 3일은 한 것 같다. 아직 수정이 필요하겠지만, 항상 고민해보자.

기존 프로젝트는 multi module로 진행했지만, 완벽한 멀티 모듈 프로젝트 이진 않았다.
모놀리틱으로 갈지, MSA를 고려해 모듈간 분리를 할지가 계속 고민된다..

확장성을 위해 모듈, 패키지간 의존성 분리를 틈틈이 고려해봐야겠다.

기존 작성 코드를 옮기며 조금 더 클린한 객체지향적인 코드로 리뷰어님의 피드백을 통해 수정해나갔다.

많이 알고 있었다고 생각한 부분에서도 헛점이 있었고, 전혀 알지 못했던 디테일한 부분도 알아나가는 중이다.

인증 및 인가 부분은 기존 프로젝트에서는 손수 구현했지만, 이번에는 Spring Security를 이용해 볼 생각이다.
한번 손수 경험한 것에 대한 보완점을 잘 만들어진 Framework를 사용함으로써 메꿔보고 싶다.

기존에 Persistence Layer를 Mybatis를 이용한 Sql Mapper를 이용해서 구현했다. 프로젝트 2개를 경험했었다.
이번엔 JPA를 이용해서 ORM으로 객체지향적 코드를 작성해보고 싶은 생각이 들었다.

리팩터링 한 부분과 새로 알게 된 점을 나열해보겠다.

Model

객체는 Request, Response에 따라 책임을 나누어 같은 모델이라도 따로 나누어 생성하는 것이 좋다.

> 이런 식으로 하나의 모델을 통합해서 사용하게 되면 변경사항이 생길 때 가변성을 띄게 될 수도 있다. 당연히 유지보수에 부적합하기 때문에 별개의 모델로 분리했다.

Utility

유틸리티로 사용할 클래스는 모든 도메인에서 이용이 가능해야 한다. 종속성을 띄면 안 된다.

public class CookieUtil {

    private static final String COOKIE_KEY = "Chicken";
    private static final String ENC_TYPE = "utf-8";

    private CookieUtil() {
    }

    /* 카트 쿠키 반환 메서드 */
    public static Optional<Cookie> getCartCookie(Cookie[] requestCookies) {
        return Arrays.stream(requestCookies)
                .filter(cookie -> cookie.getName().equals(COOKIE_KEY))
                .findFirst();
    }

    /* 카크 쿠키 값 디코딩 후 map 객체 반환 메서드 */
    public static Map<Integer, CartItemDTO> getCartItemDTOMap(Cookie responseCartCookie) throws UnsupportedEncodingException {
        String cookieValue = responseCartCookie.getValue();
        return JsonUtil.stringToMap(URLDecoder.decode(cookieValue, ENC_TYPE), Integer.class, CartItemDTO.class);
    }

    /**
     * 카트 안의 상품 번호 배열 반환 메서드
     *
     * @param responseCartCookie 응답 카트 쿠키
     * @return 상품 번호 배열
     * @throws UnsupportedEncodingException 인코딩 예외
     */
    public static Integer[] getItemNoArr(Cookie responseCartCookie) throws UnsupportedEncodingException {
        return Arrays.stream(getCartArr(responseCartCookie))
                .map(CartItemDTO::getProductNo)
                .toArray(Integer[]::new);
    }

    /**
     * 카트 안의 상품 이름 배열 반환 메서드
     *
     * @param responseCartCookie 응답 카트 쿠키
     * @return 상품 이름 배열
     * @throws UnsupportedEncodingException 인코딩 예외
     */
    public static String[] getItemNameArr(Cookie responseCartCookie) throws UnsupportedEncodingException {
        return Arrays.stream(getCartArr(responseCartCookie))
                .map(CartItemDTO::getProductName)
                .toArray(String[]::new);
    }

    /**
     * 카트 안의 상품 재고량 배열 반환 메서드
     *
     * @param responseCartCookie 응답 카트 쿠키
     * @return 상품 재고량 배열
     * @throws UnsupportedEncodingException 인코딩 예외
     */
    public static Integer[] getStockArr(Cookie responseCartCookie) throws UnsupportedEncodingException {
        return Arrays.stream(getCartArr(responseCartCookie))
                .map(CartItemDTO::getProductStock)
                .toArray(Integer[]::new);
    }

    /**
     * 카트 안의 상품 총 가격 반환 메서드
     *
     * @param responseCartCookie 응답 카트 쿠키
     * @return 상품 총 가격
     * @throws UnsupportedEncodingException 인코딩 예외
     */
    public static int getTotalAmount(Cookie responseCartCookie) throws UnsupportedEncodingException {
        return Arrays.stream(getCartArr(responseCartCookie))
                .map(CartItemDTO::getProductPrice)
                .mapToInt(Integer::intValue)
                .sum();
    }

    private static CartItemDTO[] getCartArr(Cookie responseCartCookie) throws UnsupportedEncodingException {
        Map<Integer, CartItemDTO> cartItemDTOMap = getCartItemDTOMap(responseCartCookie);
        Collection<CartItemDTO> values = cartItemDTOMap.values();
        return values.toArray(new CartItemDTO[0]);
    }
}

> 유틸리티 클래스이지만 CartItemDTO라는 모델에 종속적인 코드.

@UtilityClass
public class CartCookieUtil {

    private static final String ENC_TYPE = "utf-8";

    /* 카트 쿠키 반환 메서드 */
    public static Optional<Cookie> getCartCookie(Cookie[] requestCookies, String cookieKey) {
        return Arrays.stream(requestCookies)
                .filter(cookie -> cookie.getName().equals(cookieKey))
                .findFirst();
    }

    /* 카트 쿠키 값 디코딩 후 map 객체 반환 메서드 */
    public static <V> Map<Integer, V> changeCookieToMap(Cookie responseCartCookie, Class<V> valueType) throws UnsupportedEncodingException {
        String cookieValue = responseCartCookie.getValue();
        return JsonUtil.stringToMap(URLDecoder.decode(cookieValue, ENC_TYPE), Integer.class, valueType);
    }

    /**
     *
     * @param responseCartCookie 응답용 장바구니 쿠키
     * @param clazz 변환 타입 clazz
     * @param <E> 변환 타입
     * @return 쿠키 값 리스트
     * @throws UnsupportedEncodingException 인코딩 예외
     */
    public static <E> List<E> getCookieValueList(Cookie responseCartCookie, Class<E> clazz) throws UnsupportedEncodingException {
        Map<Integer, E> map = changeCookieToMap(responseCartCookie, clazz);
        return new ArrayList<>(map.values());
    }
}

> 제네릭을 사용해 종속성을 없앴지만, 아직 좀 손 봐야 될 것 같다. 메서드명이나 파라미터명 등등

> 추후 서비스나 컴포넌트 빈을 이용해 장바구니 상세 로직을 구현해 보는 게 좋을 것 같다.