♻️ ⚡ Improve uploading image #375
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
- utilization of coroutine - image compress - PNG -> JPEG
| viewModel.fetchImageUri(File(currentPhotoPath!!).name) | ||
| } else { | ||
| showSnackBar(getString(R.string.image_selection_failed)) | ||
| lifecycleScope.launch { |
There was a problem hiding this comment.
Fragment에서 lifecyleScope를 직접 사용하면 메모리 누수의 우려가 있습니다. viewLifecycleOwner.lifecycleScope를 사용하는 것이 좋아보입니다!
There was a problem hiding this comment.
좋은 의견 감사드립니다!
하지만 viewLifecycleOwner.lifecycleScope를 사용하였을시에 이미지 업로드 하는 과정에서 Android Home으로 갔다오면 코루틴이 중단이 되어서 업로드가 안되기 때문에 이렇게 한것인데 어떤 의견이신가요?
| @@ -120,11 +125,20 @@ class RecipeMakingFragment : Fragment() { | |||
| } | |||
|
|
|||
| private fun processImageUri(uri: Uri) { | |||
There was a problem hiding this comment.
함수 이름과는 달리 함수 내부에서는 로컬 사진의 uri를 기반으로 presignedUrl을 가져올 뿐만 아니라 압축까지 진행하는데, 함수 이름을 변경하거나 두 개의 함수로 분리하는 것이 좋아보여용ㅎㅎ
There was a problem hiding this comment.
2개로 나누기는 어려워 보여서 compressAndFetchPresignedUrl로 네이밍 변경했습니다!
| val compressedFile = imageUtils.compressAndResizeImage(uri) | ||
| currentPhotoPath = compressedFile.absolutePath | ||
|
|
||
| withContext(Dispatchers.Main) { |
There was a problem hiding this comment.
해당 작업을 Main Dispatcher로 Context Switching 하신 이유가 궁금합니다!
There was a problem hiding this comment.
음.. 이건 UI작업이라고 생각했는데 아닌가요..? 허허
| } | ||
| } catch (e: IOException) { | ||
| e.printStackTrace() | ||
| withContext(Dispatchers.Main) { |
| val compressedFile = imageUtils.compressAndResizeImage(uri) | ||
| currentPhotoPath = compressedFile.absolutePath | ||
|
|
||
| withContext(Dispatchers.Main) { |
|
|
||
| fun processImageUri(uri: Uri): String? { | ||
| return try { | ||
| // fun processImageUri(uri: Uri): String? = |
There was a problem hiding this comment.
제거하기에는 압축하지 않은 파일을 올릴때도 필요할것 같아서 남겨둔건데 주석은 다 없앨까요?
There was a problem hiding this comment.
넵! 커밋 기록에 남아있기 때문에 상관 없을 것 같아요!
| outputStream.flush() | ||
| outputStream.close() | ||
|
|
||
| originalBitmap.recycle() |
There was a problem hiding this comment.
This is an advanced call, and normally need not be called, since the normal GC process will free up this memory when there are no more references to this bitmap.
인터페이스 문서 상에서, GC가 알아서 처리해줘서 recycle()은 딱히 호출할 필요는 없다고 하는데 지우는 것은 어떻게 생각하시나요?
There was a problem hiding this comment.
혹시라도 메모리 누수가 있을 것 같아서 바로 없애려고 한건데 우선 지워보겠습니다!
Original issue description
closes #374