[1.1.0-AN_FEAT] 스플래시 때 포켓몬 front 이미지 캐시 #319

murjune · 2024-09-10T07:59:56Z

closed [1.1.0-AN_FEAT] 스플래시 때 포켓몬 front 이미지 캐시 #305

작업한 내용

스플래시 시 포켓몬 24장 내부 저장소에 저~장
전부 다 저장하니까 거의 40~50초 걸려서 처음 포켓몬 도감 화면 들어올 때 보이는 포켓몬 사진만 미리 저장했습니당

나중에 Paging + 스크롤할 때마다 이미지 Preload 해야할 것 같습니다.

PR 포인트

코드가 조금 어려울 수 있습니다 참고 자료 드립니다

우테코 5기의 Glide 캐시 정리 블로그
suspendCancellableCoroutine

🚀Next Feature

CD 하지롱

sh1mj1

고생하셨습니다!
궁금한 점 남겨두었는데 제가 직접 말로도 물어볼게요

sh1mj1 · 2024-09-11T01:35:26Z

android/data/src/main/java/poke/rogue/helper/data/cache/GlideImageCacher.kt

+        @SuppressLint("StaticFieldLeak")
+        private var instance: GlideImageCacher? = null
+
+        fun init(context: Context) {


init 에서 따로 초기화를 해주는 이유가 무엇인가요?
instance() 에서 instance: GlideImageCacher 에 할당을 해주는 것 외에 다른 이점이 또 있나요?

매번 context를 넣어주지 않아도 된다는 장점이 있습니다 ㅎㅎ

sh1mj1 · 2024-09-11T01:39:11Z

android/data/src/main/java/poke/rogue/helper/data/cache/GlideImageCacher.kt

+            }
+        }
+
+    override suspend fun clear() {


clear 를 따로 호출해줄 필요가 없는 걸까요?

이 함수를 호출해줘야 할 때가 있다면, 서버에서 내려오는 이미지가 변경이 되었을 때 호출하면 되는걸까요?

맞습니다~ 만약 서버의 url 에 해당하는 이미지가 바뀌었다면 clear 하고 다시 받아야겠죠?

sh1mj1 · 2024-09-11T01:42:10Z

android/app/src/main/java/poke/rogue/helper/presentation/splash/PokemonIntroViewModel.kt

@@ -30,7 +30,7 @@ class PokemonIntroViewModel(
                coroutineScope {
                    val warmUp = async { pokemonRepository.warmUp() }
                    val delay = async { delay(1000) }


여기서 delay(1000) 을 해야 할 필요가 있었나요..?

이래서 매직 넘버를 지양해야하는거군요!
스플래쉬 화면을 보여주는 최소 시간을 설정한 것입니다!

sh1mj1 · 2024-09-11T02:25:08Z

android/data/src/main/java/poke/rogue/helper/data/cache/GlideImageCacher.kt

+                        dataSource: DataSource?,
+                        isFirstResource: Boolean,
+                    ): Boolean {
+                        con.resume(Unit)


Timber.d(" $dataSource")
여기에 로그 찍어 놓고 실행하면 처음에는 REMOTE 로 찍히다가, 나중에는 DATA_DISK_CACHE 처럼 떠야 하는 게 맞나요?

제가 찍어봤을 때 처음 REMOTE 만 뜨고 다시는 뜨지 않아서요.
그러면 프리로딩한 캐시 이미지를 쓰고 있는 게 맞나 싶어서요

아아~ 이제 알았다 취소~

* 🎉 Chore: PR_Comment_Notification.yml PR Comment 에 '/noti' 를 포함할 경우 Andorid 채널 혹은 Backend 채널로 노티가 간다 * 🎉 Chore: �Avatar, Content, Description 수정 * docs: README v0.1 * CD: discord로 apk 전달 * CD: cd되는지 확인 * ci: artifact 버전 v3로 수정 * test * build : 병렬처리 * Revert "Merge branch 'main' into an/cd/send_to_discord" This reverts commit 925c8db, reversing changes made to 346391c. * ci: google-service추가 * ci: 버저닝 label 추가

murjune

작업 하나 추가했어요~

murjune · 2024-09-11T04:27:41Z

android/app/src/main/java/poke/rogue/helper/presentation/splash/PokemonIntroViewModel.kt

+                try {
+                    coroutineScope {
+                        val warmUp = async { pokemonRepository.warmUp() }
+                        val delay = async { delay(MIN_SPLASH_TIME) }
+                        awaitAll(warmUp, delay)
+                        pokemonRepository.warmUp()
+                    }
+                    _navigationToHomeEvent.emit(Unit)
+                } catch (e: Exception) {
+                    Timber.e(e)
+                    handlePokemonError(e)


이거 try-catch문을 추가해야하는데 제가 까먹었네요
이제 reconnect할 때 정상적으로 refresh될겁니당

저번에 회의할 때 작성했던 코드군요 !

kkosang

미션하느라 리뷰가 늦었네요 🥲
고생하셨습니다 오둥이~

kkosang · 2024-09-20T04:05:56Z

android/app/src/main/java/poke/rogue/helper/presentation/splash/PokemonIntroViewModel.kt

+                try {
+                    coroutineScope {
+                        val warmUp = async { pokemonRepository.warmUp() }
+                        val delay = async { delay(MIN_SPLASH_TIME) }
+                        awaitAll(warmUp, delay)
+                        pokemonRepository.warmUp()
+                    }
+                    _navigationToHomeEvent.emit(Unit)
+                } catch (e: Exception) {
+                    Timber.e(e)
+                    handlePokemonError(e)


저번에 회의할 때 작성했던 코드군요 !

kkosang · 2024-09-20T05:00:04Z

android/data/src/main/java/poke/rogue/helper/data/cache/GlideImageCacher.kt

+                        isFirstResource: Boolean,
+                    ): Boolean {
+                        con.resume(Unit)
+                        return false


이미지 로딩에 성공했는데 왜 return false를 반환하는건지 궁금했는데, 성공과 실패 여부를 알리는 것이 아니였군요 😂

그러네여 ,, Glide의 기본 성공 처리를 유지 여부에 관련된 것,,,, 메모,,

kkosang · 2024-09-20T05:06:11Z

android/data/src/main/java/poke/rogue/helper/data/cache/GlideImageCacher.kt

+            }
+        }
+
+    override suspend fun clear() {


이 함수를 호출해줘야 할 때가 있다면, 서버에서 내려오는 이미지가 변경이 되었을 때 호출하면 되는걸까요?

JoYehyun99

수고하셨습니다오둥~~~~~ 👍

JoYehyun99 · 2024-09-20T06:34:06Z

android/data/src/main/java/poke/rogue/helper/data/repository/DefaultDexRepository.kt

    private val biomeRepository: BiomeRepository,
    private val analyticsLogger: AnalyticsLogger,
 ) : DexRepository {
    private var cachedPokemons: List<Pokemon> = emptyList()

    override suspend fun warmUp() {
        if (localPokemonDataSource.pokemons().isEmpty()) {
-            localPokemonDataSource.savePokemons(remotePokemonDataSource.pokemons2())
+            val pokemons = remotePokemonDataSource.pokemons2()
+            cachePokemonData(pokemons)


cachePokemonData에 pokemons 를 take해서 넣어주는 것도 좋을둣요!!!

JoYehyun99 · 2024-09-20T06:34:27Z

android/data/src/main/java/poke/rogue/helper/data/cache/GlideImageCacher.kt

+        @SuppressLint("StaticFieldLeak")
+        private var instance: GlideImageCacher? = null
+
+        fun init(context: Context) {


JoYehyun99 · 2024-09-20T06:35:03Z

android/data/src/main/java/poke/rogue/helper/data/cache/GlideImageCacher.kt

+                        isFirstResource: Boolean,
+                    ): Boolean {
+                        con.resume(Unit)
+                        return false


그러네여 ,, Glide의 기본 성공 처리를 유지 여부에 관련된 것,,,, 메모,,

feat: Splash 화면에서 24개 사진 Preload

9c066d3

github-actions bot requested review from JoYehyun99, kkosang and sh1mj1 September 10, 2024 08:00

github-actions bot assigned murjune Sep 10, 2024

github-actions bot added the AN_FEAT ✨ 안드 새로운 기능 label Sep 10, 2024

sh1mj1 reviewed Sep 11, 2024

View reviewed changes

murjune added 3 commits September 11, 2024 13:06

Merge branch 'an/develop' into an/feat-image-cache

d63390b

fix: 매직 넘버 상수화 및 예외처리 문 추가

f765250

murjune commented Sep 11, 2024

View reviewed changes

murjune force-pushed the an/develop branch from e26d002 to d2365ff Compare September 11, 2024 04:50

fix

bdfc478

kkosang approved these changes Sep 20, 2024

View reviewed changes

JoYehyun99 approved these changes Sep 20, 2024

View reviewed changes

murjune added 3 commits September 22, 2024 00:57

style: ktFormat

5254bee

fix: withContext -> coroutineScope, 주석 변경

719e328

fix: Splash 시간 최소 시간 설정

1a05a48

murjune merged commit 0bd8290 into an/develop Sep 22, 2024
5 checks passed

murjune deleted the an/feat-image-cache branch September 22, 2024 09:09

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[1.1.0-AN_FEAT] 스플래시 때 포켓몬 front 이미지 캐시 #319

[1.1.0-AN_FEAT] 스플래시 때 포켓몬 front 이미지 캐시 #319

murjune commented Sep 10, 2024 •

edited

Loading

sh1mj1 left a comment

sh1mj1 Sep 11, 2024

murjune Sep 11, 2024

JoYehyun99 Sep 20, 2024

sh1mj1 Sep 11, 2024

kkosang Sep 20, 2024

murjune Sep 21, 2024

sh1mj1 Sep 11, 2024

murjune Sep 11, 2024 •

edited

Loading

sh1mj1 Sep 11, 2024

sh1mj1 Sep 11, 2024

murjune left a comment

murjune Sep 11, 2024

kkosang Sep 20, 2024

kkosang left a comment

kkosang Sep 20, 2024

kkosang Sep 20, 2024

JoYehyun99 Sep 20, 2024

kkosang Sep 20, 2024

JoYehyun99 left a comment

JoYehyun99 Sep 20, 2024

JoYehyun99 Sep 20, 2024

JoYehyun99 Sep 20, 2024

[1.1.0-AN_FEAT] 스플래시 때 포켓몬 front 이미지 캐시 #319

[1.1.0-AN_FEAT] 스플래시 때 포켓몬 front 이미지 캐시 #319

Conversation

murjune commented Sep 10, 2024 • edited Loading

작업한 내용

PR 포인트

🚀Next Feature

sh1mj1 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

murjune Sep 11, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

murjune left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

kkosang left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

JoYehyun99 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

murjune commented Sep 10, 2024 •

edited

Loading

murjune Sep 11, 2024 •

edited

Loading