25shshsh / Sample

0 stars 0 forks source link

method naming #11

Open jurogrammer opened 1 year ago

jurogrammer commented 1 year ago
private BooleanBuilder getSearch(PageRequestDTO requestDTO) {

        String type = requestDTO.getType();

요청DTO를 받아서 BooleanBuilder를 반환하는 것으로 보아 쿼리 DSL에 적용할 검색 조건을 생성하는 것으로 보임. 따라서 getSearch 보단 getSearchCondition buildSearchCondition 등이 적절해 보임.

getSearch만 봤을 땐 검색 결과를 반환할 것으로 보임

private BoardDto getSearch(PageRequestDTO requestDTO) {

        String type = requestDTO.getType();
25shshsh commented 1 year ago

검색 결과를 반환하는 메서드가 아닌, 검색조건에 제목, 내용, 작성자 등을 포함시킬 것인가 제외시킬 것인가 검색 조건을 만드는 메서드가 맞음

bno를 boardNo로 변수명을 명확하게 수정하라는 것과 같은 의미로 받아드리겠음.

25shshsh commented 1 year ago

buildSearchCondition이 가장 명확하게 와닿는듯