[#10]; feat: University 상세 / 전체 조회 API 구현 #12
Conversation
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
| fun getUniversities( | ||
| @ModelAttribute request: UniversitySearchRequest, | ||
| @RequestParam(defaultValue = "0") page: Int, | ||
| @RequestParam(defaultValue = "10") size: Int |
There was a problem hiding this comment.
한 페이지에 데이터 10개씩 보여준다고 이해하면 될까요?
There was a problem hiding this comment.
default 값으로 10을 설정했습니다. 아직 정확히 얼만큼 첫 페이지에 학교 list를 출력되지 않은 것 같은데요.
생각해보니 10개는 너무 적을 것 같네요..
프론트 측에서 ?size=50 와 같이 수동제어를 할 수 있으므로, 일단 default = 20~30으로만 수정한 후,
추후 정확한 size 정해지면 변경해요!
There was a problem hiding this comment.
피그마 확인하니, 4 * 3 (12) 형태로 되어있네요. 쉽게 수정 가능한 부분이니, 12개로 임시 수정하겠습니다
|
|
||
| data class PageInfo( | ||
| val currentPage: Int, | ||
| val totalElements: Long, |
There was a problem hiding this comment.
이거는 게시물의 개수를 의미하는 변수인가요?
There was a problem hiding this comment.
네, 조회된 총 데이터 개수를 의미합니다.
저는 페이징 응답 시 위와 같이 현재페이지, 총데이터수, 총페이지수, 마지막페이지 여부를 습관처럼 작성합니다..
|
|
||
| import org.example.beyondubackend.domain.languagerequirement.implement.LanguageRequirement | ||
|
|
||
| data class LanguageRequirementResponse( |
There was a problem hiding this comment.
무지한 질문일 수 있겠지만 university 도메인에 LanguageRequirement에 대한 응답 dto가 있는 이유가 뭘까요?? LanguageRequirement 도메인이 따로 있는데 굳이 여기에서 응답할 거가 있을까 해서요...
There was a problem hiding this comment.
좋은 질문입니다!! 아키텍처 측면을 우선으로 본다면, language_requirement/business/dto 에 있는 것도 좋습니다.
그러나 해당 dto는 university 상세 조회에서 필요한 어학 요구사항에 필요한 데이터만 출력한다는 맥락에서, 해당 dto는 UniversityDetailResponse에서만 사용될거라 생각했습니다.
그렇다면 UniversityDetailResponse 클래스 내에 해당 dto를 넣을 수도 있겠네요.
주신 의견도 맞는 말이라, 합의가 가장 중요할 것 같네요!
There was a problem hiding this comment.
저도 맥스님이 여기에 쓰신 이유가 뭘까 생각하다가 university 상세 조회에서 필요한 어학 요구사항 데이터를 보여준다는 맥락이지 않을까 예상했는데 맞네요! 이런 의미라면 저도 이해가 됩니다 머지해도 될 것 같아요!
anozanami
left a comment
anozanami
left a comment
There was a problem hiding this comment.
늦게 달아서 죄송합니다... 대부분 질문이고 이외에는 다 괜찮은 것 같습니다! 고생하셨어요
2 participants
Closes #10
특이사항
2026-02-06