-
Notifications
You must be signed in to change notification settings - Fork 300
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[JDBC 라이브러리 구현하기 - 1단계] 홍실(홍혁준) 미션 제출합니다. (#270)
* 패키지 위치 변경 및 코드 정리 * 패키지 위치 변경 및 코드 정리 * 패키지 위치 변경 및 코드 정리 * feat: user findAll 구현 * feat: user update 구현 * feat: user findByAccount 구현 * refactor: try with resources로 리팩터링 * chore: JdbcTemplate 테스트를 위한 세팅 * feat: jdbcTemplate에서 쿼리를 실행시키는 메서드 추가 * feat: jdbcTemplate에서 조회성 쿼리를 실행하는 기능 추가 * refactor: 쓰기 쿼리 부분 jdbcTemplate으로 리팩터링 * fix: 조회하는 부분에서, rs.next 추가 * feat: userDao의 조회쿼리 부분 JdbcTemplate으로 리팩터링 --------- Co-authored-by: kang-hyungu <[email protected]>
- Loading branch information
1 parent
2832af7
commit cd74a44
Showing
116 changed files
with
3,182 additions
and
253 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,37 @@ | ||
| name: SonarCloud | ||
| on: | ||
| push: | ||
| branches: | ||
| - main | ||
| pull_request: | ||
| types: [opened, synchronize, reopened] | ||
| jobs: | ||
| build: | ||
| name: Build and analyze | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v3 | ||
| with: | ||
| fetch-depth: 0 # Shallow clones should be disabled for a better relevancy of analysis | ||
| - name: Set up JDK 11 | ||
| uses: actions/setup-java@v3 | ||
| with: | ||
| java-version: 11 | ||
| distribution: 'corretto' # Alternative distribution options are available | ||
| - name: Cache SonarCloud packages | ||
| uses: actions/cache@v3 | ||
| with: | ||
| path: ~/.sonar/cache | ||
| key: ${{ runner.os }}-sonar | ||
| restore-keys: ${{ runner.os }}-sonar | ||
| - name: Cache Gradle packages | ||
| uses: actions/cache@v3 | ||
| with: | ||
| path: ~/.gradle/caches | ||
| key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} | ||
| restore-keys: ${{ runner.os }}-gradle | ||
| - name: Build and analyze | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Needed to get PR information, if any | ||
| SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }} | ||
| run: ./gradlew clean build codeCoverageReport sonar --info -x :study:build |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -171,3 +171,5 @@ Temporary Items | |
|
|
||
| tomcat.* | ||
| tomcat.*/** | ||
|
|
||
| **/WEB-INF/classes/** | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1,17 @@ | ||
| # JDBC 라이브러리 구현하기 | ||
|
|
||
| - [x] UserDaoTest 통과시키기 | ||
| - [x] user findAll 구현하기 | ||
| - [x] user update 구현하기 | ||
| - [x] user findByAccount 구현하기 | ||
| - [x] UserDao 리팩터링 | ||
| - [x] 공통부분 분리 | ||
| - [ ] JdbcTemplate 구현하기 | ||
| - [x] 조회기능 구현 | ||
| - [x] 쓰기기능 구현 | ||
| - [x] UserDao를 JdbcTemplate을 사용하도록 리팩터링 | ||
| - [x] findByAccount | ||
| - [x] findById | ||
| - [x] findAll | ||
| - [x] update | ||
| - [x] insert |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
10 changes: 5 additions & 5 deletions
10
app/src/main/java/com/techcourse/AppWebApplicationInitializer.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,121 +1,66 @@ | ||
| package com.techcourse.dao; | ||
|
|
||
| import com.techcourse.domain.User; | ||
| import nextstep.jdbc.JdbcTemplate; | ||
| import java.util.ArrayList; | ||
| import java.util.List; | ||
| import javax.sql.DataSource; | ||
| import org.slf4j.Logger; | ||
| import org.slf4j.LoggerFactory; | ||
|
|
||
| import javax.sql.DataSource; | ||
| import java.sql.Connection; | ||
| import java.sql.PreparedStatement; | ||
| import java.sql.ResultSet; | ||
| import java.sql.SQLException; | ||
| import java.util.List; | ||
| import org.springframework.jdbc.core.JdbcTemplate; | ||
| import org.springframework.jdbc.core.Mapper; | ||
|
|
||
| public class UserDao { | ||
|
|
||
| private static final Mapper<User> USER_MAPPER = (rs) -> new User( | ||
| rs.getLong(1), | ||
| rs.getString(2), | ||
| rs.getString(3), | ||
| rs.getString(4) | ||
| ); | ||
| private static final Logger log = LoggerFactory.getLogger(UserDao.class); | ||
|
|
||
| private final DataSource dataSource; | ||
| private final JdbcTemplate jdbcTemplate; | ||
|
|
||
| public UserDao(final DataSource dataSource) { | ||
| this.dataSource = dataSource; | ||
| this.jdbcTemplate = new JdbcTemplate(dataSource); | ||
| } | ||
|
|
||
| public UserDao(final JdbcTemplate jdbcTemplate) { | ||
| this.dataSource = null; | ||
| this.jdbcTemplate = jdbcTemplate; | ||
| } | ||
|
|
||
| public void insert(final User user) { | ||
| final var sql = "insert into users (account, password, email) values (?, ?, ?)"; | ||
|
|
||
| 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) {} | ||
| } | ||
| jdbcTemplate.executeUpdate(sql, | ||
| user.getAccount(), user.getPassword(), user.getEmail()); | ||
| } | ||
|
|
||
| public void update(final User user) { | ||
| // todo | ||
| final var sql = "update users set account = ?, password = ? , email = ? where id = ?"; | ||
| jdbcTemplate.executeUpdate(sql, | ||
| user.getAccount(), user.getPassword(), user.getEmail(), user.getId()); | ||
| } | ||
|
|
||
| public List<User> findAll() { | ||
| // todo | ||
| return null; | ||
| final var sql = "select id, account, password, email from users"; | ||
| return jdbcTemplate.executeQuery(sql, (rs) -> { | ||
| final List<User> users = new ArrayList<>(); | ||
| while (rs.next()) { | ||
| users.add(USER_MAPPER.map(rs)); | ||
| } | ||
| return users; | ||
| }); | ||
| } | ||
|
|
||
| public User findById(final Long id) { | ||
| final var sql = "select id, account, password, email from users where id = ?"; | ||
|
|
||
| Connection conn = null; | ||
| PreparedStatement pstmt = null; | ||
| ResultSet rs = null; | ||
| try { | ||
| conn = dataSource.getConnection(); | ||
| pstmt = conn.prepareStatement(sql); | ||
| pstmt.setLong(1, id); | ||
| rs = pstmt.executeQuery(); | ||
|
|
||
| log.debug("query : {}", sql); | ||
|
|
||
| if (rs.next()) { | ||
| return new User( | ||
| rs.getLong(1), | ||
| rs.getString(2), | ||
| rs.getString(3), | ||
| rs.getString(4)); | ||
| } | ||
| return null; | ||
| } catch (SQLException e) { | ||
| log.error(e.getMessage(), e); | ||
| throw new RuntimeException(e); | ||
| } finally { | ||
| try { | ||
| if (rs != null) { | ||
| rs.close(); | ||
| } | ||
| } catch (SQLException ignored) {} | ||
|
|
||
| try { | ||
| if (pstmt != null) { | ||
| pstmt.close(); | ||
| } | ||
| } catch (SQLException ignored) {} | ||
|
|
||
| try { | ||
| if (conn != null) { | ||
| conn.close(); | ||
| } | ||
| } catch (SQLException ignored) {} | ||
| } | ||
| return jdbcTemplate.executeQuery(sql, USER_MAPPER, id); | ||
| } | ||
|
|
||
| public User findByAccount(final String account) { | ||
| // todo | ||
| return null; | ||
| final var sql = "select id, account, password, email from users where account = ?"; | ||
|
|
||
| return jdbcTemplate.executeQuery(sql, USER_MAPPER, account); | ||
| } | ||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.