24. AOP가 필요한 상황

AOP가 필요한 상황

• 모든 메서드의 호출 시간을 측정하고 싶을 때
• 공통 관심 사항(cross-cutting concern) vs 핵심 관심 사항(core concern)
• 회원 가입 시간, 회원 조회 시간을 측정하고 싶다면?

MemberService 회원 조회 시간 측정 추가

package hello.hellospring.service;

import hello.hellospring.domain.Member;
import hello.hellospring.repository.MemberRepository;
import hello.hellospring.repository.MemoryMemberRepository;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Service;
import org.springframework.transaction.annotation.Transactional;

import java.util.List;
import java.util.Optional;

@Transactional
public class MemberService {
    private final MemberRepository memberRepository;

    public MemberService(MemberRepository memberRepository) {
        this.memberRepository = memberRepository;
    }

    /**
     * 회원가입
     */
    public Long join(Member member) {

        long start = System.currentTimeMillis();

        try {
            validateDuplicateMember(member); // 중복 회원 검증
            memberRepository.save(member);
            return member.getId();
        } finally {
            long finish = System.currentTimeMillis();
            long timeMs = finish - start;
            System.out.println("join " + timeMs + "ms");
        }
    }

    private void validateDuplicateMember(Member member) {

        long start = System.currentTimeMillis();

        try {
            memberRepository.findByName(member.getName()) // null일 가능성이 있다면 Optional로 감싼다.
                    .ifPresent(m -> { // findByName의 결과가 Optional<Member>이기때문에 바로 .ifPresent 메서드를 실행할 수 있다.
                        throw new IllegalStateException("이미 존재하는 회원입니다.");
                    });
        } finally {
            long finish = System.currentTimeMillis();
            long timeMs = finish - start;
            System.out.println("join " + timeMs + "ms");
        }
    }

    /**
     * 전체 회원 조회
     */
    public List<Member> findMembers() {
        return memberRepository.findAll();
    }

    /**
     * 단일 회원 조회
     */
    public Optional<Member> findOne(Long memberId) {
        return memberRepository.findById(memberId);
    }

}

문제

• 회원 가입, 회원 조회에 시간을 측정하는 기능은 핵심 관심 사항이 아님
• 시간을 측정하는 로직은 공통 관심 사항임
• 시간을 측정하는 로직과 핵심 비즈니스 로직이 섞여서 유지보수가 어려움
• 시간을 측정하는 로직을 별도의 공통 로직으로 만들기 매우 어려움
• 시간을 측정하는 로직을 변경할 때 모든 로직을 찾아가면서 변경해야 함