[Java]코드 리팩터링 과정에서 해결 방법에 대한 조언

Question

안녕하세요, 코드 리팩토링 과정에서 Object가 타입으로 포함되어 있는 것 자체가 위험한 코드라는 피드백을 받아 현재 해결 방법을 고민 중입니다. Generic을 생각해보라는 첨언을 들었고, Generic 타입을 공부하다 보니 의문점이 들어 문의드립니다.우선 제가 작성한 코드입니다.public Map getRestaurantInfoForUpdate(Long restaurantId) { log.info("getRestaurantInfoForUpdate(restaurantId={})", restaurantId); Map result = new HashMap<>(); Restaurant restaurant = restaurantDao.findById(restaurantId).orElse(null); result.put("restaurant", restaurant); List categories = getCategories(); result.put("categories", categories); List restaurantStatuses = List.of(RestaurantStatus.values()); result.put("restaurantStatuses", restaurantStatuses); return result; }우선, 해당 코드에서 반환한 Map은 바로 ResponseBody에 포함되어 클라이언트로 가고, 중간에 데이터 변형 등의 과정을 거치지 않습니다..다음의 해결 방법이 어떤지, 혹은 이런 경우에 어떻게 하실 것 같은지, 코드가 너무 별로다 등등의 모든 방향의 피드백 받습니다.Restaurant 관련 Data들을 한 타입을 확장하도록 합니다.public Map getRestaurantInfoForUpdate(Long restaurantId) { log.info("getRestaurantInfoForUpdate(restaurantId={})", restaurantId); Map result = new HashMap<>(); Restaurant restaurant = restaurantDao.findById(restaurantId).orElse(null); result.put("restaurant", restaurant); List categories = getCategories(); result.put("categories", categories); List restaurantStatuses = List.of(RestaurantStatus.values()); result.put("restaurantStatuses", restaurantStatuses); return result; } // domain 패키지 내 class RestaurantInfo {} // nothing // domain 패키지 내 class Restaurant extendes RestaurantInfo{ // ... ... Restaurant 정보 } class Category extendes RestaurantInfo{ // ... ... Category 정보 } class RestaurantStatus extendes RestaurantInfo{ // ... ... RestaurantStatus 정보 }public Map getRestaurantInfoForUpdate(Long restaurantId) { log.info("getRestaurantInfoForUpdate(restaurantId={})", restaurantId); Map result = new HashMap<>(); Restaurant restaurant = restaurantDao.findById(restaurantId).orElse(null); result.put("restaurant", restaurant); List categories = getCategories(); result.put("categories", categories); List restaurantStatuses = List.of(RestaurantStatus.values()); result.put("restaurantStatuses", restaurantStatuses); return result; }

Clickin · Accepted Answer

이런 경우에는 그냥 DTO하나 더 만드는게 제일 깔끔합니다.class RestaurantInfo { Restaurant restaurant; List categories; List restaurantStatuses; }

감독님테스트가하고싶어요 · Answer

Restaurant restaurant = restaurantDao.findById(restaurantId).orElse(null);Optional 사용중 지양해야할 코드로 보입니다.orElseThrow로 예외를 반환하거나, ifPresent(restaurnat->result.put("restaurnat",restaurnat))

뒷공간패스 · Answer

Dto하나 더 만든거에 쿼리한방에 다 받아올듯

wharup · Answer

그런데 곰곰히 보면..Category와 RestaurentStatus는 드롭 다운 메뉴 등에서 사용되는 코드성 정보 같습니다.그렇다면 ID로 조회하는 레스토랑과 용도도 다르고 조회하는 타이밍도 다를테니별도 API로 빼는게 좋아보입니다.

[Java]코드 리팩터링 과정에서 해결 방법에 대한 조언

4개의 답변

#generic 목록