-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Codrae #1892
Open
codrae
wants to merge
12
commits into
woowacourse-precourse:main
Choose a base branch
from
codrae:codrae
base: main
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
Codrae #1892
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
레포지토리의 내용 및 구현하여야 할 목적을 알아볼 수 있도록 간단한 README 파일을 작성 함.
Application class에 대한 주석과 사용자로부터 문자열을 입력받기 위한 코드 작성.
기본 및 커스텀 구분자를 위한 변수 선언, 입력과정에서 발생할 수 있는 예외들에 대한 처리를 하였음.
숫자들을 인식해서 덧셈 결과를 반환하는 함수를 구현하였으며, 유효성 검증부분에 빈 문자열이 입력되는 경웨 대한 에러를 추가로 처리하였음.
이전 기능정리에 너무 세부적으로 작성하여, 구현한 메서드와 크기가 맞지 않기에, README.md 파일을 그에 맞추어 수정하였음.
불필요한 전역변수 제거 및 삼항연산자와 startsWith함수 사용을 통한 코드 간결화. 또한 입출력 포맷을 문제의 예시와 동일하게 통일.
UTF-8 설정 관련 한글 깨짐 문제로 인해서 영어로 입출력 문자열 변경.
error를 try catch로 잡는것이 아니라 정상적으로 error를 날릴 수 있도록 수정.
'-'가 포함되어있으면 음수를 포함한것으로 간주, 양수만을 포함한 입력이기에 예외처리 해주었다.
magic number 사용을 지양하고, customSep에 - 및 숫자가 허용되도록 코드를 개선했다.
Test Code에서 logger가 아닌 System.out.println으로 출력하기를 요구하고 있기에 수정함.
1이 아니라 키보드로 입력하지 않을 \0 char을 사용하여 커스텀 구분자를 초기화하고, camp.nextstep.edu.missionutils.Console에 대해 세부 코드 확인 후 Console.close()코드 추가 작성하였음.
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.
No description provided.