[JDBC 라이브러리 구현하기 - 3,4단계] 호이(이건호) 미션 제출합니다. #603

This2sho · 2023-10-10T08:27:31Z

나이스 투 매튜... 🙇‍♂️
죄송합니다.. 휴일에 정신 못차리고 놀다보니 많이 늦어버렸네요..

3단계에서는 트랜잭션의 경계를 두고 {커밋 | 롤백 }이 가능하게 구현했습니다.

4단계에서는 3단계에서 구현한 부분을 주어진 DataSourceUtils와 TransactionSynchronizationManager로 수정하고 TransactionManager를 통해 리팩토링 하였습니다...

4단계 커밋 내용

- 변수 네이밍 명확하게 수정 - 사용하지 않는 log 제거

kpeel5839

my gun ho zzang...

너무나도 기가막힌 코드인 것 같습니다.

그래서 정말 죄송하지만, 작은 제안 하나 정도만 남겼습니다.

해당 코멘트 읽어봐주시고 답변해주신다면 바로 Approve 해드리도록 할게요!

함께 할 수 있어서 아주 즐거웠습니다. my gun ho zzang pu san university

kpeel5839 · 2023-10-11T01:39:36Z

docs/README.md

 - [x] JDBC 탬플릿의 중복된 try catch문 제거
+
+## 3단계
+- [x] 매튜 리뷰 반영


my gun ho ... 의 readme 에 저의 닉네임이 들어가다니 이렇게 감동적일 수가 없습니다..

oh jae yun ... u r so kind~

kpeel5839 · 2023-10-11T01:40:32Z

app/src/main/java/com/techcourse/dao/UserDao.java

+    private static final RowMapper<User> USER_ROW_MAPPER = rs -> new User(
+            rs.getLong("id"),
+            rs.getString("account"),
+            rs.getString("password"),
+            rs.getString("email"));



사실 1, 2, 3, 4 보다 이렇게 직접적으로 column 명을 명시해주는 것이 좋아보이긴 했는데 이렇게 바꿔주셨군요! 멋집니다

kpeel5839 · 2023-10-11T01:41:12Z

app/src/main/java/com/techcourse/dao/UserHistoryDao.java


 public class UserHistoryDao {

-    private static final Logger log = LoggerFactory.getLogger(UserHistoryDao.class);


사용하지 않는 로그들도 다 지워주셨네요 역시 my gun ho zzang ..

ur jae yun zzang too

kpeel5839 · 2023-10-11T01:46:01Z

...src/main/java/org/springframework/transaction/support/TransactionSynchronizationManager.java

 import javax.sql.DataSource;
 import java.sql.Connection;
 import java.util.Map;

 public abstract class TransactionSynchronizationManager {

-    private static final ThreadLocal<Map<DataSource, Connection>> resources = new ThreadLocal<>();
+    private static final ThreadLocal<Map<DataSource, Connection>> resources = init();


저는 이 부분에서 ThreadLocal 이 알아서 초기화 해주는 줄 알고 많이 해맸었는데요!

그러다가 아래와 같은 방법을 찾을 수 있었습니다.

ThreadLocal.withInitial(HashMap::new);

꼭 코드량을 줄여야 하는 것은 아니지만, 해당 방법으로 진행하게 되면, 한 줄로 thread local 을 원하는대로 초기화 해줄 수 있어요!

이런 방법이 있었네요..!
오우.. 역시 매튜 👍
바로 적용하겠습니다!

kpeel5839 · 2023-10-11T01:49:58Z

jdbc/src/main/java/org/springframework/transaction/support/TransactionManager.java

+            runnable.run();
+            return null;
+        });


조회, 업데이트를 나누어 해당 execute 메서드에서 return null 을 없애주는 방법은 어떨까요??

좋습니다!

지금 반영을 했는데 메튜가 원하는 방향이 맞을까요?!
(뭔가 결과를 반환하고 안하고를 제외하면 중복적인거 같아서)

kpeel5839 · 2023-10-11T01:52:28Z

jdbc/src/main/java/org/springframework/jdbc/core/JdbcTemplate.java

