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 라이브러리 구현하기 - 1,2단계] 준팍(박준현) 미션 제출합니다. #263

Merged
merged 7 commits into from
Sep 27, 2023

Conversation

junpakPark
Copy link
Member

존경하옵는 에밀 선생님께,

안녕하십니까?

저희가 함께하였던 레벨1 방학이
어느덧 179일이 지나간 것을 깨닫게 되었습니다.
시간이 참 빠르게 지나가는 것 같아 마음이 헛헛합니다.

솔직히 말씀드리면,
선생님께 저의 진전된 모습을 제대로 보여드리고자 하는 마음은 크나,
아직도 부족한 실력에 대한 걱정이 앞섭니다.

부족한 모습마저 사랑으로 감싸주시길 간청드립니다.

현재의 작업 순서는 다음과 같습니다:

  • UserDao 내부의 모든 메서드의 구현
  • DAO 내에서의 update 메서드 분리
  • queryForObject 메서드의 분리
  • query 메서드의 분리
  • 분리한 메서드들을 JdbcTemplate으로 이전
    • 아뢰옵기 황송하오나, 커밋 이름에는 약간의 착오가 있음을 양해 부탁드립니다...
  • JdbcTemplate에 대한 테스트 케이스 추가

추후에는 DB ConnectionPool에 대한 학습 테스트도 추가할 예정입니다.

존경하는 에밀 선생님의 세심한 지도와 가르침을 바라며,
계속해서 저에게 힘과 도움을 주시기를 희망합니다.

감사의 말씀을 전하며, 아름다운 하루 되시길 바랍니다.

Copy link
Member

@CFalws CFalws left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오우 준팍 스프링 jdbc를 구현하셨군요 ~ 굿입니다. 딱히 수정할 점이 보이지 않아 바로 어프루브 하도록 하겠습니다~!

} catch (SQLException ignored) {}
}
return jdbcTemplate.queryForObject(sql, USER_ROW_MAPPER, id)
.orElseThrow(DataAccessException::new);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

굿입니다~

public interface RowMapper<T> {

@Nullable
T mapRow(ResultSet resultSet, int rowNumber) throws SQLException;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

현재는 rowNumber가 필요없어 보이는데 이렇게 구현하신 이유는 무엇인가요?

@CFalws CFalws merged commit af46958 into woowacourse:junpakpark Sep 27, 2023
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants