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.

Sign up for GitHub

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

Jump to bottom

[4기 - 한승원] 1~2주차 과제 : 계산기 구현 미션 제출합니다(테스트코드) #193

Open
wants to merge 47 commits into
base: seungwon-test
Choose a base branch
from
Open

[4기 - 한승원] 1~2주차 과제 : 계산기 구현 미션 제출합니다(테스트코드) #193

wants to merge 47 commits into from

Conversation

SW-H
Copy link

@SW-H SW-H commented Jun 15, 2023

📌 과제 설명

사칙연산에 대한 테스트 코드를 작성했습니다.

✅ PR 포인트 & 궁금한 점

계산기 기능(사칙연산)에 대한 코드도 포함되었는데
src/test/java/CalculatorTest.java 파일 중심으로 확인부탁드립니다🤩

SW-H added 30 commits June 9, 2023 13:26
@SW-H
init : 프로젝트 생성
42c4919
@SW-H
feat : 입출력 인터페이스
2f8bb46
@SW-H
feat : 예외 처리 문구
5a6deed
@SW-H
feat : 계산기 - main문/계산
c4c3e88
@SW-H
feat : 입출력 인터페이스 구현
e2dcb42 
- 입력 : 사용자 메뉴 선택, 수식
- 출력 : 계산 결과
@SW-H
feat : 연산자 enum으로 관리
bd3aee3
@SW-H
feat : 사용자 메뉴 enum
52a1300
@SW-H
docs : java버전 명시
ee4d032
@SW-H
docs : 미사용중인 종속 삭제
842467d
@SW-H
refactor : 코드 중복 제거
c9c80e0
@SW-H
refactor : 메인 함수를 포함한 클래스 추가
25a513e
@SW-H
refactor : 불필요한 개행 삭제
6d94091
@SW-H
feat : 수식
4d764ca
@SW-H
feat : 삭제
882c684
@SW-H
feat : input/outputView 주입, 선택 메뉴별 수행 메소드를 HashMap으로 분리
1a150a5
@SW-H
feat : 출력 관련 내용 CalculatorInputView -> CalculatorOutputView 이동
a3901a5
@SW-H
refactor
451742e
@SW-H
feat : 수식 입력시 Expression 클래스 반환
775466d
@SW-H
feat : 불필요한 문법요소 제거, 연산자 우선순위 비교 메소드(hasLowerPrecedence) 추가
5e41f40
@SW-H
feat : 계산 결과 출력을 위해 인자->Integer
2590e8c
@SW-H
feat : 불필요한 Optional 삭제, 메뉴-Integer 대응하도록 수정
6b66bc0
@SW-H
feat : main 함수 별도 분리
2f88ba9
@SW-H
feat : 계산 인자 -> Double 형으로 변환(나눗셈)
f154103
@SW-H
feat : 함수 매핑 HashMap 삭제
ddc919c
@SW-H
feat : CalculatorInputView 내에서 출력 함수 호출 삭제
9b34581
@SW-H
feat : 계산 결과 Integer -> Double(나눗셈)
9b943b9
@SW-H
feat : 문자열 자르기
c6c5a68
@SW-H
feat : 계산 인자 Integer -> Double (나눗셈)
6d43a13
@SW-H
feat : 계산 인자 Integer -> Double (나눗셈)
97b393a
@SW-H
feat : main 함수
84540a2
SW-H added 17 commits June 15, 2023 20:18
@SW-H
feat : 계산 로직 수정(후위연산식 변환없이 연산자 우선 순위에 따라 계산)
eeb3796
@SW-H
feat : 수식 내의 공백 제거, split
c37cdad
@SW-H
refactor : list 타입 ArrayList -> List
50cabb1
@SW-H
feat : 연산자 우선 순위 계산
0ac007e
@SW-H
refactor : 미사용 메소드 삭제
8b4ec3a
@SW-H
feat : 없는 메뉴선택시 처리 추가
a3544d8
@SW-H
feat : 메시지 출력 메소드
ea158fe
@SW-H
feat : 메시지 출력 메소드
6f1519d
@SW-H
feat : 없는 메뉴 선택시 처리 방법 수정
7fe46d5
@SW-H
refactor
aa52739
@SW-H
feat : 계산 로직 버그 수정
202ce5d
@SW-H
feat : 입력된 수식 null인 경우 처리
716fb12
@SW-H
feat : 입력된 수식 null인 경우 처리
c2351bd
@SW-H
feat : 입력된 수식이 빈 문자열인 경우 처리
1f42c10
@SW-H
feat : 테스트 작성
31a9d73
@SW-H
feat : 테스트 작성
ae06f38
@SW-H
feat : 테스트를 위한 종속 추가
00cbfc5
@SW-H SW-H self-assigned this Jun 15, 2023
@SW-H SW-H mentioned this pull request Jun 15, 2023
Open
8 tasks
ICCHOI
ICCHOI reviewed Jun 19, 2023
View reviewed changes
Copy link

@ICCHOI ICCHOI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

안녕하세요 승원님!
테스트 코드 잘 짜주신 것 같아요.
그런데 여러 클래스의 테스트 코드가 한 곳에 모여있어서, 클래스 별 테스트 파일을 만든 후 단위 테스트를 작성한 후, (지금과 같은 스타일인) 통합 테스트를 작성해보시는 건 어떨까요?
추가적으로 given - when -then과 같은 패턴을 사용해서 작성하면 좀 더 가독성이 좋을 것 같아요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ICCHOI ICCHOI ICCHOI left review comments

@devcourse-ray devcourse-ray Awaiting requested review from devcourse-ray

Assignees

@SW-H SW-H

Labels
Team-Ray 🔥 매운맛
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@SW-H @ICCHOI