inflearn logo
강의

강의

N
챌린지

챌린지

멘토링

멘토링

N
클립

클립

로드맵

로드맵

지식공유

스프링 DB 1편 - 데이터 접근 핵심 원리

JDBC 개발 - 수정, 삭제

MemberRepositoryV0를 try-with-resources를 사용해 refactoring 해봤습니다..

673

황준호

작성한 질문수 15

3

자바와 관련된 것이라 조금 조심스럽긴 하지만, 해당 클래스를 아래와 같이 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();
    }

}

spring

답변 3

3

y2gcoder

안녕하세요, Junho Hwang 님. 공식 서포터즈 y2gcoder 입니다.

JDBC 그대로 사용하고, Java 7 이상 사용하는 환경이라면 아주 좋은 답안지가 될 것 같습니다. 코드 공유해주셔서 감사합니다 :)

Java 9부터는 try-with-resources의 사용성을 향상시켜 밑의 링크처럼 자원 할당을 바깥에서도 할 수 있다고 합니다.

https://hyoj.github.io/blog/java/basic/jdbc/jdbc-try-catch-tip.html#java-7-%EB%B6%80%ED%84%B0%EB%8A%94-try-with-resources-%EB%A5%BC-%EC%82%AC%EC%9A%A9%ED%95%98%EC%9E%90

감사합니다.

0

황준호

자바 9부터는 자원 할당을 밖에서도 가능하군요!!
감사합니다!

1

y2gcoder

파이팅입니다!

2

황준호

해당 리펙토링을 진행하면서 제가 생각한 장단점은 다음과 같습니다.
장점 ) 코드가 간결해진다. 불필요한 복잡한 close() 메서드를 삭제할 수 있다.

단점)
1. close() 메서드를 삭제함으로서 Connection, Statement, ResultSet에서 close시 exception이 발생했을때 어디서 발생했는지 로그를 통한 조작이 불가능하다.
2. try-with-resources 구문 밖에서 return 값을 한번 더 지정해 주어야 한다.

혼자 학습해 최대한 적용해 보려 했는데, 실무에서는 어떠한 방식으로 사용하는지도 궁금합니다!!
감사합니다 !

1

아주작은체리

우와 저도 비슷하게 리팩토링 해서 혹시나 하고 찾아봤는데 공식 서포터즈님 덕분에 try-with-resource 문에 대해 더 알아가네요! 모두 감사합니다 ^^

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

98

1

트랜잭션이 필요하지 않은 경우 DB 대신 파일 시스템을 사용하는 것이 더 효율적인가?

0

98

1

강의 14분쯤 Exception 질문

0

114

2