MongoDB: Eliminar Datos

Clase 23 de 33 • Curso de Node.js: Base de Datos con MongoDB y WebSockets

Clase anteriorSiguiente clase

Alberto D. Sosa

student•

El repo del profe!!

Erik Elyager

student•

Gracias por el link, me sirvió para responder una pregunta. :)

Carlos Enrique Ramírez Flores

student•

Gracias por el aporte!

Martin Alejandro Tecún Charles

student•

APORTE EXTRA Cuando hacemos la petición de eliminar por primera vez un mensaje no hay problema, pero cuando lo hacemos una segunda vez a pesar de que el mensaje ya no existe en al DB nos sigue apareciendo una respuesta ++SUCCESS++ y no una respuesta de ++ERROR++. Para ello hice el siguiente código:

async function existDB(id) {
    const exist = await Model.exists({
        _id:id
    });
    return exist;
}

Esto devuelve un TRUE o FALSE, lo cual es la respuesta si el mensaje existe en la DB.

M. Ray Medina Salcedo

student•

Debería ir dentro de la función removeMessage de store?

Usuario anónimo

user•

Yo lo implemente de la siguiente manera:

router.delete('/eliminar-mensaje/:id', (req, resp) => {
    let id = req.params.id
    controlador.eliminarMensaje(id)
        .then((mensajeEliminado) => {
            if (mensajeEliminado !== null)
                response.success(req, resp, `El mensaje  ${mensajeEliminado} ha sido eliminado correctamente`, 200)
            else
                response.error(req, resp, `Oops un error ha ocurrido`, 500, 'el usuario ya fue borrado o no existe')
        })
        .catch((error) => {
            response.error(req, resp, 'Errror Interno, disculpanos!', 500, error)
        })
})

Maximo Alberto Silva Perez

student•

El return despues del reject o del resolve no es necesario :D

John Nathan

student•

También podemos utilizar:

Model.findByIdAndDelete(id);

Carlos Dugarte

student•

Muy buena esta opción. Gracias

Jorge Luis Martínez Hernández

student•

hay un detalle en el código y es que si intentas eliminar un dato que no existe el funcionamiento hace parecer que se eliminó, en el caso de que se intente modificar un dato que no existe el programa se cae y la solución es un tanto simple, lo are solo para la función de eliminar pero para modificar es prácticamente lo mismo, lo primero es en el archivo store.js crear esta función:

const ifExist = async (id) => {
      return await Model.exists({
            _id: id
      });
}

lo único que hace es verificar mediante id si el dato existe, y devuelve true o false, luego en la función removeMessage se tiene que verificar primero si el dato existe:

const removeMessage = async (id) => {
      const siExiste = await ifExist(id);
      if(!siExiste){
            return siExiste;
      }
      return await Model.deleteOne({
            _id: id
      });
}

la función no cambio mucho, solo devolverá un false al controller en caso de que el id no exista en la BD. Como estaba el código de la función deleteMessage en controller.js aun devolviendo el false o un error con new Error() fa función deleteMessage no detecta si algo salió mal por lo que también hay que modificarla un poco:

const deleteMessage = (id) => {
      return new Promise(async (resolve,reject) => {
            if(!id){
                  reject('ID invalido');
            }
            await store.removeMessage(id)
                  .then((resultado) => {
                        if(resultado){
                              resolve();
                        }else{
                              reject('[Controller] El dato no existe en Store');
                        }
                  })
                  .catch(err => reject(err));
      });
}

con respecto al código de la clase solo agregue al then el parámetro resultado y el if para devolver resolve o reject segun sea el caso, en consola nos dirá que el dato no existe en el store y al usuario solo le mostrará el error que ya tenia, para el caso del patch es un fallo mas complejo por que si se intenta modificar un dato que no existe el servidor se muere, pero aquí la solución:

const updateMessage = (id, message) =>{
      return new Promise(async (resolve, reject)=>{
            if(!id || !message){
                  reject('[Controller] Invalid data');
                  return false;
            }
            await store.updateText(id,message)
                  .then(resultado =>{
                        if(resultado){
                              resolve(resultado);
                        }else{
                              reject('[Controller] No se encontro el dato para actualizar');
                        }
                  });
            
      });
}

solo modifiquen la función updateText en store.js para que valide si existe el id

Cristofher Jumbo Jimenez

student•

Se podria crear una funcion getOne y llamarla dentro del deleteMessage, no?

Orlando Manuel Mendoza Vargas

student•

Me encanta como el profesor termina cada clase, parece casi que hasta planeado. Te deja como con una incertidumbre, con unas ganas muy fuertes de seguir, diciendo implícitamente: "Ven conmigo, que apenas se está poniendo bueno 😎💚"

Jesús Manuel Mejía Rojas

student•

Para los que le den el error de then of undefined:

