Estoy intentando escribir una función que verifique si el ID de usuario/ID de canal es válido al recorrer el almacén de datos a continuación.
let data = { users: [ { uId: 1, email: 'kif@kmail.com', password: 'kif123', nameFirst: 'Kifaya', nameLast: 'Shehadeh', handle: 'kifayashehadeh', }, { uId: 2, email: 'yus@ymail.com', password: 'yus1234', nameFirst: 'Yusra', nameLast: 'Mahomed', handle: 'yusramahomed', }, ], channels: [ { channelId: 1, name: 'DREAM', ownerMembers: [1,2,3,4,5], allMembers: [1,2,3,4,5], isPublic: false, messages: [ { messageId: 1, uId: 1, message: "Coffee is better at 12am.", timeSent: Math.floor((new Date()).getTime() / 1000), }, { messageId: 2, uId: 2, message: "Chocolate is better 24/7.", timeSent: Math.floor((new Date()).getTime() / 1000), }, ], }, { channelId: 2, name: 'COFFEE', ownerMembers: [1,2], allMembers: [1,2,3,4,5], isPublic: true, messages: [ { messageId: 1, uId: 4, message: "Dark chocolate isn't even chocolate. Seriously.", timeSent: Math.floor((new Date()).getTime() / 1000), }, ], }, ], };Esta es actualmente la forma en que lo estoy haciendo dentro de una función:
//invalid channelId let error = true; for (const channel of data.channels) { if (channel.channelId !== channelId ) { error = false; } } //invalid user for (const user of data.users) { if (user.uId === authUserId) { error = false; } } if (error === true) { return { error : 'error' } }Este método parece muy ineficiente y más parecido a C que a javascript, me preguntaba si había alguna línea mágica de código que pudiera hacer algo de eso por mí sin ser tan ilegible. También me resulta difícil descubrir cómo hacer que la verificación de errores funcione correctamente. ¿Hay alguna forma que me permita devolver el error inmediatamente para que salga de la función la primera vez que detecte un error?
Asignaría tanto los usuarios como los canales solo a las propiedades que le interesan, .channelId y .uid , y luego usaría .includes dos veces para ver que ambos valores están incluidos en ambas matrices.
const channelIds = data.channels.map(c => c.channelId); const userIds = data.users.map(u => u.uId); if ( !channelIds.includes(channelId) || !userIds.includes(authUserId) ) { return { error: 'error' }; }Es posible ponerlo en una sola línea, pero es menos legible. (Técnicamente, cualquier código JavaScript podría colocarse en una sola línea, pero eso no lo convierte en una buena idea desde el punto de vista del mantenimiento).
if (!data.channels.map(c => c.channelId).includes(channelId) || !data.users.map(u => u.uId).includes(authUserId)) { return { error: 'error' };}