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

[사다리 - 5단계] 정민주 미션 제출합니다. #27

Open
wants to merge 2 commits into
base: joungminju
Choose a base branch
from

Conversation

JoungMinJu
Copy link

내용

함수형 문법

함수형 문법을 적용했습니다. 학습 레포지토리로 충분히 학습하고자 노력했는데 .. 익숙치 않아서 그런지 잘 활용하진 못한 것 같습니다 :( 혹시 확인되시는 개선 포인트 등이 있다면 말씀 부탁드리겠습니다!

4단계 리뷰 반영

지난 4단계때 남겨주신 리뷰(링크)를 기반으로 코드를 수정하려했으나 이 과정에서 어려움이 있었습니다 ㅠㅠ

  • Ladder를 통해서 사다리를 타야할 것 같다고 말씀해주셨는데.. 이 부분이 잘 이해가 가지 않습니다 ㅠㅠ 지금 구조로도 ladder 안의 getResult() 함수를 통해 string으로 된 결과값을 받아오도록 구현되어 있어서요..!
  • 구현 당시 제가 생각했던 부분은, (1) 요구 조건에 따라 ladder를 생성하기 위한 "사다리의 생김새"는 ladder Service에서 구현한다. (2) service에서 생성된 "사다리의 생김새"가 "사다리의 조건"에 맞는지 검증하고 이를 기반으로 사다리를 생성하는 부분은 ladder에 구현한다. 였습니다!! 피드백해주신 부분이 어떤 의미인지 감은 잡히는데 .. 지금 구현된 구조에서 어떤 분이 잘못된 건지 포인트가 잘 잡히지 않습니다 :( 번거로우시겠지만 조금의 피드백을 더 제공해주시면 최대한 반영하도록 하겠습니다!

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