스팀 앱 개발기 #80 - 버그 수정: 태그 화면 시작시 bridge.get_ranked_posts API가 여러 번 실행되는 문제

dorian-mobileapp (64)in kr-dev 커뮤니티 • 2 years ago

시작하며...

태그 화면의 오류를 다 수정했기를 기대했습니다. 그렇지만... 이 화면이 시작될 때, bridge.get_ranked_posts API가 여러 번 실행되는 것을 발견했습니다. 그것을 1번만 실행하고 서버로부터 포스트 리스트를 받아와야 정상이죠. 이 문제는 불필요한 데이터 사용을 유발할 뿐만 아니라 UI에 부하가 걸릴 수 있다는 것입니다. 반드시 수정해야 하는 것입니다.

원인

라디오 그룹의 OnCheckedChangeListener 리스너에서 TagsFragment 클래스의 readRankedPosts() 메소드를 호출한 것이 잘못 되었습니다. 라디오 버튼의 체크를 없애기 위해 RadioGroup 뷰의 clearCheck() 메소드를 호출했는데, 이 때에도 OnCheckedChangeListener 리스너가 실행되었습니다. 그걸 몰라서 readRankedPosts() 메소드가 반복 실행되었습니다.

MainViewModel 객체의 currentTag 라이브 데이터와 TagsViewModel 객체의 sort 라이브 데이터를 모두 관찰하는 것도 원인이었습니다. 처음에는 두 자료가 모두 설정된 후부터 readRankedPosts() 메소드를 실행하고 이후 둘 중 하나의 라이브 데이터만 변경되어도 이 메소드를 호출하고 싶었습니다. 그러나 처음에 이 두 라이브 데이터를 설정하면서 readRankedPosts() 메소드가 2번 호출 되었습니다.

해결 방법

OnCheckedChangeListener 리스너에서 readRankedPosts() 메소드 호출 코드 제거

private val sortCheckedChangedListener = OnCheckedChangeListener { radioGroup, radioButtonId ->
    viewModel.sort.value = when (radioButtonId) {
        R.id.radiobtn_trending -> GetRankedPostParamsDTO.InnerParams.SORT_TRENDING
        R.id.radiobtn_created -> GetRankedPostParamsDTO.InnerParams.SORT_CREATED
        R.id.radiobtn_payout -> GetRankedPostParamsDTO.InnerParams.SORT_PAYOUT
        else -> GetRankedPostParamsDTO.InnerParams.SORT_TRENDING
    }
}

TagsFragment 클래스의 currentSortObserver 필드 제거
라디오 버튼 클릭 처리 리스너 정의
- readRankedPosts() 메소드 호출하여 포스트 리스트 새로 받아 옴

private val radiobtnSortClickListener = OnClickListener {
    readRankedPosts()
}

TagsFragment 클래스의 onViewCreated 메소드에서 라디오 버튼 클릭 처리 리스너 설정

binding.apply {
    radiobtnTrending.setOnClickListener(radiobtnSortClickListener)
    radiobtnCreated.setOnClickListener(radiobtnSortClickListener)
    radiobtnPayout.setOnClickListener(radiobtnSortClickListener)
    // ...
}

TagsFragment 클래스의 onViewCreated 메소드에서 MainViewModel 객체의 currentTag 라이브 데이터 관찰을 맨 뒤로 미룸
- 이 코드를 호출하여 currentTagObserver가 바로 호출되기 때문에 TagsViewModel 객체의 sort 라이브 데이터가 먼저 설정되어야 함