New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DTO 작성시 nullPointer.. #102

Open

YOONSEOUNGHWAN opened this issue Sep 7, 2022 · 1 comment

Contributor

YOONSEOUNGHWAN commented Sep 7, 2022

@DaTa
public class ResponsePetitionDto {
private String status="";
private String category;
private String title;
private String createDate;

만에 하나라도 정보가 없을 경우 nullPointer를 뱉어내는데,
어떤 관점이 더 좋은지 궁금해서 남깁니다...

예외처리를 미리 해놓는 것은 추후 유지보수에서 좋지 않으므로, DTO는 데이터 맵핑의 역할로 보자면 데이터가 모두 완전하다는 전제하에 맵핑해준느 것이므로, 굳이 초기값을 넣어놓을 필요가 없음.
그럼에도 불구하고, 에러가 뜨는 것을 막기 위해서 초기값을 걸어둔다.
nullPointerException을 따로 Handling할 수는 없고, 그렇다면 DTO 에 있는 모든 속성을 검사해야하는데, 3가지 중 어떤 생각이실까요?

Contributor

SeungYongChoi commented Sep 10, 2022

우선 초기값을 넣어두는것은 좋지 않다고 봅니다.

데이터가 없는 경우 nullPointerException이 발생하나요? 보통 null값으로 나오는걸로 알고 있는데 한번 확인해 봐야할 것 같습니다.
어떤 api인가요?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment