Tengo un estudiante objeto, tiene identificación de propiedad, nombre, nombre de grupo. allStudents es la matriz de objetos de estudiantes, quiero convertirlo en una identificación para el mapa de estudiantes, pero para cada estudiante, quiero generar una nueva propiedad "etiqueta", si el estudiante tiene nombre de grupo, el valor de la etiqueta es "nombre + nombre de grupo" , de lo contrario es nombre. Así que escribo debajo del código, funciona:
const idsToStudents = allStudents.reduce((tempMap, student) => { const getStudentLabel = (student) => { if (student.groupName) { return [student.name, `(${student.groupName})`].join(' '); } return student.name; }; const studentLabel = getStudentLabel(student); return { ...tempMap, [student.id]: { ...student, label: studentLabel} }; }, {});Defino la función getStudentLabel dentro de la función reductora, ¿hay una mejor manera de hacer esto en lugar de declarar la función getStudentLabel una y otra vez en la función reductora? Puede ignorar qué hace exactamente getStudentLabel, solo piense que toma a cada persona como parámetro y devuelve algo basado en la persona, ¿hay alguna manera de definir la función solo una vez, pero aún así puedo llamarla para cada persona en el reductor?
¡Gracias!
Está pasando a student como parámetro de la función, por lo que no necesita declararlo dentro de reduce . Esto también funcionaría:
const getStudentLabel = (student) => { if (student.groupName) { return [student.name, `(${student.groupName})`].join(' '); } return student.name; }; const idsToStudents = allStudents.reduce((tempMap, student) => { const studentLabel = getStudentLabel(student); return { ...tempMap, [student.id]: { ...student, label: studentLabel} }; }, {});Y también puedes acortar un poco el código:
const getStudentLabel = ({ name, groupName }) => groupName ? `${name} (${groupName})` : name; const idsToStudents = allStudents.reduce((tempMap, student) => ({ ...tempMap, [student.id]: { ...student, label: getStudentLabel(student) } }), {});No me preocuparía por redefinir las funciones dentro de los cierres. Cualquier implementación razonable de javascript optimizará eso para que se utilice una memoria adicional mínima. No diría que es exacto decir que está "definiendo" la función más de una vez; solo la está definiendo una vez en su código. La función se instancia cada vez, pero esta instancia aprovechará las memorias caché de las partes estáticas de la función. Así que tu forma de hacerlo está perfectamente bien. Como mencionó Ori Drori, no es necesario que tenga la variable estudiante en la función interna, pero podría ser una buena idea de todos modos, para que sea muy explícito acerca de las dependencias de la función.
Object.fromEntries puede funcionar en lugar de reduce .
const getStudentLabel = ({name, groupName}) => groupName ? name + ` (${groupName})` : name; const idsToStudents = Object.fromEntries( allStudents.map(student => [student.id, { ...student, label: getStudentLabel(student) }]) );