Skip to content

[장바구니 리팩토링 Homework: Layered Architecture] 먼지 과제 제출합니다. - #11

Open
stardusty-lab wants to merge 2 commits into
woowacourse:mainfrom
stardusty-lab:vlmbuyd-stardusty
Open

[장바구니 리팩토링 Homework: Layered Architecture] 먼지 과제 제출합니다.#11
stardusty-lab wants to merge 2 commits into
woowacourse:mainfrom
stardusty-lab:vlmbuyd-stardusty

Conversation

@stardusty-lab

@stardusty-lab stardusty-lab commented Jun 4, 2026

Copy link
Copy Markdown

리뷰 요청 & 논의하고 싶은 내용

1) 이번 단계에서 가장 많이 고민했던 문제와 해결 과정에서 배운 점

  • 분리한 기능
    • api 함수 분리
    • 생성 관련 form state 분리
    • 유효성 검수 분리

2) 이번 리뷰를 통해 논의하고 싶은 부분

  • 3가지로 분리한 건 단순히 관심사의 분리인거지 layered architecture라고 볼 수 없다
  • handler 함수 내에서 a. 유효성 검증 - b. api 호출 - c. 관련 state 업데이트 를 순서대로 하는데 반복되는 경향이 있지 않나?
  • 관련 state 업데이트가 도메인 로직이라고 볼 수 있지 않을까?
  • usecase는 3가지: 데이터 로드, 생성, 삭제
    • usecase 별로 a, b ,c가 반복될 수 있는 것 같다

stardusty-lab and others added 2 commits June 2, 2026 15:23
Co-authored-by: stardusty-lab <stardusty-lab@users.noreply.github.com>
Co-authored-by: vlmbuyd <vlmbuyd@users.noreply.github.com>
Co-authored-by: stardusty-lab <stardusty-lab@users.noreply.github.com>
Co-authored-by: vlmbuyd <vlmbuyd@users.noreply.github.com>
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