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

Step3 키친포스 요구사항 테스트를통한 코드 보호 #276

Open
wants to merge 18 commits into
base: yunnote
Choose a base branch
from

Conversation

YunNote
Copy link

@YunNote YunNote commented Apr 10, 2022

아직 다 완성하지는 못했습니다 ..

먼저 된부분까지만 먼저 올리고 작업되는대로 계속 올려보겠습니다.

후딱 진도 따라가겠습니다

감사합니다

@YunNote YunNote changed the title Step3 Step3 키친포스 요구사항 테스트를통한 코드 보호 Apr 10, 2022
@YunNote
Copy link
Author

YunNote commented Apr 22, 2022

안녕하세요 재성님
말씀주신대로 강의영상 다시 보면서 코드 수정하였습니다 .

  1. 자꾸 Test코드에 대한 처리를 Exception을 기준으로 처리하게되는듯한데 혹시 더 좋은 방법이 있을까요 ?

  2. README.md에 작성한 요구사항과 테스트 코드를 맞추다보니 요구사항에 없는 테스트코드도 작성하게 되는거같은데 이렇게 해도 상관이 없는것일까요 ?

  3. 객체를 생성하기 위하여 메서드를 사용하였는데 메서드도 쓰다보니 중복이 생기고 비슷한 코드가 보이는듯한데 제가 작성한 코드에서 수정을 하게된다면 더 좋은 방법에 대해 참고할만한 키워드가 있을까요 ?

감사합니다

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