studio 조회 관련 기능 개발 - Githubissues

TeamFILL-IN / server-renew

spring base

The Unlicense

2 stars 0 forks source link

studio 조회 관련 기능 개발 #28

Closed oownahcohc closed 8 months ago

oownahcohc commented 9 months ago

관련 이슈

25
22 : 해당 이슈 관련해서 질문 드릴게 있어서 언급했습니다. 저는 도메인을 pure 하게 가져가기 위해서는 도메인 엔티티가 영속성 엔티티를 몰라야 한다고 생각해서, StudioEntity 내부에 toStudio 메서드를 두어 매핑해주는 형태로 가져갔는데요, 어떻게 생각하시나요?!

studio 전체 조회

현재 studio 데이터가 적기 때문에, 전체 조회 시 모든 스튜디오를 내려주는 것으로 개발 진행했습니다.

studio 상세 조회

기존 studio 의 time, price 컬럼 타입이 TEXT 였기 때문에, 해당 부분 api response 가 매우 자유분방합니다. api response 관련해서 클라이언트와 논의가 필요할 듯 합니다. [ 현상소 데이터 ]
최대한 맞추고자 RunnigTimeEntity 에 OperationStatus, DayOfWeek 에 EVERY_DAY, WEEK_DAY, WEEK_END 등 추가했습니다.
time, price 의 response 형식은 도메인 로직과는 별개로 사용자 인터페이스와 데이터 표현 관련 로직(뷰 로직)으로 판단해서, 각각 뷰 모델로 분리했습니다.

studio 검색

studio 검색 관련해서는 글 작성한게 살짝 있는데 다듬어지는대로 #27 이슈에 공유하도록 할게요!!

InputKeyword : 사용자가 입력한 검색어에 대한 객체입니다. 공백 기준으로 입력 검색어를 split 합니다
KeywordManager : 공백 기준으로 나뉜 inputKeyword 의 토근에 대해서 각각 등급을 설정하는데요, 저희 서비스는 어차피 현상소, 스튜디오 검색이기 때문에 ~ 현상소, ~ 스튜디오 식으로 들어오는 검색어에서 현상소, 스튜디오 는 검색어 키워드로서의 가치가 낮다고 판단했습니다. 따라서 이와 관련된 단어들을 StopWord 로 설정해놓고 이외의 token들의 가치를 더 높게 설정해서 비즈니스 검색어를 만들어주도록 짰습니다.

daehwan2da commented 8 months ago

https://github.com/TeamFILL-IN/server-renew/pull/33/files 해당 PR 참고해 api response 를 ResponseEntity 로 변경 부탁드립니다! (status code 를 담기 위해 변경했었습니다.)

oownahcohc commented 8 months ago

https://github.com/TeamFILL-IN/server-renew/pull/33/files 해당 PR 참고해 api response 를 ResponseEntity 로 변경 부탁드립니다!

넵 확인했습니다!!