feature#44 search 검색 후 화면 개발 #46
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
필터 open/close 여부에 따라 AnimatedVisibility 적용
공통 ProductCard를 통해 리스트 노출
- 중복 category name 기준으로 distinct 처리
- handleFilterClear() : 필터 초기화 및 검색 결과 재조회 - handleSubCategoryClick() : 서브카테고리 클릭 시 조회 - updateFilterProductCount() : 필터링된 상품 개수 계산 - updateFilteredProductList() : 필터링된 상품 리스트 - handleCategoryHeaderClick() : 상위 카테고리 클릭 이벤트 - initializeCategoryFilters() : 상위 카테고리 목록
jaeho
reviewed
Aug 20, 2025
| if (subCategoryNames.isEmpty()) { | ||
| return emptyList() // 선택된 필터가 없으면 빈 리스트 반환 | ||
| } | ||
| return getAll() |
There was a problem hiding this comment.
ProductDao 전반적으로 getAll() 를 사용해서 쿼리하고 있는데요. DB 쿼리에 파라메터로 넣지 않는 이유가 있으신가요?
There was a problem hiding this comment.
@f-lab-pepe @jaeho
단순하고 빠르게 기능을 구현하기 위해서 사용했던 것 같습니다. 멘토님의 코멘트를 생각해보다가 getAll()을 계속 사용하면 중복으로 조회가 발생하므로 불필요한 비용이 발생하는 것 같습니다. 현재 구조에서는 고정된 상품 리스트를 사용하고 있으므로 core모듈에서 공통으로 1번만 조회한 후 여러 모듈에서 재사용 하는 것이 성능면에서 최적화 될 것이라고 생각됩니다.
이렇게 진행하는 것이 괜찮은 방법일까요?
jaeho
reviewed
Aug 20, 2025
|
|
||
| @Composable | ||
| fun SearchResultScreen( | ||
| viewModel: SearchViewModel = hiltViewModel(), |
There was a problem hiding this comment.
@jaeho @f-lab-pepe
넵 해당 부분은 #48 리팩토링 하는 과정에서 제거 완료하였습니다!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔖 요약
(SearchResultScreen, SearchResultScreenContent)SearchFilterScreen)🛠 작업 내용
AnimatedVisibility사용하여 필터 컴포넌트 visibility 처리❗️ 참고 사항