search

f-lab-edu / check-in-before-leaving

사람들이 많이 모이는 핫플에 대한 현재 상황을 "체크인"요청으로 공유 받고 줄서기 또는 간단한 요청을 하여 보상을 제공합니다.
1 stars 0 forks source link

24주차 PR답변 모음 #130

Open ScottSung7 opened 2 hours ago

ScottSung7 commented 2 hours ago
  1. 도메인의 인터페이스의 DTO 참조 해결.

답변: 이 부분은 Hotfix로 수정하도록 하겠습니다! https://github.com/f-lab-edu/check-in-before-leaving/issues/131

답변링크

ScottSung7 commented 2 hours ago
  1. import문 제거 단축키

Optimize Imports는 정말 좋군요! 매번 하나씩 제거 했었습니다.

답변: 이 경우에는 인텔리제이에 특정 포메팅이 등록되어 있어야 하는 걸까요?? clean up을 눌러 보았지만 변경되는게 없습니다. 검색하다보니까 Optimize Imports를 쓰니 제거 되는데 혹시 이걸 말씀하신 걸까요?

답변링크

ScottSung7 commented 2 hours ago
  1. Search Controller 에서 도메인 서비스 사용

답변: 감사합니다! 도메인 서비스를 관련 비즈니스 처리에 유용하게 쓸 수 있어서 좋은 것 같습니다.

답변링크

ScottSung7 commented 2 hours ago
  1. 컨트롤러에서 inner class로 request, response 사용

답변: 네! 맞습니다. inner class로 request, response를 사용하는게 훨씬 찾기도 쉽고 좋았습니다. 컨트롤러가 매우 길어지는 경우가 아니면 이런 방식을 우선 사용하려고 합니다.

답변링크

ScottSung7 commented 2 hours ago
  1. inner class 사용 관련 주의 사항

답변: 음.. 이 부분이 조금 걱정되기는 했습니다. 그래도 public하고 열어 놓아서 다른 곳에서 재사용할 수도 있기는 한데, 다른 곳에서 사용되는 경우는 캡슐화 위반이 아닌가 걱정이 되었습니다. 여러군데에서 사용하기 시작하면 따로 파일을 생성하려고 합니다!

답변링크

ScottSung7 commented 2 hours ago
  1. instance of 개선 관련

답변: 아무래도 1번이 가장 큰 이유인듯 합니다. 왜냐하면 Check In, Line Up, Etc는 쓰기때는 따로 로직을 가지지만, 조회되는 경우는 종종 helpId 라는 부모의 식별자로 조회가 되어야 했습니다.

생소하다고 하셔서 조금 걱정되기는 하는데 혹시.. 염려되는 부분이 있어서 말씀주신걸까요?

답변링크

ScottSung7 commented 2 hours ago
  1. 성공 메시지 정책

답변: 네 맞습니다! 성공시에는 성공 메시지를 주려고 합니다.

답변링크