Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
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: next-step/java-bowling
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
Loading
...
head repository: next-step/java-bowling
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: gmelon
Choose a head ref
Loading
Checking mergeability… Don’t worry, you can still create the pull request.
  • 3 commits
  • 44 files changed
  • 2 contributors

Commits on Nov 17, 2022

  1. 🚀 1단계 - 질문 삭제하기 기능 리팩토링 (#988)

    * docs: 요구사항 정리
    
    * refactor: QnAService 의 '게시글 작성자가 본인인지 검증'로직을 Question으로 이동
    
    * refactor: Question 의 List<Answer>을 일급 콜렉션 Answers로 변환, Answer 검증 로직 이전
    
    * refactor: Answers 의 검증 로직을 Question의 검증 로직에 병합
    
    * refactor: DeleteHistory의 일급 콜렉션 DeleteHistories를 생성
    
    * docs: DeleteHistories를 가변객체로 변경함에 따른 요구사항 변경
    
    * refactor: QnAService 의 Answer을 삭제하는 로직을 Answers로 이동
    
    * refactor: QnAService의 Question을 삭제하는 로직을 Question으로 이동
    
    * refactor: Question의 delete() 로직에서 위 로직을 모두 수행하고, 질문과 답글이 모두 포함된 DeleteHistories를 반환하도록 변경
    
    * test: 댓글 삭제 메소드 테스트 케이스 추가
    
    * test: 테스트 순서에 따라 결과가 달라지는 문제 수정
    
    * refactor: DeleteHistories를 불변객체로 변경
    
    * refactor: DeleteHistory 생성 시 사용되는 LocalDateTime를 서비스 레이어에서 주입하도록 변경
    
    * test: delete() 메서드에서 반환하는 DeleteHistories가 올바른 상태의 DeleteHistory 객체를 포함하고 있는지 확인하도록 수정
    gmelon authored Nov 17, 2022
    Configuration menu
    Copy the full SHA
    980b330 View commit details
    Browse the repository at this point in the history

Commits on Jan 2, 2023

  1. 2단계 - 볼링 점수판(그리기) (#1006)

    * docs: 요구사항 문서 작성
    
    * docs: 요구사항 문서 작성
    
    * feat: Pins 구현
    
    * feat: Status 상속 구조 구현
    
    * test: FinishedTest 클래스를 각 구현 클래스 별로 분리
    
    * feat: state 및 Pin 클래스에서 결과 문자열을 가지고 제공하도록 구현
    
    * feat: Frame 인터페이스 정의
    
    * feat: Frame을 구현하는 NormalFrame 구현
    
    * feat: Frame의 일급콜렉션 Frames 구현
    
    * feat: FinalFrame 구현
    
    * feat: UI 클래스, Controller, FrameResultsDto 구현
    
    * feat: Player 클래스 구현
    
    * feat: Player까지 한번에 출력하도록 FrameResultsDto, InputView, Controller 수정
    
    * test: Frames 테스트 케이스 구현
    
    * refactor: NormalFrame init() 정적 팩토리 메소드 추가, 인스턴스 변수가 없는 Status 클래스에 equals(), hashcode() 구현
    
    * refactor: FinalFrame init() 정적 팩토리 메소드 추가
    
    * test: Player 정상 생성 테스트 케이스 추가
    
    * refactor: ResultDto를 ResultLines로 변경, OutputView에서 생성하던 결과 리스트를 ResultLines에서 생성하도록 변경
    gmelon authored Jan 2, 2023
    Configuration menu
    Copy the full SHA
    a210e73 View commit details
    Browse the repository at this point in the history

Commits on Jan 16, 2023

  1. 3단계 - 볼링 점수판(점수 계산) (#1007)

    * docs: 점수 표시 요구사항 작성
    
    * refactor: NormalFrame과 FinalFrame의 공통 로직을 AbstractFrame으로 추출
    
    * feat: Score 클래스 구현
    
    * refactor: Score 클래스 불변객체로 변경
    
    * feat: State 클래스들에 score(), calculateScore() 메소드 구현
    
    * fix: 한 Frame에서 10개 이상의 볼링핀을 쓰러트릴 수 있던 문제 수정
    
    * feat: Frame에 현재 점수 계산 메소드, 이전 프레임에서 위임된 점수 계산 메소드 추가
    
    * feat: 점수를 출력하도록 OutputView와 ResultLines 로직 수정
    gmelon authored Jan 16, 2023
    Configuration menu
    Copy the full SHA
    2b43ed5 View commit details
    Browse the repository at this point in the history
Loading
Morty Proxy This is a proxified and sanitized view of the page, visit original site.