컨트롤러 테스트시 size 없이 전달할때 npe

롬복 @Builder.Default를 넣으면 컴파일시에 기본 값 page=1, size=10이 제거 되네요. ㅠㅠ;

- 소스

@Builder
public class PostSearch {

    private static final int MAX_SIZE = 2000;

    @Builder.Default
    private Integer page = 1;

    @Builder.Default
    private Integer size = 10;

    public long getOffset() {
        return (long) (max(1, page) - 1) * min(size, MAX_SIZE);
    }

    public PostSearch() {
        System.out.println("good");
    }

    public PostSearch(Integer page, Integer size) {
        this.page = page;
        this.size = size;
    }

    public Integer getPage() {
        return page;
    }

    public Integer getSize() {
        return size;
    }

    public void setPage(Integer page) {
        this.page = page;
    }

    public void setSize(Integer size) {
        this.size = size;
    }
}

- @Builder.Default 제거 후

public class PostSearch {
    private static final int MAX_SIZE = 2000;
    private Integer page = 1;
    private Integer size = 10;

    public long getOffset() {
        return (long)(Math.max(1, this.page) - 1) * (long)Math.min(this.size, 2000);
    }

    public PostSearch() {
        System.out.println("good");
    }

    public PostSearch(Integer page, Integer size) {
        this.page = page;
        this.size = size;
    }

    public Integer getPage() {
        return this.page;
    }

    public Integer getSize() {
        return this.size;
    }

    public void setPage(Integer page) {
        this.page = page;
    }

    public void setSize(Integer size) {
        this.size = size;
    }

    public static PostSearch.PostSearchBuilder builder() {
        return new PostSearch.PostSearchBuilder();
    }
}

- @Builder.Default 추가

public class PostSearch {
    private static final int MAX_SIZE = 2000;
    private Integer page = 1;
    private Integer size = 10;

    public long getOffset() {
        return (long)(Math.max(1, this.page) - 1) * (long)Math.min(this.size, 2000);
    }

    public PostSearch() {
        System.out.println("good");
    }

    public PostSearch(Integer page, Integer size) {
        this.page = page;
        this.size = size;
    }

    public Integer getPage() {
        return this.page;
    }

    public Integer getSize() {
        return this.size;
    }

    public void setPage(Integer page) {
        this.page = page;
    }

    public void setSize(Integer size) {
        this.size = size;
    }

    public static PostSearch.PostSearchBuilder builder() {
        return new PostSearch.PostSearchBuilder();
    }
}

관련링크: https://github.com/projectlombok/lombok/issues/1347

그런데 @NoArgsConstructor, @AllArgsConstructor를 달면 기본 생성자에 default 값을 넣어줘서 NPE가 발생하지 않네요. 저도 처음알았습니다.

정리를해보면

@Builder
@AllArgsConstructor
@NoArgsConsturctor 를 달고 컴파일 된 값을 확인하면

 public PostSearch() {

        this.page = $default$page();

        this.size = $default$size();

    }

위와 같이 기본 생성자에 defaultValue를 넣어줍니다.

그런데

@AllArgsConstructor

@NoArgsConsturctor 를 제거하고

public class PostSearch {

    public PostSearch() {

    }

    public PostSearch(Integer page, Integer size) {

        this.page = page;

        this.size = size;

    }

위와같이 직접 생성자를 생성하면 defaultValue가 들어가지 않습니다.

안녕하세요 호돌맨님 !

저는 ModelAttribute에서 값을 주입하지 못해 npe가 발생한다고 생각해 본문처럼 PostSearch 클래스에

@NoArgsConstructor

@AllArgsConstructor

두 어노테이션을 붙여서 해결했었는데요

이 방법이 아닌걸까요? ㅠㅠ

컴파일된 PostSearch를 열어보니까 기본생성자에서 기본값 셋팅을 하고 있더라구요

기본 생성자에서 왜 @Builder.Default 이 지정된 기본값으로 셋팅 하는지 잘 모르겠습니다 ㅎㅎ;;

(@Builder.Default는 헷갈리기만 하고 이 설명에서는 의미가 없어서 제외합니다.)

1. page값이 null로 들어가는 이유

그림 1-1

그림 1-2

그림 1-1처럼 @Builder를 작성하면 모든값-생성자(=@AllArgsconstructor)가 만들어집니다. 그리고 get parameter는 해당 생성자를 통해 값이 주입됩니다. 그런데 page값을 보내지 않으면 그림 1-2처럼 null이 할당되고 에러가 발생하겠죠. 그러므로 page 기본 값 1로 지정 했어도 null로 바뀌게 됩니다.

자, 이번에는 문제의 @NoArgsConstructor와 @AllArgsConstructor를 같이 입력했습니다. 사실 @AllArgsConsturctor는 빌더를 통해 만들어지죠? 의미가 없습니다. 이번 설명에는 @NoArgsConsturctor가 붙은게 중요합니다. 이렇게 되면 @Setter(public void setPage(..))를 통해 값이 주입됩니다. 어라?! 그런데 우리는 page값을 넘기지 않았습니다. setPage() 가 호출되지 않습니다. 기본 값으로 넣어둔 1으로 계속 남게됩니다.
그러므로 @NoArgsConstructor와 @AllArgsConstructor를 같이 써서 해결 됐다라고 할 수 없습니다. @NoArgsConstructor 덕분에 PostSearch의 인스턴스가 생성 되었지만 setPage를 통해 값이 주입되지 않았기 때문에 테스트가 성공한겁니다. (setter, 생성자 주입 우선순위 검색!)

그렇다면 어떻게 코드를 고치면 좋을까요? 우선 @AllArgsConstructor는 안티 패턴으로 사용하지 않는것이 좋습니다.

1. 그냥 @NoArgsConsturctor를 사용는 것도 방법입니다. (사실 개인적으로는 @NoArgsConsturctor도 선호하지는 않음)

2. 생성자에서 null 체크를 하는것도 방법입니다.

3. page, size의 타입 Integer를 int로 바꿔보는것도 방법도 있습니다. 그리고 나서 page를 빼고 보내면 어떤 일이 발생할까요?