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

추상화 #35

Closed
BBBOMi opened this issue Oct 16, 2020 · 0 comments · Fixed by #36
Closed

추상화 #35

BBBOMi opened this issue Oct 16, 2020 · 0 comments · Fixed by #36
Labels
step
Projects
1부 다중 통화 산술(Multi-Currency Arithmetic)

Comments

@BBBOMi
Copy link
Owner

BBBOMi commented Oct 16, 2020 •
edited
Loading

  1. 해결하고자 하는 목표
    • TO-DO 리스트에서 꼭 구현해야 하는 최소한의 작업 수행과 추상화

  1. 해당 이슈에서 한 작업
    • 다른 사람들을 염두에 둔 테스트를 작성
    • 한 번의 선언부에 대한 수정이 시스템 나머지 부분으로 번져갔고, 문제를 고치기 위해 컴파일러의 조언을 따랐음
    • 잠시 실험을 시도했다가 좋지 못해서 버렸음.
@BBBOMi BBBOMi added the step label Oct 16, 2020
@BBBOMi BBBOMi added this to In progress in 1부 다중 통화 산술(Multi-Currency Arithmetic) via automation Oct 16, 2020
BBBOMi added a commit that referenced this issue Oct 19, 2020
@BBBOMi
Test : Sum의 plus 메서드 테스트 추가 #35
0148bc4 
-  NullPointerException 발생(plus 메서드 stub 구현 때문)
- 테스트 시나리오가 Money의 코드와  중복
BBBOMi added a commit that referenced this issue Oct 19, 2020
@BBBOMi
Feat : plus 메서드 구현 #35
3ac5e6c 
- stub 구현에서 실제로 구현
- 테스트 통과 확인
BBBOMi added a commit that referenced this issue Oct 19, 2020
@BBBOMi
Test : Sum 에서의 times 메서드 테스트 #35
421433d 
- Expression으로 올리기 전에 Sum에서 먼저 작동하게 함
- 컴파일 에러 발생
BBBOMi added a commit that referenced this issue Oct 19, 2020
@BBBOMi
Feat : times 메서드 생성 #35
1adf929 
- 컴파일 에러 해결을 위해 생성
- 메서드 완성을 위해 Expression 클래스에 메서드 정의
- 기존에 Money 클래스에 있던 times 메서드의 가시성을 높임
- 테스트 검증 완료
BBBOMi added a commit that referenced this issue Oct 19, 2020
@BBBOMi
Test : $5 + $5 에서 Money를 반환하는 테스트 작성 #35
4998c6c 
- 테스트 실패
BBBOMi added a commit that referenced this issue Oct 19, 2020
@BBBOMi
Refactor : $5 + $5에서 Money 반환하는 케이스 제거 #35
eca825f 
- plus 메서드의 반환형을 Money로 변경하는 것보다 기존에 있는 reduce를 이용하여 Money로 변환하는 것이 더 좋다고 판단
- 테스트는 좋지 않은 테스트이므로 제거
@BBBOMi BBBOMi mentioned this issue Oct 19, 2020
Merged
@BBBOMi BBBOMi linked a pull request Oct 19, 2020 that will close this issue
Feat : 코드 추상화 #36
Merged
@BBBOMi BBBOMi mentioned this issue Oct 19, 2020
Open
23 tasks
@BBBOMi BBBOMi closed this as completed Oct 19, 2020
1부 다중 통화 산술(Multi-Currency Arithmetic) automation moved this from In progress to Done Oct 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
step
Projects
1부 다중 통화 산술(Multi-Currency Arithmetic)
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feat : 코드 추상화 BBBOMi/TDD-practice
1 participant
@BBBOMi