Tengo 5 botones con ID diferentes y dependiendo de qué ID tengan los botones, guardo el contenido en la base de datos. Creo que mi solución no está muy bien implementada, pero tampoco sé cómo podría hacerlo mejor, porque, por ejemplo, los literales de plantilla en las claves de objeto no funcionan.
if (button.id === 'song1') { await model.findOneAndUpdate( { userId: user.id, }, { 'song1.name': songName, 'song1.link': songLink, } ); } else if (button.id === 'song2') { await model.findOneAndUpdate( { userId: user.id, }, { 'song2.name': songName, 'song2.link': songLink, } ); } else if (button.id === 'song3') { await model.findOneAndUpdate( { userId: user.id, }, { 'song3.name': songName, 'song3.link': songLink, } ); } else if (button.id === 'song4') { await model.findOneAndUpdate( { userId: user.id, }, { 'song4.name': songName, 'song4.link': songLink, } ); } else if (button.id === 'song5') { await model.findOneAndUpdate( { userId: user.id, }, { 'song5.name': songName, 'song5.link': songLink, } ); }¿Alguna idea para codificar eso mejor?
Esto podría conducir a una respuesta basada en la opinión, por lo que esta publicación podría cerrarse, pero para responder a su pregunta sobre los literales de cadena en los nombres de las propiedades de los objetos, ¿quizás podría crear el objeto de actualización usando la notación de corchetes? es decir.
update_info = {}; update_info[button.id + '.name'] = songName update_info[button.id + '.link'] = songLink await model.findOneAndUpdate( { userId: user.id, }, update_info );Esto haría que el código fuera más dinámico.