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: imjinbro/java-bowling
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: master
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 3 commits
  • 33 files changed
  • 1 contributor

Commits on Apr 9, 2018

  1. Step1 (#67)

    * step#1 - 기본 구조 생성
    
    * step#1 - 플레이어 이름 받기 뷰, 로직 구현
    
    * step#1 - 점수 얻어오기 구현
    
    * step#1 - 점수 기록하기 구현(기준, 오버헤드예외처리)
    
    * step#1 - 점수 기록 기준, 예외처리 보강
    
    * step#1 - 점수 설정에 따른 프레임 상태 변화 구현(코드 정리는 아직 안함)
    
    * step#1 - 프레임 넘버 상태에 따라 증가하도록 구현
    
    * step#1 - 프레임 구분 - 노말/라스트 각각(1차)
    
    * step#1 - 콘솔뷰 메세지 출력, 상태에 따라 메세지 구성 기능 구현, 투구 끝날 때마다 프레임 결과 생성되고 출력하도록 구현, 결과 테스트 구현
    
    * step#1 - 던질 때마다 결과 저장되도록 구현
    
    * step#1 - 결과 저장 Map -> List 변경
    
    * step#1 - 프레임 결과 출력 구현
    
    * step#1 - 컨트롤러, 결과 출력 수정
    
    * step#1 - 불필요한 주석 제거
    
    * step#1 - Frames Player 관계설정, 상태에 따라 isBonus, isFinish 평가 달리하도록 수정(instanceof 대체), Miss ScoreMessage 누락부분 수정, status에 요청해서 변경된 status 받도록 수정
    
    * step#1 - 뷰 코드 간결화
    imjinbro authored and javajigi committed Apr 9, 2018
    Configuration menu
    Copy the full SHA
    2ca64d9 View commit details
    Browse the repository at this point in the history

Commits on Apr 13, 2018

  1. Step2 (#71)

    * 머지 충돌 해결
    
    * step#2 - 상태값에 따라 보너스 카운트 각각 주어지게 구현
    
    * step#2 - 다음 기능 구현에 필요한 객체와 역할 확실히 분리하기위해서 명칭 변경(pin, frameResult)
    
    * step#2 - 노말프레임 보너스 점수 기록 구현(메세지 저장까지)
    
    * step#2 - 노말/라스트프레임 중복제거 시도
    
    * step#2 - 각 프레임 토탈 점수 기능 구현을 위한 로직 구현
    
    * step#2 - 내부적으로 보너스점수 기록되도록 변경, 토탈 점수 계산 기능 구현
    
    * step#2 - 노말, 라스트프레임 정규 투구, 보너스 투구 메소드 이름 변경, Frames 점수 추가 기능 구현
    
    * step#2 - 출력 기능 구현
    
    * step#2 - 점수 기록 기준과 끝나는 기준 분리
    
    * step#2 - 라스트 프레임 점수 기록 표시 잘못된 조건 고침
    
    * step#2 - 노말/라스트 프레임 각각 역할 나눔
    
    * step#2 - 노말/라스트 프레임 각각 역할 + FrameScore 테스트 코드 뒤늦게 작성해서 확인, 코드 수정
    
    * step#2 - 노말 / 라스트 프레임 테스트 및 코드 수정, 불필요한 코드 삭제
    
    * step#2 - 잘못된 기준 메소드 로직 변경
    
    * step#2 - 프레임 상황에 맞게 점수판 메세지 출력되도록 변경
    
    * step# - 정규투구 예외처리(합 10개)
    
    * step# - 정규투구 예외처리(합 10개)
    imjinbro authored and javajigi committed Apr 13, 2018
    Configuration menu
    Copy the full SHA
    6072587 View commit details
    Browse the repository at this point in the history

Commits on Apr 23, 2018

  1. Step3 (#81)

    * step#2 additional ver - 패키지 분배
    
    * step#2 additional ver - 프레임 간의 연결
    
    * step#2 additional ver - 프레임 간의 연결 후 결과 가져오기(Ready 상태 추가)
    
    * step#2 additional ver - 노말 / 라스트 프레임 점수 출력 메세지 다른 것 구현
    
    * step#2 additional ver - 보너스 점수 할당 구현
    
    * step#2 additional ver - 연결된 프레임 합산 점수 기능, 점수판 출력 기능 구현
    
    * step#2 additional ver - 합산 기능 테스트 코드 수정
    
    * step#2 additional ver - 보너스 점수 연결리스트 구조 활용 리팩토링
    
    * step#2 additional ver - 보너스 점수 연결리스트 구조 활용 리팩토링(2)
    
    * step#2 additional ver - 라스트 프레임 보너스 핀 저장소 추상화
    
    * step#2 additional ver - 라스트 프레임 보너스 메세지 오류 수정
    
    * step#2 additional ver - 테스트 코드 약간 수정
    
    * step#3 - 플레이어 순차적으로 돌아가도록 구현
    
    * step#3 - 플레이어들 투구 후 점수판 출력되도록 구현
    
    * step#3 - 출력 오류 수정
    
    * step#3 - 프레임 넘버 변수 private 수정
    
    * step#3 - 보너스 횟수 내부에서 처리하도록 수정
    
    * step#3 - 프레임 상태, 노말 프레임 리팩토링
    
    * step#3 - 상태 생성 전 핀 생성(예외잡기)
    
    * step#3 - 프레임, 노말, 라스트 점수 구하기 리팩토링
    
    * step#3 - 점수판 출력(프레임 누적) 리팩토링
    
    * step#3 - 라스트 프레임 상태 구현
    
    * step#3 - 프레임 점수 가져오기 예외 처리(10점 세번 연속일 때)
    
    * step#3 - isFinish 사용하지않고 있던 것 사용하도록 수정(createFrame시 불필요한 status 전달 부분)
    
    * step#3 - 혼란스러운 메소드 명칭 변경(남은 핀 횟수 더하기 : score -> pin), Frame 전달 없이 상태 설정 시 구분하기(인스턴스말고 플래그만 전달 - 시도1)
    
    * step#3 - 노말 프레임, 라스트 프레임 상태 관리 각각 다르게
    
    * step#3 - goodbye bowling...! 최고의 공부였습니다 있는 조건에서 문제 해결하기(하나의 문제를 해결하려고 새로운 것부터 만들려고 하지말자를 배웠습니다)
    imjinbro authored and javajigi committed Apr 23, 2018
    Configuration menu
    Copy the full SHA
    eab9d3e View commit details
    Browse the repository at this point in the history
Loading