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

$5 * 2 = $10 #2

Closed
BBBOMi opened this issue Jun 23, 2020 · 1 comment
Closed

$5 * 2 = $10 #2

BBBOMi opened this issue Jun 23, 2020 · 1 comment
Labels
step

Comments

@BBBOMi
Copy link
Owner

BBBOMi commented Jun 23, 2020 •
edited
Loading

  1. 해결하고자 하는 목표
    • 어떤 금액(주가)을 어떤 수(주식의 수)에 곱한 금액을 결과로 얻을 수 있어야 한다.

  1. 해당 이슈에서 한 작업
    • 우리가 알고 있는 작업해야 할 TO-DO 목록을 만들었음.
    • Operation이 외부에서 어떻게 보이길 원하는지 말해주는 이야기를 코드로 표현.
    • JUnit에 대한 상세한 사항들은 잠시 무시.
    • Stub 구현을 통해 테스트를 통과.
    • 좋지 못한 방법으로 테스트를 통과.
    • 돌아가는 코드에서 상수를 변수로 변경 -> 점진적으로 일반화
    • 새로운 할 일들을 한 번에 처리하는 대신 TO-DO 목록에 추가하고 넘어감.
BBBOMi added a commit that referenced this issue Jun 23, 2020
@BBBOMi
Test : 간단한 곱하기 테스트 케이스 생성 #2
49fbcd7 
해결하고자 하는 이슈에 대해 간단한 시나리오 생성
* 시나리오 생성 중 발생한 이슈는 TO-DO 리스트에 추가할 예정
@BBBOMi
Copy link
Owner Author

BBBOMi commented Jun 23, 2020

테스트 케이스를 작성하다 생긴 컴파일 에러 목록

  • Dollar 클래스가 없음
  • 생성자가 없음
  • times(int) 메서드가 없음
  • amount 필드가 없음

BBBOMi added a commit that referenced this issue Jun 23, 2020
@BBBOMi
Feat : Dollar class 생성 #2
bf2316d 
컴파일 에러를 해결하기 위한 클래스 생성
* 테스트 케이스는 통과 실패
BBBOMi added a commit that referenced this issue Jun 23, 2020
@BBBOMi
Refactor : test case를 통과하기 위해 값 초기화 #2
b47ddd2 
테스트 케이스 통과 확인
BBBOMi added a commit that referenced this issue Jun 23, 2020
@BBBOMi
Refactor : times()를 통해서 값을 갖도록 변경 #2
831157c
BBBOMi added a commit that referenced this issue Jun 23, 2020
@BBBOMi
Refactor : 생성자와 메서드 수정 #2
ba084c3 
변경을 통해 테스트 코드와 코드 사이의 중복을 걷어냄
This was referenced Jun 23, 2020
Merged
Open
@BBBOMi BBBOMi closed this as completed Jun 23, 2020
@BBBOMi BBBOMi added the step label Jun 24, 2020
@BBBOMi BBBOMi mentioned this issue Jun 24, 2020
Merged
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
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@BBBOMi