Skip to content

[1차 VER1.0] Java ToyProject upload by TaeyunChoi - #10

Open
cxxxtxxyxx wants to merge 80 commits into
FastCampusKDTBackend:mainfrom
cxxxtxxyxx:TaeyunChoi
Open

cxxxtxxyxx wants to merge 80 commits into
FastCampusKDTBackend:mainfrom
cxxxtxxyxx:TaeyunChoi

Conversation

@cxxxtxxyxx

@cxxxtxxyxx cxxxtxxyxx commented May 9, 2023

Copy link
Copy Markdown

빡세게 리뷰 부탁드립니다.

피어리뷰도 환영입니다. 마구마구 의견 내주시면 감사하겠습니다.

  • 분기 문을 사용하기 싫어서 리플렉션도 사용해보고, Enum 객체에 람다를 활용해서 정리해보았습니다.
  • 결과적으로 분기문은 사용은 줄었지만, 코드가 더 어려워지고, 복잡도가 증가한 것 같습니다.
  • 어떻게 설계하는 게 조금 더 괜찮은 방향인지 멘토님의 의견이 궁금합니다.

ps.

  • 계속 리팩토링 진행 중에 있습니다.
  • 기능도 아직 예외처리가 덜 된 부분이 있어 진행 중입니다.

cxxxtxxyxx added 30 commits May 10, 2023 04:23
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