Skip to content
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
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Codrae #1892

wants to merge 12 commits into from

Conversation

codrae
Copy link

@codrae codrae commented Oct 21, 2024

No description provided.

레포지토리의 내용 및 구현하여야 할 목적을 알아볼 수 있도록 간단한 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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant