1 / 26

TDD 퀴즈 1 리뷰 - 비디오 대여 -

TDD 퀴즈 1 리뷰 - 비디오 대여 -. 백기선. 코드 리뷰 차례. Dazzilove( 류성희 ) Javajigi( 박재성 ) Kanisuka( 박호연 ) Keesun( 백기선 ) LeeKiWon( 이기원 ) Nije( 김재진 ) Sungyoon( 김성윤 ). Dazzilove( 류성희 ) - 비디오 대여 -. 프로젝트 체크아웃. JUnit 3 라이브러리 추가 해줘야 함 . 인코딩 MS949 테스트 코드와 제품 코드 구분 없음. 클래스 구조. rentItems. Customer.

aimee-cummings
Download Presentation

TDD 퀴즈 1 리뷰 - 비디오 대여 -

An Image/Link below is provided (as is) to download presentation Download Policy: Content on the Website is provided to you AS IS for your information and personal use and may not be sold / licensed / shared on other websites without getting consent from its author. Content is provided to you AS IS for your information and personal use only. Download presentation by click this link. While downloading, if for some reason you are not able to download a presentation, the publisher may have deleted the file from their server. During download, if you can't get a presentation, the file might be deleted by the publisher.

E N D

Presentation Transcript

  1. TDD 퀴즈 1 리뷰-비디오 대여- 백기선

  2. 코드 리뷰 차례 • Dazzilove(류성희) • Javajigi(박재성) • Kanisuka(박호연) • Keesun(백기선) • LeeKiWon(이기원) • Nije(김재진) • Sungyoon(김성윤)

  3. Dazzilove(류성희) -비디오 대여-

  4. 프로젝트 체크아웃 • JUnit 3 라이브러리 추가 해줘야 함. • 인코딩 MS949 • 테스트 코드와 제품 코드 구분 없음.

  5. 클래스 구조 rentItems Customer Video videos RentalReport

  6. 비디오에 아이디가 있다? 요구 사항에 없는 내용… 추가 구현인지미리 설계한 것인지… 왜 필요할까?

  7. 비디오가 비디오 목록을 가진다? 비디오 샵이 비디오를 가지고 있는 것이 자연스럽지 않을까?

  8. 보너스 포인트 속성 중복 • bonusPoint 속성 중복 • Video에도 있고 • VideoKindItem에도 있음

  9. 리팩터링 필요 Replace Conditional with Polymorphism Extract Method

  10. String 애착증 enum의 장점인 type-safety를 과감히 내던진 이유는?

  11. 테스트 코드 리팩터링 필요 그런데 테스트 내용이 조금… setVideoKind()에 로직이 들어가있음. Getter/Setter에 로직이 들어가는 것에 대해 어떻게 생각하세요?

  12. 테스트 커버리지 99.7%

  13. 옥의티

  14. Javajigi(박재성) -비디오 대여-

  15. 프로젝트 체크아웃 • 메이븐 프로젝트 • 인코딩 MS949 • 테스트와 제품 코드 구분 • 불필요한 의존성 존재 - JMock

  16. 클래스 구조 rentalVideos Customer RentalVideo VideoType Video rentalHistory Documentary Sports Movie

  17. 대여 행위는 고객이 주도한다. • 고객이 어떤 비디오를 얼마 동안 빌린다. “대여” rent()는 Customer의 책임일까? VideoShop의 책임일까?

  18. 대여 날짜가 필수? 대여 날짜를 비디오가 반드시 가지고 있어야 할까? 어떤 요구 사항에 의해 추가된 것일까? 추가 기능인가?

  19. 대여 상태 확인 대여 날짜, 대여 기간, 현재 날짜로 대여 상태를 확인할 수 있다. 요구 사항에 없는 추가 기능.

  20. Mocking 사용 RentalVideo를 단위 테스트 하기 위해서 Video Mock 객체 사용. Customer와 Calendar는?

  21. 테스트 코드 리팩터링 필요

  22. 비디오 마다 한 번씩 대여를 해야 하나? 요구 사항 고객은 한번에 여러 개의 비디오를 대여할 수 있으나 각각의 대여(Rental)기간은 다를 수 있다. => 대여 행위(rent)를 여러 번 하여 여러 개의 비디오를 대여 한다.

  23. 테스트 커버리지 87.3% 군대 군대 None-TDD 코드가 보임

  24. doortts(채수원) -비디오 대여-

  25. 클래스 구조 rentals Customer Rental Video AbstractVideoType MovieType DocumentaryType SportsType

  26. 테스트 커버리지 100%

More Related