😇 리스트 초기화의 중요성..

인턴을 하면서 남는 시간동안 개발을 하다보니 블로그를 쓸 시간이 없어서 늦어버렸네유🥹 

오늘은 얼마 전 stateFlow 형태의 변수의 값을 업데이트해주다가 겪은 문제를 공유해보려 합니다! 

 

어제까지 영화 검색 기능을 구현하고 있고 있었는데, 검색창에서 영화를 검색한 후 검색버튼을 클릭하면 여름 키워드를 가진 20개의 영화 데이터가 온다.

🌱푸릇푸릇한 여름색감 영화가 나오기를 기대하면 검색해보았다

 

디버깅 결과 이렇게 20개가 왔다.

 

😱 문제 발생

그런데 문제는 다시 동일한 키워드를 검색할 때 발생하였다..!!!

같은 키워드 데이터라면 1번만 보여야한다. 그러나 20개의 데이터 아래에 동일한 20개의 데이터가 추가되어서 총 40개의 데이터가 보이고 있었다. 

 

리스트가 초기화가 안되나? 싶어서 뷰모델 상에서 list.clear() 도 해주고 _movieList.value = emptyList(), _movieList.update{ emptyList() } 등등 여러 방법을 시도했는데 아무리해도 계속 중복된 데이터가 화면에 보였다.

 

🥸리스트 클리어의 중요성 

그런데 문제는 뷰모델이 아니라,,, use case에 있었다. 

현재 유즈 케이스 상에서 레포지토리를 참조하고, 서버 호출을 해준 후 flow 형태로 데이터를 반환해준다. 

그리고 뷰모델에서 해당 flow를 수집 collect 한 후 그리드뷰에 표시해주고 있었다.

 

 

여기서 이제 여행관련 영화를 찾고 싶어서 여행을 검색한다면?

- 원하는 것은 : 여행 관련 데이터로 화면에 보이는 포스터들이 변경되는 것이다.

- 그러나 실제 수행된 것은 : 바라 라고 검색하여 뜬 영화 포스터들 뒤에 여행 관련 영어 포스터가 더해진 형태였다.

 

문제는 단순했다. 실제로는 거의 3일 넘게 뷰모델 코드를 변경했지만 ㅠㅠㅠ 유즈케이스가 원인이었다.

서버 호출의 응답값인 result를 얻고

result 내부의 results에서 posterPath가 있다면 이 포스터 uri를 list에 더해주는 작업을 하고 있었다.

 

나는 유즈 케이스가 매번 새로 호출된다고 생각하여서 list도 매번 초기화될 거라고 생각했다.그러나 아니었다.

실제로는 list에 이전 데이터들이 누적해서 더해지고 있었던 것이다...!

 

list에 포스터 uri을 더해주기 전에 list를 초기화하기 위해 clear()하는 작업을 추가해주었다. 

   list.clear()
                result?.getOrNull()?.results?.forEach {
                    if (it.posterPath != null) {
                        list.add(it)
                    }
                }

 

드디어 해결 ...!!!! 

 

왜 유즈 케이스가 매번 새로 호출되지 않는 것인지는 알아보고 내용을 추가해봐야겠당 

 

 리스트 초기화되지 않는 원인 알아본 결과

뷰모델 생성 시 유즈 케이스에 대한 객체를 생성하고, 뷰모델이 살아있는 동안 그 객체는 1개이기 때문에 리스트도 계속 누적되는 것이었다....!!!!!!! 

현재는 list에 포스터 리스트를 추가해주기 전에 list.clear()을 통해 초기화해주어서 해결하였다. 그러나 한 검색어 당 페이지에 따른 데이터 누적을 추가해야하므로 이 로직은 추후 수정해야한당