Skip to main content
2 de 2
se añaden cambios al ultimo segmento del jsfiddle para ordenar la ultima fila.
moscoquera
  • 667
  • 3
  • 11

El problema no está en la función reduce sino en el ordenamiento de los datos y la inclusión de los datos Ausentes. No indagué mucho en el codigo ni pretendo hacerle mejoras, solo solucioné los errores que generaban que los datos se mostraran en las columnas incorrectas.

  1. No se ordenan los datos: si se quiere mostrar el numero de la semana por orden, siempre es necesario ordenarlos en algun punto. en el ejemplo que tienes, las semanas quedan ordenadas en la siguiente forma:

    25 | 26 | 24 | 23

Las semanas quedan en el mismo orden en el que se presentan en el arreglo llamado data. por lo que se puede aplicar sort en el arreglo semanas para que por lo menos la cabecera quede bien.

var semanas = data.reduce( (semanas_, elemento)=>{
  if(semanas_.indexOf(elemento.week) == -1)
    semanas_.push(elemento.week)
  semanas_.sort();
  return semanas_;
}, []);
  1. La información en cada fila se encuentra indexada incorrectamente, por ejemplo si en total se tienen 7 semanas, pero cada file tiene 2, 3 o 4 semanas, en la forma en que se estaba indexando siempre se mostrara la información en las primeras columnas de la fila y las demás se rellenarán con ceros. Hay muchas formas de hacerlo, una puede ser insertar en cada fila valores que representen las semanas faltantes, yo elegí al momento de mostrar cada celda, comprobar si existe esa semana para la fila actual, si existe mostrar el valor, poner cero sí no.

La diferencia entre los dos métodos depende mas del rendimiento esperado, pero para la mayoría de los casos no hay grandes penalidades en tiempo de ejecución.

tbody.innerHTML += f + semanas.reduce( function(a, c, i){
        var w = fila.week.find(function(current,index,fila_weeks){
        return current.numero===c;
            });
          if (w===undefined){
            w=0;
          }else{
            w=w.cantidad;
          }
        return a + '<td>' + w + '</td>'},   '') + '<td>' + fila.total + '</td>';   
});

Edit: se añade el ordenamiento a la fila de totales.

El código completo lo puedes probar en este jsfiddle.

moscoquera
  • 667
  • 3
  • 11