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

[산타] 리나 미션 제출합니다. #14

Open
wants to merge 8 commits into
base: leena-main
Choose a base branch
from

Conversation

1jeongg
Copy link
Collaborator

@1jeongg 1jeongg commented Feb 25, 2024

구현하면서 고민되었던 점

  • 네이밍이 너무 어렵네요,, ㅠㅠ 특히나 할인금액 구하는 부분 정하는데 어려웠습니다.. 대충 파파고나 플러그인 사용해서 네이밍을 하다보니 어렵네요,, 팁이 있다면 공유 부탁드립니다🙏
  • 테스트코드 작성할 때 기존에 있던 매직 넘버들을 모두 활용하도록 테스트하는게 나을까요?

자랑스러운점

  • 테스트코드를 열심히 짰어요 (❁´◡`❁)
  • enum class를 적당히 잘 쓴 것 같아 만족스럽네요!

죄송한점

좀 급하게 과제를 한 느낌이 없지 않네요 ㅎㅎ 다음부턴 미리미리 해두겠습니다!
객체지향적으로 만들고 싶은데... 쉽지 않네요ㅠㅠ 관련해서 더 공부해봐야 겠습니다 🔥🔥

@1jeongg 1jeongg self-assigned this Feb 25, 2024
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