Estoy trabajando en un Advent Code Challenge y me he topado con un pasillo. Resolví este error antes, pero en este caso, estoy atascado. El siguiente código me está dando un Uncaught TypeError: Cannot read properties of undefined. en relación con el switch (diagArray[i][j]) .
Mi pensamiento es que diagArray está fuera del alcance. ¿Es eso cierto?
Por favor, cualquier ayuda es genial!
const diagArray = [ [0,1,0,0,0,1,1,1,0,0,0,1], [1,1,0,1,0,0,0,0,0,0,0,1], [1,1,1,0,0,1,0,0,1,0,1,1] ]; // example data for(let i = 0; i < diagArray.length; i++) { for (let j = 0; j < diagArray.length; i++) { let gammaRate = '' let epsilonRate = '' let ones = 0 let zeros = 0 let rates = [] switch (diagArray[i][j]) { case 1: ones++ break case 0: zeros++ break default: break } if (ones > zeros) { gammaRate+=1 epsilonRate+=0 } else { gammaRate+=0 epsilonRate+=1 } rates.push([gammaRate, epsilonRate]) // console.log(rates) } }for(let i = 0; i < diagArray.length; i++) { for (let j = 0; j < diagArray.length; i++) {Esto aquí tiene múltiples problemas.
El primer ciclo for está bien, pero el segundo debe estar limitado por la longitud de la matriz interna:
diagArray[i].length Y actualmente también está incrementando los primeros recuentos de bucles con i++ . Eso tiene que ser j++
Ejemplo de trabajo:
const diagArray = [ [0,1,0,0,0,1,1,1,0,0,0,1], [1,1,0,1,0,0,0,0,0,0,0,1], [1,1,1,0,0,1,0,0,1,0,1,1] ]; // example data for (let i = 0; i < diagArray.length; i++) { for (let j = 0; j < diagArray[i].length; j++) { let gammaRate = '' let epsilonRate = '' let ones = 0 let zeros = 0 let rates = [] switch (diagArray[i][j]) { case 1: ones++ break case 0: zeros++ break default: break } if (ones > zeros) { gammaRate+=1 epsilonRate+=0 } else { gammaRate+=0 epsilonRate+=1 } rates.push([gammaRate, epsilonRate]) console.log(rates) } }Además, normalmente no es aconsejable tener bucles como este para iterar matrices, ya que, como ha descubierto, es muy fácil estropear los índices.
Recomiendo interactuar sobre los valores de las matrices en su lugar:
for (const row of diagArray) { for (const value of row) { const diagArray = [ [0,1,0,0,0,1,1,1,0,0,0,1], [1,1,0,1,0,0,0,0,0,0,0,1], [1,1,1,0,0,1,0,0,1,0,1,1] ]; // example data for (const row of diagArray) { for (const value of row) { let gammaRate = '' let epsilonRate = '' let ones = 0 let zeros = 0 let rates = [] switch (value) { case 1: ones++ break case 0: zeros++ break default: break } if (ones > zeros) { gammaRate+=1 epsilonRate+=0 } else { gammaRate+=0 epsilonRate+=1 } rates.push([gammaRate, epsilonRate]) console.log(rates) } }Es porque en el segundo ciclo for dice diagArray.length , y debería ser diagArray[i].length para obtener la longitud de la segunda matriz.
Su segunda iteración fue aumentar la variable i en lugar de j mientras iteraba a la longitud de diagArray en lugar de diagArray[i] .
Puede usar la API de la consola para depurar sus valores y averiguar de dónde se origina el problema sin tener que adivinar.
const diagArray = [ [0,1,0,0,0,1,1,1,0,0,0,1], [1,1,0,1,0,0,0,0,0,0,0,1], [1,1,1,0,0,1,0,0,1,0,1,1] ]; // example data for(let i = 0; i < diagArray.length; i++) { for (let j = 0; j < diagArray[i].length; j++) { let gammaRate = '' let epsilonRate = '' let ones = 0 let zeros = 0 let rates = [] console.log(`${i}:${j} = ${diagArray[i][j]}`) switch (diagArray[i][j]) { case 1: ones++ break case 0: zeros++ break default: break } if (ones > zeros) { gammaRate+=1 epsilonRate+=0 } else { gammaRate+=0 epsilonRate+=1 } rates.push([gammaRate, epsilonRate]) // console.log(rates) } }