Así que tengo esta función forEach que reemplaza el texto en una celda de la tabla después de enviar algunos datos. Estoy reemplazando tres celdas de tabla diferentes según la fila de la tabla y las tengo separadas en tres declaraciones diferentes.
Como esto:
replaceData.forEach((item, i) => { $(`.tableData tbody tr:eq(${item.index})`) .children(["td:eq(6)"]) .text(values[i] === null ? "-" : values[i]); $(`.tableData tbody tr:eq(${item.index})`) .children("td:eq(7)") .text(reason[i]); $(`.tableData tbody tr:eq(${item.index})`) .children("td:eq(8)") .text(notes[i] === null ? "-" : notes[i]); });Intenté combinar esas tres declaraciones de esta manera, pero esto no funcionó. ¿Alguien sabe cómo puedo combinar estas tres declaraciones, si es posible? ¿O la única forma de lograr esto es manteniéndolos como su propia declaración?
$(`.tableData tbody tr:eq(${item.index})`) .children(["td:eq(6)", "td:eq(7)", "td:eq(8)"]) .text([ values[i] === null ? "-" : values[i], reason[i], notes[i] === null ? "-" : notes[i] ]);Hacerlos por separado parece lo correcto, poner todo en 1 sola línea no gana nada y pierde mucha legibilidad, especialmente cuando se usan muchas operaciones ternarias.
Pero podría limpiar su código aquí simplemente creando una referencia a la fila, en lugar de repetirla cada vez.
p.ej.
replaceData.forEach((item, i) => { const row = $(`.tableData tbody tr:eq(${item.index})`); row.children('td:eq(6)').text(values[i] === null ? '-' : values[i]) row.children('td:eq(7)').text(reason[i]); row.children('td:eq(8)').text(notes[i] === null ? '-' : notes[i]) })Si tiene muchas columnas, incluso podría crear una función en línea simple para que las cosas sean aún más fáciles de seguir.
p.ej.
replaceData.forEach((item, i) => { const row = $(`.tableData tbody tr:eq(${item.index})`); const R = c => row.children(`td:eq(${c})`); R(6).text(values[i] === null ? '-' : values[i]); R(7).text(reason[i]); R(8).text(notes[i] === null ? '-' : notes[i]); })