JAVA(Spring) 메소드 리팩토링 객체 접근 문제

Question

안녕하세요.java의 Spring 프레임 워크를 사용하여 개발을 진행하고 있습니다. 코드를 리팩토링하는 과정에서 발생한 문제입니다. 실무의 상황을 간략화한 예시를 통해 설명드리겠습니다.Q. 질문드리고 싶은 건 makeFruit메소드를 호출할 때 fruits 객체를 넘기지 않고 동일한 결과를 얻어내고 싶습니다.<코드 예시>@Service public class FruitManagementService { public List getFruits () throws Exception { List fruits= new ArrayList<>(); while(임의의 조건) { makeFruit(fruits); } return fruits; } public void makeFruit(List fruits) { 'fruit 객체를 만들기 위한 복잡한 로직' fruits.add(fruit); } }코드 설명)getFruits 메소드는 fruits 리스트를 makeFruit이라는 메소드를 호출할 때 인수로 넘깁니다.그리고 makeFruit은 fruit 객체를 만든후 인수로 넘어온 리스트에 추가합니다.getFruits 메소드는 그렇게 만들어진 fruits 리스트를 리턴합니다.아마도 makeFruit 메소드에서 Fruit 객체를 만들어서 리턴을 해주고, getFruits에서 리턴된 결과를 받아서 리스트에 추가하면 되지 않느냐라고 물으실 수도 있겠지만, 현재 상황에서는 makeFruit 메소드에서 뭔가를 반환하는건 좀 어렵습니다.만약 fruits 리스트를 전역변수로 만들어서 관리한다면 어떨까 싶었지만, 저 클래스 자체가 @Service 즉, 싱글톤으로 관리되기 때문에 다수의 사용자들이 저 메소드를 호출할 때마다 객체 상태가 변경되어 문제가 발생할 것 같습니다.어떻게 해결하는게 좋을까요?

Clickin · Answer

리팩토링에서 private 메소드 리턴값을 변경하면 안되는데 인자는 제거하고싶다는 이상한 조건은 납득하기 어렵네요makeFruit말고 buildFruit같은 별도의 메소드를 만들어서 기존의 객체 생성 로직을 이관하고 List를 리턴하도록 만든 다음makeFruit에서는 새로 만든 메소드를 사용해서 리턴하면 문제없는 부분이 아닐까요?그러면 기존 코드 호환성도 모두 유지됩니다.이런 식으로 만든다는거죠public void makeFruit(List fruits) { //'fruit 객체를 만들기 위한 복잡한 로직' -> buildFruit로 이관한다면 Fruit fruit = buildFruit(); fruits.add(fruit); }

JAVA(Spring) 메소드 리팩토링 객체 접근 문제

1개의 답변

Q&A 목록