Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: code-squad/java-bowling
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
...
head repository: code-squad/java-bowling
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: david-learner
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 4 commits
  • 34 files changed
  • 1 contributor

Commits on Apr 18, 2018

  1. Step1 (#73)

    * Feat : 스텝1 요구사항 및 테스트코드 추가
    
    * Feat : 스텝1 기능 추가
    
    * Refactor : 자동 단위테스트 수정
    
    * REMOVE ALL - I wanna re-construct
    
    * Feat : 스텝1 기능 구현
    
    * Refactor : 모든 파일 초기화를 위한 백업
    
    * commit for commit
    
    * commit for commit
    
    * Feat : 프레임 상태 업데이트 로직 추가
    
    * Feat : 프레임 상태에 따른 종료 판단 로직 추가
    
    * Refactor : Pins에 print 출력 추가 및 State에 print 로직 제거
    
    * Feat : Ready 상태 추가
    State 자식 클래스들은 각각의 조건에 따라 Pins를 가진다.
    Ready와 NotFinish와의 차이는 멤버변수 Pins의 유무다.
    
    * Feat : 마지막 프레임 결과 판단 로직 추가
    State 자식 클래스들 step2의 힌트를 토대로 구현
    현재 state 저장을 위한 Cloneable 구현
    
    * Refactor : 마지막 프레임 종료 로직 개선
    
    * Refactor : 테스트 케이스 추가
    컨벤션 수정 및 불필요한 클래스(코드) 제거
    david-learner authored and javajigi committed Apr 18, 2018
    Configuration menu
    Copy the full SHA
    fb02472 View commit details
    Browse the repository at this point in the history

Commits on Apr 25, 2018

  1. Step2 (#91)

    * Feat : Score클래스 추가
    
    * Feat : BowlingGame 추가
    
    * Feat : Normal Frame Score 계산 로직 추가
    
    * Feat : 마지막 프레임 스코어 계산 로직 추가
    
    * Refactor : 불필요한 주석 및 코드 제거, 테스트 추가
    
    * Refactor : 테스트 추가
    david-learner authored and javajigi committed Apr 25, 2018
    Configuration menu
    Copy the full SHA
    1ffeba1 View commit details
    Browse the repository at this point in the history

Commits on Apr 26, 2018

  1. Step3 - 포비코드 분석 전 Step3 (#92)

    * Feat : Score클래스 추가
    
    * Feat : BowlingGame 추가
    
    * Feat : Normal Frame Score 계산 로직 추가
    
    * Feat : 마지막 프레임 스코어 계산 로직 추가
    
    * Refactor : 불필요한 주석 및 코드 제거, 테스트 추가
    
    * Refactor : 테스트 추가
    
    * Feat : can use multi users
    
    * Refactor : 테스트 오류 수정
    david-learner authored and javajigi committed Apr 26, 2018
    Configuration menu
    Copy the full SHA
    f7896f8 View commit details
    Browse the repository at this point in the history
  2. Step3 - 포비코드 분석 전 step3 (#94)

    * Feat : Score클래스 추가
    
    * Feat : BowlingGame 추가
    
    * Feat : Normal Frame Score 계산 로직 추가
    
    * Feat : 마지막 프레임 스코어 계산 로직 추가
    
    * Refactor : 불필요한 주석 및 코드 제거, 테스트 추가
    
    * Refactor : 테스트 추가
    
    * Feat : can use multi users
    
    * Refactor : 테스트 오류 수정
    
    * Refactor : 테스트 수정 및 추가
    
    * Refactor : 마지막 프레임 추가점수 버그 수정
    마지막 프레임 상태값에 따라 추가점수를 계산하는 로직에서 states를 모두 반복하여 계산하는데, 이는 NormalFrame에서는 괜찮지만 LastFrame에서 createScore()에서 이미 첫 번째 state를 가지고 계산하기 때문에 첫 번째 state를 중복계산하는 버그가 발생한다.
    
    * Refactor : 테스트 추가 및 수정
    
    * Refactor : 불필요한 테스트 제거 및 컨벤션 수정
    david-learner authored and javajigi committed Apr 26, 2018
    Configuration menu
    Copy the full SHA
    b7a8bc0 View commit details
    Browse the repository at this point in the history
Loading