[Spring Core] 김민준B 과제 제출합니다. by jabell0310 · Pull Request #4 · BCSDLab-Edu/2024-BackEnd

jabell0310 · 2024-06-03T07:08:02Z

구글 문서에 작성해두겠습니다.

ImTotem

고생하셨습니다
점점 깔끔해지는게 보이네요

ImTotem · 2024-06-05T09:32:31Z

src/main/java/com/example/demo/exception/GlobalExceptionHandler.java

+    public ResponseEntity<ErrorResponse> RequestNullExistException() {
+        ErrorResponse response = new ErrorResponse(ErrorCode.REQUEST_NULL_VALUE);
+        return new ResponseEntity<>(response, HttpStatusCode.valueOf(400));
+    }
+
+    @ExceptionHandler(MemberNotExistException.class)
+    public ResponseEntity<ErrorResponse> MemberNotExistException() {
+        ErrorResponse response = new ErrorResponse(ErrorCode.MEMBER_NOT_EXIST);
+        return new ResponseEntity<>(response, HttpStatusCode.valueOf(400));
+    }
+
+    @ExceptionHandler(BoardNotExistException.class)
+    public ResponseEntity<ErrorResponse> BoardNotExistException() {
+        ErrorResponse response = new ErrorResponse(ErrorCode.BOARD_NOT_EXIST);
+        return new ResponseEntity<>(response, HttpStatusCode.valueOf(400));
+    }
+
+    @ExceptionHandler(ArticleNotFoundException.class)
+    public ResponseEntity<ErrorResponse> ArticleNotFoundException() {
+        ErrorResponse response = new ErrorResponse(ErrorCode.ARTICLE_NOT_EXIST);
+        return new ResponseEntity<>(response, HttpStatusCode.valueOf(404));
+    }
+
+    @ExceptionHandler(MemberNotFoundException.class)
+    public ResponseEntity<ErrorResponse> MemberNotFoundException() {
+        ErrorResponse response = new ErrorResponse(ErrorCode.MEMBER_NOT_FOUND);
+        return new ResponseEntity<>(response, HttpStatusCode.valueOf(404));
+    }
+
+    @ExceptionHandler(BoardNotFoundException.class)
+    public ResponseEntity<ErrorResponse> BoardNotFoundException() {
+        ErrorResponse response = new ErrorResponse(ErrorCode.BOARD_NOT_FOUND);
+        return new ResponseEntity<>(response, HttpStatusCode.valueOf(404));
+    }
+
+    @ExceptionHandler(AlreadyHasEmailException.class)
+    public ResponseEntity<ErrorResponse> AlreadyHasEmailException() {
+        ErrorResponse response = new ErrorResponse(ErrorCode.EMAIL_EXIST);
+        return new ResponseEntity<>(response, HttpStatusCode.valueOf(409));
+    }
+
+    @ExceptionHandler(MemberHasArticleException.class)
+    public ResponseEntity<ErrorResponse> MemberHasArticleException() {
+        ErrorResponse response = new ErrorResponse(ErrorCode.MEMBER_ARTICLE_EXIST);
+        return new ResponseEntity<>(response, HttpStatusCode.valueOf(400));
+    }
+
+    @ExceptionHandler(BoardHasArticleException.class)
+    public ResponseEntity<ErrorResponse> BoardHasArticleException() {


자바 naming convention은 지켜주세요

참고하고 다음 실습 때 고치도록 하겠습니다!

ImTotem · 2024-06-05T09:34:44Z

src/main/resources/application.yml

gitignore로 제외해주세요

넵 제외하겠습니다! 궁금해서 묻는건데 제외하는 이유가 있나요??

application.yml에는 유출되면 보안에 치명적인 정보가 많이 들어있어요.
그래서 원격에 올릴 경우 막대한 피해를 보게 될 수도 있어서 application.yml등과 같은 민감한 파일은 올리지 않는 것이 좋아요.

아하 그렇군요! 알겠습니다!

ImTotem · 2024-06-05T11:25:14Z

src/main/java/com/example/demo/exception/ErrorCode.java

+    private final int status;
+    private final String code;


Status Code는 상태 코드라고 불리며 숫자로 나타냅니다.
결국 Status Code는 상태를 나타내는 코드(숫자)를 의미한다고 생각해요.
따라서 code가 int, status가 String이어야 된다고 생각해요.

status가 int, code가 String인 이유가 있나요?

앗... 지금 보니까 설정이 뒤바뀌었네요.. 고치도록 할게요!

ImTotem · 2024-06-05T11:30:12Z

src/main/java/com/example/demo/exception/GlobalExceptionHandler.java

+    @ExceptionHandler(RequestNullExistException.class)
+    public ResponseEntity<ErrorResponse> RequestNullExistException() {
+        ErrorResponse response = new ErrorResponse(ErrorCode.REQUEST_NULL_VALUE);
+        return new ResponseEntity<>(response, HttpStatusCode.valueOf(400));


ErrorCode.REQUEST_NULL_VALUE.getStatus()를 안쓰고 HttpStatus.valueOf(400)로 직접 지정한 이유가 있나요?

getStatus를 써도 같은 결과가 발생하는 군요! 하나 더 배워갑니다!

ImTotem · 2024-06-05T11:31:32Z

src/main/java/com/example/demo/exception/ErrorCode.java

깔끔 하네요 👍

ImTotem · 2024-06-05T11:35:21Z

src/main/java/com/example/demo/exception/GlobalExceptionHandler.java

+public class GlobalExceptionHandler {
+
+    @ExceptionHandler(RequestNullExistException.class)
+    public ResponseEntity<ErrorResponse> RequestNullExistException() {


Exception에 message가 담겨있는 message를 반환해야할 경우에는 어떻게 핸들링 해야할지 생각해보세요 :)

사용자 지정 Exception 매개변수에 message를 넣어서 반환하면 됩니다!

jabell0310 · 2024-06-17T14:35:38Z

고생하셨습니다 점점 깔끔해지는게 보이네요

감사합니다.👍

jabell0310 added 10 commits June 3, 2024 15:31

chore: yml 내 데이터베이스에 연동

45211c7

chore: build.gradle에 AOP, Lombok dependencies 추가

e192196

refactor: Entity 객체에 Lombok 적용

a7c8dc9

feat: 에러 응답 객체 생성

33326af

feat: 에러 열거형 객체 생성

3ce3fc3

feat: 사용자 정의 예외 생성

f3a2bfc

feat: AOP를 활용한 전역 예외 처리 객체 생성

58fb4db

feat: authorId 필드 추가

d7e977c

feat: getById()에서 조회 실패로 발생하는 예외를 null 값으로 처리

0b8bab0

feat: 비즈니스 로직에 사용자 예외 발생 부분 추가

c8d1a4e

ImTotem reviewed Jun 5, 2024

View reviewed changes

jabell0310 added 12 commits June 23, 2024 10:14

fix: gitignore에 application.yml 추가

733ebcc

fix: status와 code의 자료형 수정

e1eb6e7

fix: 전역예외처리 메서드 자바 컨벤션에 맞게 수정

cfb4ea0

fix: yml 파일 jpa 속성 추가

18e2fd3

fix: Entity 객체 Mysql 테이블과 매핑

eca08d9

fix: Repository 객체 JPA의 EntityManager를 사용하여 대체

745fc8f

fix: 클래스 이름 Jpa로 수정

07ee75c

test: 영속성 컨텍스트를 활용한 CRUD 테스트 코드 추가

a66e562

feat: 연관 관계 및 영속성 전이, 고아 객체 개념 적용

3688fb6

refactor: Spring Data JPA를 사용하여 기존 코드 보완

4904efa

refactor: 사용하지 않는 Test 코드 삭제

378413f

error: 오류 미해결

01f368b

Conversation

jabell0310 commented Jun 3, 2024

Uh oh!

ImTotem left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jabell0310 commented Jun 17, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants