Skip to content
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.

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

[JDBC 라이브러리 구현 4단계] 메리(최승원) 미션 제출합니다 #584

Merged
merged 54 commits into from
Oct 15, 2023
Merged
Changes from 1 commit
Commits
Show all changes
54 commits
Select commit Hold shift + click to select a range
dc70d1d
패키지 위치 변경 및 코드 정리
kang-hyungu Sep 21, 2023
7182a49
패키지 위치 변경 및 코드 정리
kang-hyungu Sep 21, 2023
06379e5
패키지 위치 변경 및 코드 정리
kang-hyungu Sep 21, 2023
ae100c4
feat: jdbc를 이용한 모든 유저 정보 수정 기능 추가
swonny Oct 3, 2023
42708f0
feat: jdbc를 이용한 모든 유저 정보 조회 기능 추가
swonny Oct 3, 2023
2463faf
feat: jdbc를 이용한 유저 account로 정보 조회하는 기능 추가
swonny Oct 3, 2023
e48fbcd
feat: 단일 조회 객체 RowMapper 생성
swonny Oct 3, 2023
aa39a33
refactor: 아이디를 통한 유저 조회 중복 제거
swonny Oct 3, 2023
95aaf0c
feat: 계정을 통한 유저 조회 기능 구현
swonny Oct 3, 2023
8ff2b3a
feat: 복수 조회를 위한 메서드 생성 및 findAll 중복 제거
swonny Oct 3, 2023
4f04737
feat: update 중복 제거
swonny Oct 4, 2023
f2e785e
feat: insert 중복 제거
swonny Oct 4, 2023
edf550d
refactor: 단일 조회 try-with-resources 사용하도록 수정
swonny Oct 4, 2023
6d9a536
refactor: 단일 조회 아닌 경우에 대한 예외 처리 추가
swonny Oct 4, 2023
1a99a87
fix: 단일 조회 누락된 로직 추가
swonny Oct 4, 2023
8f7b50b
refactor: 복수 조회 메서드 try-with-resources 사용하도록 변경
swonny Oct 4, 2023
efdff4e
refactor: 수정 메서드 try-with-resources 사용하도록 변경
swonny Oct 4, 2023
1d0d0bb
refactor: 불필요한 제네릭 삭제
swonny Oct 4, 2023
a4bc31d
refactor: 데이터가 존재하지 않는 경우 예외 발생이 아닌 Optional을 반환하도록 변경
swonny Oct 4, 2023
3c8ffa6
refactor: 불필요한 필드 제거
swonny Oct 4, 2023
128b208
refactor: 조회하려는 사용자가 없는 경우 DAO에서 예외 발생하도록 변경
swonny Oct 4, 2023
66f2857
feat: 로그 출력
swonny Oct 4, 2023
c68ca2d
refactor: 단일 조회 시 파라미터를 가변인자로 받도록 변경
swonny Oct 4, 2023
f6fc7f6
refactor: 중복되는 로직 메서드로 추출
swonny Oct 4, 2023
14a555d
refactor: 복수 조회 메서드 네이밍 변경
swonny Oct 5, 2023
e1dbf23
refactor: 중복되는 자원 열고 닫는 로직 콜백 패턴 사용하여 제거
swonny Oct 5, 2023
97fa8af
refactor: 콜백에 사용되는 람다 함수 재정의
swonny Oct 5, 2023
99a2021
refactor: jdbc 연결 실패 시 커스텀 unchecked 예외 던지도록 수정
swonny Oct 5, 2023
3963ba3
Merge remote-tracking branch 'upstream/swonny' into step1
swonny Oct 8, 2023
d1c4068
test: 실습
swonny Oct 8, 2023
7a18ba7
refactor: userHistoryDao에서 jdbcTemplate 사용하도록 수정
swonny Oct 8, 2023
35e449d
refactor: 서비스 레이어에서 connection 생성해 같은 트랜잭션에서 재사용하도록 수정
swonny Oct 8, 2023
49309d7
test: 변경된 코드에 맞춰 테스트 코드 수정
swonny Oct 8, 2023
89fcfbe
refactor: 중복되는 자원 열고 닫는 로직 제거
swonny Oct 8, 2023
0c1873a
refactor: 메서드 네이밍 수정
swonny Oct 8, 2023
30ed91e
feat: 트랜잭션 종료 로직 추가
swonny Oct 8, 2023
c570870
refactor: 커스텀 예외를 던지도록 변경
swonny Oct 8, 2023
33dff91
refactor: preparedStatement try-catch 내부에서 생성하도록 변경
swonny Oct 8, 2023
3fc681d
fix: 첫 번째 파라미터만 추가되는 로직 수정
swonny Oct 8, 2023
b4316ef
test: UserHistoryDao 메서드 시그니처 변경에 따라 mock 메서드 시그니처 변경
swonny Oct 8, 2023
8b5b02d
test: UserDao 메서드 시그니처 변경에 따라 테스트 메서드 시그니처 변경
swonny Oct 8, 2023
fe1a0ce
refactor: 함수형 인터페이스 대소문자 변경
swonny Oct 8, 2023
6b0f3f5
Merge remote-tracking branch 'upstream/swonny' into step2
swonny Oct 9, 2023
c732706
refactor: 커넥션을 가져오는 부분을 DataSourceUtils에서 대신하도록 수정
swonny Oct 9, 2023
398d8c8
test: 요구사항에 맞춰 테스트 수정
swonny Oct 9, 2023
5d63d58
test: 커넥션 생성하는 로직 TransactionSynchronizationManager로 이동
swonny Oct 9, 2023
c588c40
refactor: 트랜잭션 관리 로직 TransactionSynchronizationManager로 이동
swonny Oct 9, 2023
70941b9
refactor: 트랜잭션 서비스 추상화
swonny Oct 9, 2023
95dcdfd
refactor: 불필요한 함수형 인터페이스 제거
swonny Oct 9, 2023
973c714
fix: 트랜잭션 종료되지 않은 경우 commit하지 않도록 변경
swonny Oct 9, 2023
e63cb29
fix: 트랜잭셔 매니저의 트랜잭션 시작과 종료가 정상적으로 동작하도록 수정
swonny Oct 14, 2023
05a153f
refactor: 트랜잭션 매니저 생성 및 TxUserService에서 트랜잭션 중복 로직 제거
swonny Oct 14, 2023
f21c08e
refactor: 트랜잭션 매니에서 커밋, 롤백 수행하도록 수정
swonny Oct 14, 2023
88ffa92
refactor: 예외 처리 로직을 TransactionManager에서 수행하도록 변경
swonny Oct 14, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
feat: update 중복 제거
swonny committed Oct 4, 2023
commit 4f04737237f404f709ea6d8bc2b2f03f684ec80c
33 changes: 3 additions & 30 deletions app/src/main/java/com/techcourse/dao/UserDao.java
Original file line number Diff line number Diff line change
@@ -73,36 +73,9 @@ public void insert(final User user) {

public void update(final User user) {
final var sql = "update users set (account, password, email) = (?, ?, ?)";

Connection conn = null;
PreparedStatement pstmt = null;
try {
conn = dataSource.getConnection();
pstmt = conn.prepareStatement(sql);

log.debug("query : {}", sql);

pstmt.setString(1, user.getAccount());
pstmt.setString(2, user.getPassword());
pstmt.setString(3, user.getEmail());
pstmt.executeUpdate();
} catch (SQLException e) {
log.error(e.getMessage(), e);
throw new RuntimeException(e);
} finally {
try {
if (pstmt != null) {
pstmt.close();
}
} catch (SQLException ignored) {
}

try {
if (conn != null) {
conn.close();
}
} catch (SQLException ignored) {
}
final int updatedRows = jdbcTemplate.update(sql, user.getAccount(), user.getPassword(), user.getEmail());
if (updatedRows < 1) {
throw new RuntimeException("수정된 데이터가 없습니다.");
}
}

34 changes: 34 additions & 0 deletions jdbc/src/main/java/org/springframework/jdbc/core/JdbcTemplate.java
Original file line number Diff line number Diff line change
@@ -107,4 +107,38 @@ public <T> List<T> query(final String sql, final RowMapper<T> rowMapper) {
}
}
}

public <T> int update(final String sql, final Object... parameters) {
Connection conn = null;
PreparedStatement pstmt = null;
try {
conn = dataSource.getConnection();
pstmt = conn.prepareStatement(sql);

log.debug("query : {}", sql);

for (int parameterIndex = 0; parameterIndex < parameters.length; parameterIndex++) {
pstmt.setObject(parameterIndex + 1, parameters[parameterIndex]);
}

return pstmt.executeUpdate();
} catch (SQLException e) {
log.error(e.getMessage(), e);
throw new RuntimeException(e);
} finally {
try {
if (pstmt != null) {
pstmt.close();
}
} catch (SQLException ignored) {
}

try {
if (conn != null) {
conn.close();
}
} catch (SQLException ignored) {
}
}
}
}