generated from gonza-st/playground-template
-
Notifications
You must be signed in to change notification settings - Fork 0
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
[경재] 숫자야구 구현 #1
Open
gyeongjae-ham
wants to merge
18
commits into
gyeongjae
Choose a base branch
from
gyeongjae-feature
base: gyeongjae
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[경재] 숫자야구 구현 #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gyeongjae-ham
commented
Dec 10, 2024
- 요구사항 정리
- 숫자 야구 숫자 객체 생성 및 검증
- 숫자 야구 게임 객체 생성 및 검증
- 입,출력 구현
- 게임 실행부 작성
요구사항 정리
숫자야구 숫자를 검증하는 BaseballGameNumber를 생성하고 검증한다
안쓰는 객체를 이용하는 테스트 제거
숫자 야구 숫자를 생성하는 객체 생성 및 검증
게임을 초기화하고 진행하는 객체 생성 및 검증
입력 객체 생성
출력 객체 생성
게임 진행하는 main class 추가
hongxeob
requested changes
Dec 15, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다 경재님~ 몇가지 코멘트 남겨드렸습니다!
- 현재는 패키지 구조없이 설정되어 있는데 패키지 구조까지 고민해 보시면 좋을 거 같습니다!
java-playground/src/main/java/org/gonza/javaplayground/OutputView.java
Outdated
Show resolved
Hide resolved
java-playground/src/main/java/org/gonza/javaplayground/OutputView.java
Outdated
Show resolved
Hide resolved
java-playground/src/main/java/org/gonza/javaplayground/OutputView.java
Outdated
Show resolved
Hide resolved
java-playground/src/test/java/org/gonza/javaplayground/FixNumberGenerator.java
Outdated
Show resolved
Hide resolved
java-playground/src/test/java/org/gonza/javaplayground/BaseballGameTest.java
Outdated
Show resolved
Hide resolved
java-playground/src/main/java/org/gonza/javaplayground/BaseballGameNumber.java
Show resolved
Hide resolved
java-playground/src/main/java/org/gonza/javaplayground/InputView.java
Outdated
Show resolved
Hide resolved
java-playground/src/main/java/org/gonza/javaplayground/BaseballGameNumber.java
Outdated
Show resolved
Hide resolved
java-playground/src/main/java/org/gonza/javaplayground/BaseballGameNumber.java
Outdated
Show resolved
Hide resolved
반복되는 생성문 method화 시키기
입력과 출력 구분해서 작성
생성자 부분 fixture로 대체
method 명 좀 더 명확하게 수정
아웃 조건 추가
Heonbyeong
reviewed
Dec 18, 2024
java-playground/src/main/java/org/gonza/javaplayground/BaseballGameNumber.java
Outdated
Show resolved
Hide resolved
currenjin
reviewed
Dec 18, 2024
java-playground/src/main/java/org/gonza/javaplayground/BaseballGameNumber.java
Outdated
Show resolved
Hide resolved
java-playground/src/main/java/org/gonza/javaplayground/RandomNumberGenerator.java
Outdated
Show resolved
Hide resolved
java-playground/src/main/java/org/gonza/javaplayground/JavaPlaygroundApplication.java
Show resolved
Hide resolved
Heonbyeong
reviewed
Dec 18, 2024
java-playground/src/main/java/org/gonza/javaplayground/JavaPlaygroundApplication.java
Outdated
Show resolved
Hide resolved
Heonbyeong
reviewed
Dec 18, 2024
java-playground/src/main/java/org/gonza/javaplayground/JavaPlaygroundApplication.java
Show resolved
Hide resolved
Heonbyeong
reviewed
Dec 18, 2024
Heonbyeong
reviewed
Dec 18, 2024
java-playground/src/main/java/org/gonza/javaplayground/RandomNumberGenerator.java
Outdated
Show resolved
Hide resolved
병헌님 & 현진님 리뷰 반영
leeMK09
reviewed
Dec 19, 2024
java-playground/src/main/java/org/gonza/javaplayground/BaseballGameNumber.java
Show resolved
Hide resolved
사용 객체에서 자릿수를 전달하도록 수정
testcode에 반영
사용 객체에서 자릿수 넣어주기
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.