MemberRepositoryV0를 try-with-resources를 사용해 refactoring 해봤습니다..
673
작성한 질문수 15
자바와 관련된 것이라 조금 조심스럽긴 하지만, 해당 클래스를 아래와 같이 try-with-resources를 사용해 refactoring 해 보았습니다.
테스트 코드는 잘 돌아가는데, 아래와 같이 변경해도 상관없나해서 질문 남깁니다!
감사합니다!
package hello.jdbc.repository;
import hello.jdbc.connection.DBConnectionUtil;
import hello.jdbc.domain.Member;
import lombok.extern.slf4j.Slf4j;
import java.sql.*;
import java.util.NoSuchElementException;
/**
* JDBC - DriverManager 사용
*/
@Slf4j
public class MemberRepositoryV0 {
// jdbc를 이용해 member 객체 저장
public Member save(Member member) throws SQLException {
String sql = "insert into member(member_id, money) values (?,?)";
try (Connection conn = getConnection();
PreparedStatement pstmt = conn.prepareStatement(sql)) {
pstmt.setString(1, member.getMemberId());
pstmt.setInt(2, member.getMoney());
pstmt.executeUpdate();
return member;
} catch (SQLException e) {
log.error("db error", e);
throw e;
} catch (NullPointerException e) {
log.info("NPE 발생", e);
}
return member;
}
public Member findById(String memberId) throws SQLException {
String sql = "select * from member where member_id = ?";
try (Connection conn = getConnection();
PreparedStatement pstmt = createPreparedStatement(sql, conn, memberId);
ResultSet rs = pstmt.executeQuery()) {
if (rs.next()) {
Member member = new Member();
member.setMemberId(rs.getString("member_id"));
member.setMoney(rs.getInt("money"));
return member;
} else {
throw new NoSuchElementException("member not found memberId=" + memberId);
}
} catch (SQLException e) {
log.error("db error", e);
throw e;
} catch (NullPointerException e) {
log.info("NPE 발생", e);
}
return new Member();
}
private PreparedStatement createPreparedStatement(String sql, Connection conn, String memberId) throws SQLException {
PreparedStatement pstmt = conn.prepareStatement(sql);
pstmt.setString(1, memberId);
return pstmt;
}
public void update(String memberId, int money) throws SQLException {
String sql = "update member set money=? where member_id=?";
try (Connection conn = getConnection();
PreparedStatement pstmt = conn.prepareStatement(sql)) {
pstmt.setInt(1, money);
pstmt.setString(2, memberId);
int resultSize = pstmt.executeUpdate();
log.info("resultSize={}", resultSize);
} catch (SQLException e) {
log.error("db error", e);
throw e;
} catch (NullPointerException e) {
log.info("NPE 발생", e);
}
}
public void delete(String memberId) throws SQLException {
String sql = "delete from member where member_id = ?";
try (Connection connection = getConnection();
PreparedStatement preparedStatement = connection.prepareStatement(sql)) {
preparedStatement.setString(1, memberId);
int resultSize = preparedStatement.executeUpdate();
log.info("resultSize={}", resultSize);
} catch (SQLException e) {
log.error("db error", e);
throw e;
} catch (NullPointerException e) {
log.info("NPE 발생", e);
}
}
private static Connection getConnection() {
return DBConnectionUtil.getConnection();
}
}
답변 3
3
안녕하세요, Junho Hwang 님. 공식 서포터즈 y2gcoder 입니다.
JDBC 그대로 사용하고, Java 7 이상 사용하는 환경이라면 아주 좋은 답안지가 될 것 같습니다. 코드 공유해주셔서 감사합니다 :)
Java 9부터는 try-with-resources의 사용성을 향상시켜 밑의 링크처럼 자원 할당을 바깥에서도 할 수 있다고 합니다.
감사합니다.
2
해당 리펙토링을 진행하면서 제가 생각한 장단점은 다음과 같습니다.
장점 ) 코드가 간결해진다. 불필요한 복잡한 close() 메서드를 삭제할 수 있다.
단점)
1. close() 메서드를 삭제함으로서 Connection, Statement, ResultSet에서 close시 exception이 발생했을때 어디서 발생했는지 로그를 통한 조작이 불가능하다.
2. try-with-resources 구문 밖에서 return 값을 한번 더 지정해 주어야 한다.
혼자 학습해 최대한 적용해 보려 했는데, 실무에서는 어떠한 방식으로 사용하는지도 궁금합니다!!
감사합니다 !
spring initialiser 어떤걸 선택해야될지 모르겠어요
0
31
2
트랜잭션 템플릿이 체크예외는 커밋하는 이유가 궁금합니다.
0
50
1
jdbc 학습하다 궁금해서 질문드려요
0
82
1
정상이체와 이체중 예외발생 차이
0
68
2
이번 강의부터가 JDBC 직접 사용 맞나요?
0
87
1
순수한 서비스 계층에 대한 의문???
0
82
2
3_4test, 4test 자동 리소스 안됨
0
58
2
데이터베이스 선택 관련 질문.
0
68
2
after 메서드 사용 유무
0
56
2
데이터베이스 접근 및 DB 락
0
54
1
@SpingBootTest, @TestConfiguration
0
56
1
Read Committed 격리 수준 사용 관련 질문
0
48
1
AopCheck Test 부분의 EnhancerBySpring 확인 법??
0
66
2
커리큘럼 관련 질문
0
76
1
서비서 계층 스프링 프레임워크 사용
0
87
1
validation(toMember)의 위치
0
83
2
데이터 계층에서 서비스 로직의 datasource를 인식하는 방법
0
165
2
member 상수 질문
0
120
2
ctrl + F6 이후로 con 인식이 안 됩니다
0
153
2
DrivenManager 와 Connection 반환에 대한 질문입니다!
0
109
1
섹션5 트랜잭션 템플릿관련 질문(이기종 DB)
0
116
1
세션1 테이블 조회결과 그림은 Redo 로그에 속하나요?
0
99
1
트랜잭션이 필요하지 않은 경우 DB 대신 파일 시스템을 사용하는 것이 더 효율적인가?
0
98
1
강의 14분쯤 Exception 질문
0
114
2