async function removeMessage(id) {
  return await Model.deleteOne({
    _id: id,
  });
}

Solo es agregar Asyn/Await

tranky

student•

¿por qué al profe no le dio error?

Mateo Costa

student•

¿Por qué hay que usar async? Gracias de antemano.

Ricardo Rito Anguiano

student•

En este curso aprendes mas de MongoDb que en el mismo curso de MongoDB jsjsjs

Juan Esteban Galvis

student•

En un módulo ya hicimos todo un CRUD con Node y Mongo 💚 C - Create (POST) R - Read (GET) U - Update (PATCH) D - Delete (DELETE)

Ya para profundizar más en Mongo definitivamente tomar su curso: https://platzi.com/cursos/mongodb/

Wandy Rafael Santana Evangelista

student•

Hola Devs: -Aca mi solucion usando TypeScript: El Network del componente Message:

El Controller del componente Message:

El Store del componente Message:

-Aca esta el commit exacto de esta clase: Click Aqui Recuerda, #NuncaParesDeAprender 💚

Max Andy Diaz Neyra

student•

Hola, para transformar el TS a JS, que usas?

Nancy Gomes

student•

¿Por qué dentro del ++store.js++ las funciones add y remove no son asíncronas mientras que el list y el update sí lo son?

Carol Estefanny Masmela Escamilla

student•

No tienen ninguna razon real. Supongo qeu solo lo quiso poner asi. Pero tienes un punto en mi opinion para ser concistente en el codigo seria bueno tener o solos async/await syntax o solo promesas :thu

Jonathan Gonzalez Botero

student•

Yo tambinén estoy de acuerdo con esto, no mezclar las cosas, si en el controller trabajamos solo con promesas, pues que todas sean así, y si en el store solo con async pues seria lo idea todo igual

Diego Alejandro Cifuentes Arango

student•

Este curso, me volo la cabeza, aunque ojala hubiera alguna conexión con un fronted para saber como conectarlo y poder bien el funcionamiento :/

Usuario anónimo

user•

Para conectar el back-end con el front-end es muy sencillo literal solo tomas la url o ip de tu API, y la consumes desde el front-end es decir irás haciendo fetchs a los endpoints de tu API que puede estar corriendo en tu localhost o con alguna IP que este en la nube :)

Wilson Castro Gil

student•

Como se puede eliminar más de un registro?

Anibal Fernando Ortega Piedrahita

student•

Lo que puedes realizar es un drop de toda la colección. No se si eso es lo que quieres. Saludos

Gloriq Lopez

student•

con deleteMany, le pudes pasar un filtro para que solo elimine lo que filtra o sin filtro y eliminara todo

model.deleteMany({ filtro   });

Líncol Saenz

student•

Genial, muy emocionado.

Jorge Zerpa

student•

Buen dia dev! al momento de borrar un mensaje me aparece esta respuesta:

revise el código y esta bien, ¿alguna idea? Googlee un poco y dice que puede se una cuestión de permisos de usuario en MongoDB atlas, pero aún no puede resolverlo...

de antemano muchas gracias y que tengan un gran día, me esta encantando el curso, excelente profesor e información <3

Andres Felipe Leon Gomez

student•

tuve el mismo error y luego de sacarme de mis cabales descubrí que era un typo, no se en tu caso

con el error

router.delete(":/id"

arreglado

router.delete("/:id"

la ubicacion del : / era lo que impedia la ruta al delete

Jorge Zerpa

student•

Hola Andrés! muchas gracias, tenía el mismo error <3

Omar España

student•

me dice que filter no esta definida

Erik Elyager

student•

Acá te he dado una respuesta.

Omar España

student•

ayuda

[response error]ReferenceError: filter is not defined```

Erik Elyager

student•

Guiándome por el contexto de la clase, en el archivo store.js del componente message asegúrate de tener definida la variable filter así:


async function getMessages(filterChat) {
    return new Promise((resolve, reject) =&gt; {
        let filter = {}; //&lt;-------#########AQUI
        if (filterChat !== null) {
            filter = { chat: filterChat };
        }
        Model.find(filter)
            .populate('user')
            .exec((error, populated) =&gt; {
                if (error) {
                    reject(error);
                    return false;
                }

                resolve(populated);
            });
    })
}

aquí el repo del proyecto.

Omar España

student•

tengo este error ayuda por favor

[response error]ReferenceError: filter is not defined

Erik Elyager

student•

Acá te he dado una respuesta.

jorge llanque

student•

Cómo puedo practicar con el método options?

Andrés Muñoz

student•

solo haz un request OPTIONS a /message o a /message/:id y te tienen que salir los métodos http que ya tienes habilitados

Juan Camilo Noreña López

student•

Cómo se cuando es mejor usar "async await " y cuando es mejor una promesa?