[Spring] @AllArgsConstructor, @RequiredArgsConstructor 사용을 지양한 이유

 

 

인턴 생활을 하면서 @AllArgsConstructor, @RequiredArgsConstructor 이 두 어노테이션을 본 적이 없었는데, 그냥 사용을 안하는거구나라고만 생각했었다. 그래서 이번 프로젝트에서도 사용을 지양하고 있었는데 문뜩 왜 지양할까라는 의문이 들었다!

 

일단 spring di부터 작성하자면!

 

Spring DI란?

DI는 스프링 핵심 기술이라고 생각하는 의존성 주입이다. 객체들간의 의존성을 줄이기 위해 사용되는 스프링 IoC 컨테이너의 구체적인 구현방식을 말한다. 의존성 주입하는 방법에는 생성자 주입, 필드 주입, setter 주입 세개가 존재하는데 그 중에서 스프링에서 가장 많이 사용하고 권장하는 방식은 생성자 주입이다.

 

@Service
public class UserService {

    private final UserRepository userRepository;

    @Autowired // 생성자가 하나만 있을 경우에는 생략 가능
    public UserService(UserRepository userRepository, MemberService memberService) {
        this.userRepository = userRepository;
    }

}

 

이런식으로 쓰면 된다.

 

하지만 이런 방식을 스프링에서는 @AllArgsConstructor, @RequiredArgsConstructor 이 어노테이션 하나만으로 생성자 주입을 일일이 해주지 않아도 된다.

 

DTO에서 @AllArgsConstructor 사용하게 되면 전체 파라미터를 가지는 생성자 선언을 할 수 있고, Controller와 Service에서는 @RequiredArgsConstructor이걸 사용해서 생성자 선언을 간결하게 하면서 DI가 될 수 있도록 한다.

 

근데 이 사용을 지양해야 하는 이유가 있다고 한다

@Getter
@AllArgsConstructor
public class Member {

    public String email;

    public String nickname;
 }

 

해당 객체를 생성하기 위해서는 아래와 같은 생성 방식을 사용하게 되는데

Member member = new Member("email", "nickname");

 

만약 변수 선언 순서를 아래와 같이 바꿨다고 쳤을 때 순서가 바뀌었어도 객체 생성에는 문제가 없게 되고 결국 잘못된 객체가 생성될 가능성이 있게 된다.

@Getter
@AllArgsConstructor
public class Member {

    public String nickname;

    public String email;
 }

 

 

따라서 나도 DTO에는 해당 어노테이션을 사용 안하고 @Getter와 @Builder만 사용하고 있다!

근데 생각해보면 Controller나 Service에서  @RequiredArgsConstructor를 사용해도 문제가 없는거 아닌가라는 생각을 해서 찾아봤다.

 

[인프런]@AllArgsConstructor, @RequiredArgsConstructor 사용금지를 권하는 글을 보았습니다.

해당 글을 보면 김영한 강사님은 @AllArgsConstructor 사용은 지양하지만 @RequiredArgsConstructor는 사용하고 있다고 한다! 

 

근데 나는 일단은? 다른 곳에서도 저 어노테이션을 사용하지 않으니까 일관성을 유지하려고 사용을 안하기도 하고, 코드를 더 꼼꼼히 체크하는 버릇을 들이기 위해서 사용하지 않기로 했다!!