[Spring Core] 신관규 과제 제출합니다. #14

Soundbar91 · 2024-06-25T02:06:22Z

질문

게시판 삽입 예외 발생 후 식별자 증가

6번 게시물을 삽입하고 7번 게시물을 삽입할 때 예외를 발생시켰습니다.
예외를 발생시켰기 때문에 롤백돼서 그다음으로 들어가는 게시물의 식별자는 7번일거라고 생각했습니다.
하지만 데이터베이스를 확인하면 7번이 아닌 8번으로 들어가 있습니다.
현재 식별자를 애플리케이션에서 증가시키는 전략이기 때문에 이와 같은 현상이 나타나는 건지, 아니면 다른 이유가 있는지 궁금합니다 !

게시판, 회원 삭제 예외

JDBC 코드에서는 예외를 정상적으로 잡았는데, JPA로 바꾸니 예외를 잡지 못하는 현상이 나타났습니다.

@Override
    public void deleteById(Long id) {
        try {
            entityManager.remove(findById(id));
        } catch (Exception e) {
            throw new ApplicationException(MEMBER_REFERENCE);
        }
    }

현재는 임시(?)로 Exception e로 작성했는데 역시나 예외를 잡지 못하고 있으며, PersistenceException와 DataIntegrityViolationException로 설정해도 아예 못 잡고 있습니다.
에러 메시지에는 Controller만 찍혀있어서 다른 곳에서 예외가 발생한건지, 아니면 놓치고 간 부분이 있는지 조언을 구하고 싶습니다.

EntityTransaction

EntityManager가 수행하는 로직은 트랜잭션 내부에서 실행되야 한다를 만족하기 위해 EntityTransaction에서 begin() 메소드를 통해 트랜잭션을 얻고 시작하는 코드를 작성했습니다.

@Override
    public Article insert(Article article) {
        try {
            entityTransaction.begin();
            entityManager.persist(article);
            return entityManager.find(Article.class, article.getId());
        } catch (PersistenceException e) {
            entityTransaction.rollback();
            throw new ApplicationException(FK_NOT_EXISTS);
        }
    }

실습과제를 진행하던 중 Service에서 @Transactional으로 트랜잭션을 여는데 begin() 메소드로 또 트랜잭션을 열어야 하나?라는 생각이 들었습니다.
그래서 EntityTransaction와 관련있는 코드는 모두 삭제하고 진행을 했습니다.
이렇게 했을 때 향후에 다른 문제가 발생하는지 궁금합니다 !

duehee

다양하게 잘 적용해보셨네요~
다음 실습도 화이팅해주세요!

duehee · 2024-06-26T10:22:11Z

build.gradle

 	runtimeOnly 'com.mysql:mysql-connector-j'
 	testImplementation 'org.springframework.boot:spring-boot-starter-test'
+	implementation 'org.springframework.boot:spring-boot-starter-data-jpa'
+	implementation 'org.springframework.boot:spring-boot-starter-validation'


개인적인 의견이긴 한데, 같은 dependencies들은 묶고, 나머지는 띄어쓰기 하는 게 조금 더 보기 좋더라구요

따로 생각을 안 하고 추가만 했었는데, 말씀해 주신 것처럼 가독성 좋게 바꿔보겠습니다.

굿 반영 잘 되었네요

duehee · 2024-06-26T10:24:21Z

