starkoora / wanted-pre-onboarding-challenge-fe-components

7 stars 0 forks source link

과제 2) 비즈니스 로직을 분리해보고 느낀 점 블로그에 정리하기 #14

Open starkoora opened 10 months ago

starkoora commented 10 months ago

선택 과제) 본인이 작성했던 코드 중에서 비즈니스 로직을 분류 해보고 해당 로직을 적절히 캡슐화 하며 느낀 점 블로그에 정리하기

movie42 commented 10 months ago

숙제

제출합니다. 이게 맞는지도 모르겠고 하고 나니 또 다른 문제가 생긴 것같고... 쉽지 않네요.

Happhee commented 10 months ago

[ React ] 비즈니스 로직 분리하기

과제 제출합니다..!

yeonhub commented 10 months ago

[[2차 과제] 비즈니스 로직 분리]]

heogeon0 commented 10 months ago

2차과제 - 비즈니스로직분리 늦었지만 과제 제출합니다!

starkoora commented 10 months ago

@movie42 가장 먼저 드는 생각은 오히려 복잡해진거 아닌가 입니다. formItem을 반복문으로 처리하는 것과 여러개의 FormItemContainer로 처리하는 것 사이에는 어떤 차이가 있을까요? 그리고 한편 useJoinForm의 함수들이 각각 dispatch라는 부수 함수에 너무 강하게 연동되어 있지 않나 라는 생각이 듭니다. yup 등의 library를 사용하여 validation schema를 표현하고, rhf에 넘겨 검증 로직들 처리하도록 하면 어떨까요? @Happhee 비즈니스 로직에 대해 명확히 이해하신 것 같습니다. 한편 checkFormToSubmit는 내부에 api 호출 관련 로직을 가질 필요가 없어보입니다. 이름 부터가 check에 충실하려고 하고 있고요. boolean을 리턴하도록 하고 외부에서 결과물에 따라 함수를 처리하면 어떨까요?

starkoora commented 10 months ago

@heogeon0 Select 컴포넌트 내부에 구현되어 있던 로직을 외부에서 주입받도록 변경하여 유연성을 높이신게 핵심이었군요. 이를 IoC(제어의 역전)이라고도 부르는데, 추상화 과정에서 반드시 활용되어야 하는 개념인데 잘 적용되어 있는 것 같습니다 👍 @yeonhub 최종 코드만 놓고 보자면 아직 개선해야 할 부분이 많은 것이 사실이나 updateStock을 도출하는 고민의 과정이 잘 드러나 있는 것 같아 조좋은 것 같습니다. 이번 강의에서 다룬 내용으로만 끝내지 마시고 수업 중에 추천 드린 <쏙쏙 책>을 한번 훑으시면서 계속 적용해나가신다면 많은 개선이 있을 거라고 생각됩니다.

Happhee commented 10 months ago

@starkoora 네! 한번더 로직을 분리해 보겠습니다..! 감사합니다!