Skip to content
This repository has been archived by the owner on Aug 13, 2022. It is now read-only.

[#9] refactor: signup service, and anything related #14

Open
wants to merge 1 commit into
base: ft/8
Choose a base branch
from
Open

Conversation

msugo1
Copy link
Collaborator

@msugo1 msugo1 commented Nov 27, 2021

Issues

  • 중복된 메서드, but 다른 엔티티들
    ... MyBatis 쓰듯이 table을 구상했기 때문에, 엔티티 간 가져야 할 프로퍼티들에 대해서는 별도로 고민하지 못했음
    ex. owner - restaurants, customer/rider - x
  • 엔티티를 분리하면 되나, sign up 등은 별도로 만들어져야 한다... (one for each, customer/owner/rider)
    = 공통으로 받는 입력 값에 대해서는 중복된 메서드가 만들어진다.
    = 이거를 어떻게 최소화 할 수 있을까

Done

  • 원래 생각했던 Idea
    = when (type)을 사용해 type에 따라 다른 repository, entity를 리턴하도록 할 수 있지 않을까?
    = 안되더라....
  • 따라서, 공통된 부분을 abstract service/repository로 추출
  • 필요한 부분(각 엔티티, repository)을 가져오는 부분은 별도로 구현
  • 리팩토링에 따른, 테스트 메소드 수정

* Issues
- 중복된 메서드, but 다른 엔티티들
  ... MyBatis 쓰듯이 table을 구상했기 때문에, 엔티티 간 가져야 할 프로퍼티들에 대해서는 별도로 고민하지 못했음
  ex. owner - restaurants, customer/rider - x
- 엔티티를 분리하면 되나, sign up 등은 별도로 만들어져야 한다... (one for each, customer/owner/rider)
  = 공통으로 받는 입력 값에 대해서는 중복된 메서드가 만들어진다.
  = 이거를 어떻게 최소화 할 수 있을까

* Done
- 원래 생각했던 Idea
  = when (type)을 사용해 type에 따라 다른 repository, entity를 리턴하도록 할 수 있지 않을까?
  = 안되더라....
- 따라서, 공통된 부분을 abstract service/repository로 추출
- 필요한 부분(각 엔티티, repository)을 가져오는 부분은 별도로 구현
- 리팩토링에 따른, 테스트 메소드 수정
@msugo1 msugo1 self-assigned this Nov 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant