스프링부트 팩토리 메서드 패턴 사용 질문

Question

팩토리 메서드 패턴을 사용해보고 싶어서 개인적으로 하던 프로젝트에 적용해 봤습니다.단순한 객체 생성이 아니고 db접근 로직이 있는 클래스를 만드는거라 여기 써도 되나 싶은 마음이 있었지만 그래도 해봤어요.public class MapController { private final MapService mapService; @GetMapping public ResponseEntity> getMapData(@RequestParam MapType mapType, @RequestParam Double longitude, @RequestParam Double latitude) { return ResponseEntity.ok(mapService.getMapData(mapType, longitude, latitude)); } }@Service @RequiredArgsConstructor public class MapService { private final MarkableFactory markableFactory; public List getMapData(MapType mapType, Double longitude, Double latitude) { Markable mapData = markableFactory.getMapData(mapType); return mapData.getMarkableData(longitude, latitude); } }@Component public class SimpleMarkableFactory implements MarkableFactory { private final LostPostRepository lostPostRepository; private final FindPostRepository findPostRepository; private final EntityManager entityManager; public SimpleMarkableFactory(LostPostRepository lostPostRepository, FindPostRepository findPostRepository, EntityManager entityManager) { this.lostPostRepository = lostPostRepository; this.findPostRepository = findPostRepository; this.entityManager = entityManager; } @Override public Markable getMapData(MapType mapType) { if (mapType.equals(MapType.FIND)) { return new FindMarkableData(entityManager, findPostRepository); } else if (mapType.equals(MapType.LOST)) { return new LostMarkableData(entityManager, lostPostRepository); } else { return new AllMarkableData(entityManager, findPostRepository, lostPostRepository); } } }이렇게 했는데 생각해보니 이러면 mapData를 가져올 때마다 FindMarkableData, LostMarkable 클래스를 새로 생성하는 거 같더라고요.FindMarkableData, LostMarkableData는 단순하지 않고 db에 접근해서 데이터를 조회, 변환하는 함수들을 가지고 있어서 여러 개의 메서드가 있습니다.@RequiredArgsConstructor public class FindMarkableData implements Markable { private final EntityManager entityManager; private final FindPostRepository findPostRepository; @Override public List getMarkableData(Double longitude, Double latitude) { String pointFormat = GeometryUtil.calculateAndFormatPoint(latitude, longitude); List findPostList = findFindPostByLocation(pointFormat); Location location = new Location(latitude, longitude); return convertToMapDtoList(findPostList, location); } ... }이런 클래스가 매번 새로 생성돼도 괜찮은 건지 모르겠어서 올려봅니다.

DB가디비딥 · Answer

지나가는 6~7년차 개발자입니다..코드개선 전체를 적기는 다소 무리가있기에추상 메소드 패턴이 좀 섞인 느낌이네요워낙 고수분들이 많아서 제글이 도움이 될 public interface MarkableFacto Markable createMarkable();객체 생성 역할좀 다른데에 위임하시는게 좋을거 같고..지금 추상 패턴이 좀더 쌔보여서 @Component public class MarkableFactoryProvider { private final FindMarkableFactory findMarkableFactory; private final LostMarkableFactory lostMarkableFactory; private final AllMarkableFactory allMarkableFactory; @Autowired public MarkableFactoryProvider(FindMarkableFactory findMarkableFactory, LostMarkableFactory lostMarkableFactory, AllMarkableFactory allMarkableFactory) { this.findMarkableFactory = findMarkableFactory; this.lostMarkableFactory = lostMarkableFactory; this.allMarkableFactory = allMarkableFactory; } public MarkableFactory getFactory(MapType mapType) { switch (mapType) { case FIND: return findMarkableFactory; case LOST: return lostMarkableFactory; default: return allMarkableFactory; } } }MapType에 따라서 적절한 팩토리 반환이 좀 필요해보입니다.. 타입에따른 안전성 체크도 필요해 보이구요..복잡도가 높아보이는 코드네요..

장독깨기 · Answer

팩토리 패턴이 추상화한 객체를 생성하는 디자인 패턴이잖아요.그니까, 매번 새로 생성하는 것이 맞습니다.근데, 글쓴이가 좀 이상한가 이런 생각이 드는 이유가요,한 번의 요청에 객체 한번 생성하고 그 객체 써먹고 끝이잖아요.그래서, 그렇습니다. 매번 객체를 생성하지 않아도 될 거 같잖아요.근데, 이 객체를 한 번의 요청에 여러 개 들고 있어야 한다 생각해보면요,객체를 매번 새로 생성하는 것이 당연하겠죠.실제 이럴 경우에 팩토리 패턴을 사용하면 되는 거구요.

Xellos Choi · Answer

디자인패턴 공부하는 중이시라는 걸 감안해서 팩토리 패턴에서 인스턴스 생성시 if문을 안쓰는 방법을 고민해보시면 좀 더 좋은 코드가 될 것 같아 추천드립니다

스프링부트 팩토리 메서드 패턴 사용 질문

3개의 답변

기술 목록