-            final var pss = newArgumentPreparedStatementSetter(parameters);
-            pss.setValues(ps);
-            return ps.executeUpdate();
+        execute(new PreparedStatementCreator(sql), statement -> {


훨씬 이해하기 좋아진 것 같아요 my gun ho !

덕분입니다 ur jae yun !

kpeel5839 · 2023-10-11T01:55:28Z

app/src/main/java/com/techcourse/service/AppUserService.java

+    public List<UserHistory> findLogsByUserId(final long id) {
+        return userHistoryDao.findByUserId(id);
+    }
+}


테스트를 위해 메서들르 추가해주셨군요! 좋습니다

kpeel5839

hyohoyohyohoyhoyhoyhohyohyohyo

지금까지 코드 리뷰하면서 정말 많이 배울 수 있었습니다.

당신 정말 멋져요우..

앞으로도 계속 선한 영향력을 주시기를...

kpeel5839 · 2023-10-12T10:52:40Z

...src/main/java/org/springframework/transaction/support/TransactionSynchronizationManager.java

@@ -7,16 +7,10 @@

 public abstract class TransactionSynchronizationManager {

-    private static final ThreadLocal<Map<DataSource, Connection>> resources = init();
+    private static final ThreadLocal<Map<DataSource, Connection>> resources = ThreadLocal.withInitial(HashMap::new);


바로 반영해주셨군요 아주 기모찌 합니다.

kpeel5839 · 2023-10-12T10:53:56Z

jdbc/src/main/java/org/springframework/transaction/support/TransactionManager.java

+        final Connection connection = DataSourceUtils.getConnection(dataSource);
+        try {
+            connection.setAutoCommit(false);
            runnable.run();
-            return null;
-        });
+            connection.commit();
+        } catch (SQLException e) {
+            rollback(connection);
+            throw new DataAccessException(e);
+        } finally {
+            DataSourceUtils.releaseConnection(connection, dataSource);
+        }


제가 말한게 정확히 맞습니다! 역시 hoyhoy gun ho zzang..

kpeel5839 · 2023-10-12T10:56:44Z

...src/main/java/org/springframework/transaction/support/TransactionSynchronizationManager.java

+    public static boolean isActiveTransaction(Connection connection) {
+        return activeTransactions.get().getOrDefault(connection, false);
+    }
+
+    public static void setActiveTransaction(Connection connection, boolean status) {
+        activeTransactions.get().put(connection, status);
+    }


오 setActiveTransaction 과 isActiveTransaction 이 추가되면서 훨씬 로직들이 단단해진 느낌이네요! 아주 좋습니다

This2sho added 11 commits October 8, 2023 21:55

study: 학습 테스트 수행

38a876a

docs: 3단계 구현 기능 목록 작성

0d14867

refactor: 매튜 리뷰 반영

3ebcdd6

- 변수 네이밍 명확하게 수정 - 사용하지 않는 log 제거

refactor: ResultSet에서 컬럼명으로 받도록 수정

9948c6c

refactor: UserHistoryDao JdbcTemplate 쓰도록 변경

ac565a0

feat: UserHistoryDao에서 변경 이력 확인할 수 있도록 구현

29dffb3

feat: Transaction 설정을 위한 기능 구현

fb3c95e

docs: 기능 요구사항 작성

cd7d868

refactor: Transaction synchronization 적용

e1a6d55

refactor: 3단계에서 작성한 코드 제거

ab2d58e

refactor: TransactionManager에서 트랜잭션을 관리할 수 있도록 수정

6f48ae4

kpeel5839 requested changes Oct 11, 2023

View reviewed changes

This2sho added 3 commits October 11, 2023 13:28

refactor: ThreadLocal 초기화 방법 변경

f3a013d

refactor: null 반환 대신 새로운 메소드로

2edac4c

refactor: Transaction 없이 사용했을 때 커낵션을 종료할 수 있도록 수정

5d566d1

This comment was marked as spam.

Sign in to view

kpeel5839 approved these changes Oct 12, 2023

View reviewed changes

kpeel5839 merged commit b899b46 into woowacourse:this2sho Oct 12, 2023
1 check failed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[JDBC 라이브러리 구현하기 - 3,4단계] 호이(이건호) 미션 제출합니다. #603

[JDBC 라이브러리 구현하기 - 3,4단계] 호이(이건호) 미션 제출합니다. #603

This2sho commented Oct 10, 2023 •

edited

Loading

kpeel5839 left a comment

kpeel5839 Oct 11, 2023

This2sho Oct 11, 2023

kpeel5839 Oct 11, 2023

This2sho Oct 11, 2023

kpeel5839 Oct 11, 2023

This2sho Oct 11, 2023

kpeel5839 Oct 11, 2023

This2sho Oct 11, 2023

kpeel5839 Oct 11, 2023

This2sho Oct 11, 2023

This2sho Oct 11, 2023

kpeel5839 Oct 11, 2023

This2sho Oct 11, 2023

kpeel5839 Oct 11, 2023

This comment was marked as spam.

kpeel5839 left a comment

kpeel5839 Oct 12, 2023

kpeel5839 Oct 12, 2023

kpeel5839 Oct 12, 2023


		public class UserHistoryDao {

		private static final Logger log = LoggerFactory.getLogger(UserHistoryDao.class);

[JDBC 라이브러리 구현하기 - 3,4단계] 호이(이건호) 미션 제출합니다. #603

[JDBC 라이브러리 구현하기 - 3,4단계] 호이(이건호) 미션 제출합니다. #603

Conversation

This2sho commented Oct 10, 2023 • edited Loading

kpeel5839 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

This comment was marked as spam.

kpeel5839 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

This2sho commented Oct 10, 2023 •

edited

Loading