리팩토링. 조회 및 변경 함수로 분리

getMemberAndSendMai() 이라는 메서드가 있고 이 안에서 for loop 로 멤버를 찾고, 특정 멤버에 대해서만 이메일을 보내고, 이메일 보낸 멤버들을 리턴하는 코드가 있다고 합시다. 이 메서드를 통해 특정 멤버 리스트를 알고 싶기만 한데도, 이걸 호출하면 무조건 이메일까지 호출해버리니 테스트하기가 어렵습니다. 이런 경우 아래처럼 2개로 분리하도록 합니다. getMember(); sendMail(); 만약 위 2개가 항상 함께 동작해야 한다면 편의를 위해 getMemberAndSendMai()도 만들어주면 됩니다. 총 3개의 메서드가 만들어지며, 테스트시에는 전체를 테스트하려면 getMemberAndSendMai()을 호출하면되고, 단순히 조회용도이면 getMember() 를 호출하면 됩니다. 이렇게 조회,변경을 분리하여 테스트가 쉽도록 하는 습관을 가지도록 해야 겠습니다....

원문링크 : 리팩토링. 조회 및 변경 함수로 분리