fix: firstOrNull { it != null }로 변경

filterNotNull().firstOrNull()은 리스트 전체를 순회하여 새로운 리스트를 만든 후 첫 번째 원소를 찾습니다. firstOrNull { it != null }을 사용하면 null이 아닌 첫 번째 원소를 찾는 즉시 순회를 멈추므로 더 효율적입니다.

Author: HI-JIN2

app/src/main/java/com/eatssu/android/data/remote/dto/response/MealReviewListResponse.kt

@@ -49,7 +49,7 @@ fun MealReviewListResponse?.toDomain(): List<Review> {
             rating = data.rating ?: 0,
             writeDate = data.writtenAt ?: "",
             content = data.content ?: "",
-            imgUrl = data.imageUrls.filterNotNull().firstOrNull(),
+            imgUrl = data.imageUrls.firstOrNull { it != null },
         )
     } ?: emptyList()
 }

app/src/main/java/com/eatssu/android/data/remote/dto/response/MenuReviewListResponse.kt

@@ -47,7 +47,7 @@ fun MenuReviewListResponse?.toDomain(): List<Review> {
             rating = data.rating ?: 0,
             writeDate = data.writtenAt ?: "",
             content = data.content ?: "",
-            imgUrl = data.imageUrls.filterNotNull().firstOrNull(),
+            imgUrl = data.imageUrls.firstOrNull { it != null },
         )
     } ?: emptyList()
 }

app/src/main/java/com/eatssu/android/data/remote/dto/response/MyReviewListResponse.kt

@@ -45,7 +45,7 @@ fun MyReviewListResponse?.toDomain(): List<Review> {
             rating = data.rating ?: 0,
             writeDate = data.writtenAt ?: "",
             content = data.content ?: "",
-            imgUrl = data.imageUrls.filterNotNull().firstOrNull(),
+            imgUrl = data.imageUrls.firstOrNull { it != null },
         )
     } ?: emptyList()
 }