Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

검색 후 무한스크롤시 검색과 관련이 없는 게시물 나타나는 오류 수정 #20

Merged
merged 2 commits into from
Jul 28, 2024

Conversation

GeonH0
Copy link
Collaborator

@GeonH0 GeonH0 commented Jul 28, 2024

  1. 게시글을 검색할 경우 검색어와 관계없는 게시물이 나타나는 문제를 발견했습니다.
  2. ListViewController의 Delegate 설정이 누락되어 수정하였습니다
수정 전 수정 후

원인

  • 검색후 무한스크롤을 진행할 경우

다음과 같이 그냥 fetchNextRecipes를 호출하기때문에 검색과 상관없는 게시물들이 로드하는 문제점이 발생하였습니다.

해결 방법

  • 검색시 다음 페이지를 하는 메서드를 생성한후 다음페이지를 호출하는 메서드에 검색중인지를 식별하는 isSearching변수가 True일 경우 해당 함수를 호출하게 하였습니다.
    private func fetchNextSearchRecipes() {
        guard !isFetching else { return }
        guard let query = currentSearchQuery else { return }
        isFetching = true
        searchFeedListUseCase.execute(title: query, pageNumber: currentPage)
            .subscribe(onSuccess: { [weak self] result in
                self?.handleResult(result)
            }, onFailure: { [weak self] error in
                self?.handleResult(.failure(error))
            })
            .disposed(by: disposeBag)
    }

Copy link

@GeonH0 GeonH0 merged commit b34e36a into main Jul 28, 2024
2 checks passed
@GeonH0 GeonH0 deleted the feature/FixSearchScroll branch September 12, 2024 01:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant