codej99 / springrestapi Goto Github PK

안녕하세요. 개발자님 코드를 보면서 열심히 공부 중인 학생입니다.

1. 의견: 컨트롤러 메소드들의 return 타입이 CommonResult, SingleResult, ListResult 입니다. 현재는 HTTP response status line에 200 혹은 500 상태 코드만 나오는 상황인데요. RESTful API에 맞게 status line의 상태코드도 직접 지정해주면 어떨까 생각합니다.

2. 해결책 제시:
   ex) 생성의 경우 CREATED를 이용하여 201 코드를 내려주면 더 명확하지 않을까 생각합니다. 다른 것들도 마찬가지구요.
   ResponseEntity
   return new ResponseEntity<>(responseService.getCustomException(), HttpStatus.CREATED);

만약에 이 방법이 괜찮다면 return 타입을 ResponseEntity로 변경 후 pull request 하겠습니다.

아직 공부하는 학생입장이라 부족하지만, 가볍게 읽어주시면 감사하겠습니다!