removiendo los callbacks quedaría mas o menos así
private suspend fun historialUsuario(): List<Oferta> {
val user = FirebaseAuth.getInstance().currentUser
return db.collection("canjeadas").whereEqualTo("idUsuario", user!!.uid).get()
.await().documents.map { document ->
val idOferta = document.getString("idOferta")!!
val oferta = db.collection("ofertas").document(idOferta).get().await()
Oferta(
document.id,
oferta.getString("titulo")!!,
oferta.getString("descripcion")!!
)
}
}
pero hay varias consideraciones que deberías tener en cuenta:
- en lugar de usar strings directamente en tu código, lo cual es muy propenso a errores, deberias usar constantes u obtener los nombres de las propiedades por reflejo
- firestore no es una base de datos relacional por lo que deberías evitar hacer consultas tipo JOIN siempre que sea posible. Por ejemplo en este caso los documtos de la colección "canjeadas" podrían tener un campo "tituloOferta".
- para una mejor abstracción, tu clase "Oferta" debería tener un método para crear una instancia a partir de un DocumentSnapshop
Ejemplo:
class Oferta(val id: String, val titulo: String, val descripcion: String) {
companion object {
fun DocumentSnapshot.toOferta(): Oferta = Oferta(
// ejemplo de reflection
getString(Oferta::titulo.name)!!,
getString(Oferta::descripcion.name)!!,
...
)
}
}
Aplicando esto, la legibilidad de tu funcción mejora considerablemente
private suspend fun historialUsuario() = db.collection("canjeadas")
.whereEqualTo("idUsuario", Firebase.auth.currentUser!!.uid)
.get().await()
.documents.map { it.toOferta() }