Skip to main content
se eliminaron 6 caracteres en el cuerpo
Origen Enlace

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() }

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() }

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() }
Origen Enlace

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() }