src/main/java/com/example/demo/controller/dto/request/ArticleCreateRequest.java

    String description
 ) {
-
+    public ArticleCreateRequest {


request에서 예외를 던지는 게 맞을까요? DTO는 데이터 전달 객체이고, 비즈니스 로직 및 예외처리는 Service단에서 해결하는 게 맞다고 생각이 들어요

DTO와 관련이 없는 책임을 준거 같습니다. 반영하겠습니다 !

duehee · 2024-06-26T10:25:53Z

src/main/java/com/example/demo/domain/Article.java

+
 import java.time.LocalDateTime;

+@Getter


Getter 사용 및 도메인 내에서의 Setter 사용에 대해 잘 생각해보셨군요!

duehee · 2024-06-26T10:31:56Z

src/main/java/com/example/demo/repository/ArticleRepositoryJdbc.java

+                    article.getId()
+            );
+            return findById(article.getId());
+        } catch (DataIntegrityViolationException e) {


다양한 예외처리 굿👍

duehee · 2024-06-26T10:32:59Z

src/main/java/com/example/demo/repository/ArticleRepositoryJpa.java

+@Repository
+public class ArticleRepositoryJpa implements ArticleRepository {
+
+    @PersistenceContext


배우신 내용을 잘 활용하셨네용

duehee

이번 주도 고생하셨습니당
코멘트 확인해주세요~

duehee · 2024-07-03T08:11:56Z

build.gradle

 	runtimeOnly 'com.mysql:mysql-connector-j'
 	testImplementation 'org.springframework.boot:spring-boot-starter-test'
+	implementation 'org.springframework.boot:spring-boot-starter-data-jpa'
+	implementation 'org.springframework.boot:spring-boot-starter-validation'


굿 반영 잘 되었네요

duehee · 2024-07-03T08:12:33Z

src/main/java/com/example/demo/controller/ArticleController.java

    @GetMapping("/articles")
    public ResponseEntity<List<ArticleResponse>> getArticles(
-        @RequestParam Long boardId
+            @RequestParam Long boardId


들여쓰신 이유가 있나용?

인텔리제이 코드 정렬 기능(?)을 사용하니 들여쓰기가 들어간거 같습니다.

duehee · 2024-07-03T08:18:53Z

src/main/java/com/example/demo/controller/dto/request/ArticleCreateRequest.java

-    Long boardId,
-    String title,
-    String description
+    @NotNull(message = "회원 아이디는 필수로 입력해야 합니다.") Long authorId,


어노테이션의 적극적인 사용..!

duehee · 2024-07-03T08:26:15Z

src/main/java/com/example/demo/controller/dto/request/MemberCreateRequest.java

+        return Member.builder()
+                .name(this.name)
+                .email(this.email)
+                .password(this.password).build();


이 친구는 혼자 붙어있네요

Suggested change

.password(this.password).build();

.password(this.password)

.build();

duehee · 2024-07-03T08:26:48Z

src/main/java/com/example/demo/domain/Article.java

-    private Long boardId;
+
+    @NotNull
+    @ManyToOne(fetch = LAZY)


static import를 사용하셨네요!😮

duehee · 2024-07-03T08:27:38Z

src/main/java/com/example/demo/domain/Member.java

+@AllArgsConstructor
 public class Member {

+    @Id @GeneratedValue(strategy = IDENTITY)


저는 어노테이션 떨어트려놓는게 더 보기 좋더라구요 ㅎ.ㅎ.....

Suggested change

@Id @GeneratedValue(strategy = IDENTITY)

@Id

@GeneratedValue(strategy = IDENTITY)

duehee · 2024-07-03T08:31:13Z

src/main/java/com/example/demo/repository/ArticleRepositoryJdbc.java

-@Repository
+import static com.example.demo.exception.ErrorCode.*;
+
 public class ArticleRepositoryJdbc implements ArticleRepository {


JPA를 사용하시면서 위에서 메소드를 다 삭제하셨는데, 아래에 있는 RepoJDBC, RepoJPA를 유지하신 이유가 있나요?
아래 Repository들이 그런 내용으로 서술되어 있어서 여쭤봅니다

처음에는 주석처리해서 유지하지 않는 방향으로 갔었으나, 나중에 깃헙에서 코드를 볼 수 있는 상황이 있을 경우 보기 편하게 하기 위해서 주석처리를 지우고 유지하자고 판단했습니다.

duehee · 2024-07-03T08:38:33Z

src/main/resources/application.yml

+    properties:
+      hibernate:
+        show_sql: true
+        format_sql: true


마지막에 경고 표시가 떠있는데, 이거 처리하는 방법이 있답니다

오.. 어디서 본 거 같은 블로그

duehee

고생하셨습니다~
이제 슬슬 파일이 쌓이기 시작하다보니 조금씩 헷갈리네요😵
로그인 잘 구현해주셨고, 수업도 화이팅하세요!

duehee · 2024-07-10T08:06:43Z

src/main/java/com/example/demo/service/ArticleService.java

-        Member member = memberRepository.findById(article.getAuthorId());
-        Board board = boardRepository.findById(article.getBoardId());
-        return ArticleResponse.of(article, member, board);
+    public ArticleResponse getByArticleId(Long id) {


ㅎㅎㅎㅎ 의견 다 반영해주셨네요~!
메소드 명이 더 명확해져서 좋아요

duehee · 2024-07-10T08:21:17Z

src/main/java/com/example/demo/service/BoardService.java

+                    .orElseThrow(() -> new ApplicationException(BOARD_NOT_FOUND));
+
+            boardRepository.delete(board);
+            boardRepository.flush();


어느 부분에는 flush()를 선언하고, 어느 부분에서는 사용하지 않았는데 이유가 있을까요?
위의 게시물에는 삭제하는 부분엔 flush가 없는데, 아래의 게시판을 삭제하는 경우에는 적용하고 계시길래 말씀 드려봤어요

게시판를 삭제할 때 작성된 글이 있는 경우 커스텀으로 정의한 예외를 발생 시켜야하는데, flush() 메소드를 통해 강제로 flush을 발생시키기 위해 게시판에만 적용했었습니다. flush() 메소드가 있는 메소드들은 다 예외 처리때문에 작성했었습니다.

duehee · 2024-07-10T08:25:44Z

src/main/resources/templates/index.html

@@ -0,0 +1,19 @@
+<!DOCTYPE html>


웹까지..! 대단하시네용

duehee · 2024-07-10T08:47:29Z

src/main/java/com/example/demo/service/LoginService.java

+@Service
+public class LoginService {
+
+    private final MemberRepository memberRepository;
+    private final PasswordEncoder passwordEncoder;
+
+    public LoginService(MemberRepository memberRepository, PasswordEncoder passwordEncoder) {
+        this.memberRepository = memberRepository;
+        this.passwordEncoder = passwordEncoder;
+    }


Suggested change

@Service

public class LoginService {

private final MemberRepository memberRepository;

private final PasswordEncoder passwordEncoder;

public LoginService(MemberRepository memberRepository, PasswordEncoder passwordEncoder) {

this.memberRepository = memberRepository;

this.passwordEncoder = passwordEncoder;

}

@Service

@RequiredArgsConstructor

public class LoginService {

private final MemberRepository memberRepository;

private final PasswordEncoder passwordEncoder;

소소한 팁인데, @RequiredArgsConstructor를 사용하면 final( + NotNull) 선언한 변수들에 대해서 생성자 주입을 해준답니다.
코드 짤 때 더 간결해지고, 좋아요!
블로그 참고

duehee · 2024-07-10T08:52:02Z

src/main/java/com/example/demo/service/LoginService.java

+        Member member = memberRepository.findByEmail(loginRequest.email())
+                .orElseThrow(() -> new ApplicationException(MEMBER_NOT_FOUND));
+
+        if (!passwordEncoder.matches(loginRequest.password(), member.getPassword())) return null;


맞지 않는 경우에 null을 반환하는 이유가 있나요?
예외 처리를 하는 것은 어떨까요?

비밀번호가 틀린 경우 다시 로그인 화면으로 리다이렉트를 하기 위해 null로 처리했습니다.
처음에는 예외 처리 방향으로 갔다가, 리다이렉트를 어떻게 하지라는 생각에 null로 변경했었습니다. 너무 쉬운 방향으로 갔던 것 같습니다.
예외 처리하는 방식으로 해보겠습니다 !

Soundbar91 added 28 commits June 16, 2024 13:44

feat: 게시물 조회 예외 처리 추가

b2f24ee

feat: 게시판 조회 예외 처리 추가

9db00f1

feat: 사용자 조회 예외 처리 추가

a1286bc

feat: 사용자 수정 예외 처리 추가

4c61209

feat: 게시물 수정 예외 처리 추가

c14562b

feat: 게시물/게시판/사용자 생성 null 예외 처리 추가

48743a0

feat: 게시물 생성 예외 처리 추가

9faed4e

feat: 사용자 삭제 예외 처리 추가

57ba800

feat: 게시판 삭제 예외 처리 추가

e3af0d3

refactor: 게시물 조회 예외 처리 수정

e15f15f

refactor: 게시판 조회 예외 처리 수정

71156f5

refactor: 회원 조회 예외 처리 수정

c51d743

style: 라인 포멧팅 및 static import

9f4c138

refactor: application.yml 수정

02800bb

chore: JPA 의존성 추가

de1db1e

chore: Lombok 의존성 추가

27c74e5

chore: Validation 의존성 추가

8856214

build: jpa 설정 정보 추가

da4c91d

refactor: Article 리펙터링

1807914

refactor: ArticleRepositoryJpa 교체

25b516d

refactor: Board 리펙터링

2187c91

refactor: BoardRepository Jpa 교체

5826784

refactor: Member 리펙터링

fd80235

refactor: MemberRepository Jpa 교체

c6965ad

chore: 미사용 import 제거

e227a2e

refactor: entity 컬럼 어노테이션 추가

48864b1

refactor: EntityTransaction 삭제

461422f

feat: 회원/게시판 삭제 예외 임시 작성

a3015bb

duehee reviewed Jun 26, 2024

View reviewed changes

refactor: Domain 리펙터링

811f597

Soundbar91 added 15 commits June 29, 2024 21:21

refactor: 사용자/게시물/게시판 생성 예외 처리 수정

eb0afbf

refactor: Repository 구현체 주석 처리

afa3ce9

refactor: Member 예외 처리 수정

989fbe3

refactor: Board 예외 처리 수정

266f23a

refactor: Article 예외 처리 수정

175404d

refactor: ErrorCode 수정

84ecd97

refactor: Spring Data JPA 적용

b5a9a86

refactor: 영속성 전의 및 고아 객체 적용

996fff8

build: dataSource 변경

452b487

refactor: builder 패턴 변경

1eb19a3

refactor: 예외 처리 코드 변경

520dcec

refactor: 미 사용 import 제거

1e7596e

refactor: Article 연관관계 수정

3206275

refactor: Builder 패턴 수정

bf2ecb1

과제 제출

4a5f493

duehee reviewed Jul 3, 2024

View reviewed changes

Soundbar91 added 13 commits July 5, 2024 16:14

refactor: 10주차 피드백 반영

1248456

build: Spring Security 의존성 추가

ac02295

feat: 회원가입 기능 추가

14574b6

feat: 로그인 기능 추가

41b53df

feat: 쿠키 전달 기능 추가

8f41770

feat: 쿠키 검증 로직 추가

28d85d0

refactor: 로그인 기능 수정

4c751e5

refactor: 회원가입 기능 수정

7301e71

refactor: 미사용 코드 제거

f80d614

feat: /index 경로 추가

82a7c51

build: 의존성 수정

0b43b71

refactor: 로그인 로직 수정

ec6e866

refactor: 미사용 import 제거

f17f522

duehee reviewed Jul 10, 2024

View reviewed changes

	.password(this.password).build();
	.password(this.password)
	.build();

	@Id @GeneratedValue(strategy = IDENTITY)
	@Id
	@GeneratedValue(strategy = IDENTITY)

[Spring Core] 신관규 과제 제출합니다. #14

Are you sure you want to change the base?

[Spring Core] 신관규 과제 제출합니다. #14

Uh oh!

Conversation

Soundbar91 commented Jun 25, 2024

질문

게시판 삽입 예외 발생 후 식별자 증가

게시판, 회원 삭제 예외

EntityTransaction

Uh oh!

duehee 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!

duehee 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!

duehee 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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants