Skip to content

리팩토링

Yoonu cho edited this page May 17, 2020 · 1 revision

[#187] 코드 리팩토링

1. 리팩토링 전 기존 코드 흐름을 보면서 생각하고 고민하는 시간을 갖자.

  • 백엔드에서 고려할 부분

    • 해당 날짜에 선택한 품종이 없을때는 items 프로퍼티에서 이미 빈 string이기 때문에 item 프로퍼티까지 접근할 필요는 없다 (예외처리)
    • 아이템이 null일때 (예외처리)
    • 아이템이 하나라 Object일때 Array로 만들어준다. 비교는 아이템의 length 조건으로 한다.
    • 기본 리스트 아이템, 품종선택 아이템 - defaultItem
    • 검색어 입력시 아이템 -filterItems
    • addArr()
    • 어떠한 조건이라도 [[]] 이중 배열을 유지한다.
  • 프론트엔드에서 고려할 부분

    • 그럼 totalCount는 프론트에서 for문으로 구하려는데 비지니스 로직이라고 생각이 되어서 mobx내에서 계산해야할거 같은데 addArr()가 백엔드에 있는한 나오지 않을거 같다.

2. 이후 의문점이 있는 코드에 대한 개선 (#180 포함)

3. 코드 리팩토링 (함수로 나누기)

Clone this wiki locally