Utiliser eval Fonction Pour accéder à des tableaux

Question

j'ai réussi à accéder différents tableaux et leurs éléments en utilisant une boucle et la fonction eval comme indiqué ci-dessous:

var Array1 = [A,B,C,D]; 
var Array2 = [D,B,C,A]; 
var Array3 = [B,C,A,D]; 
var Array4 = [A,D,B,C]; 

for(var row = 1; row <=4; row++){ 
    for(var column = 0; column <=3; column++){ 
    if(row<4 && eval("Array" + row)[column] == eval("Array" + (row +1))[column]){ 
     console.log("Value of Array" + row + "column" + column + "is equal to" + "value of Array" + eval(row + 1) + "column" + column + "\n"); 
    } 
    } 
} 

La question que j'est, ce que j'utilise correctement la fonction eval . Si ce n'est pas la bonne façon d'utiliser la fonction eval, comment accéder dynamiquement à différents tableaux de la boucle for sans utiliser la fonction eval?

Answer 1

L'utilisation de eval comme ceci, bien que cela puisse fonctionner, est une mauvaise idée et rend très facile l'écriture de code dangereux. Puisque eval exécutera son argument indépendamment de ce qui est réellement passé, les bogues qui aboutissent à un mauvais argument peuvent avoir des conséquences beaucoup plus graves que si vous n'utilisiez pas eval. Les réponses à this SO question offrent un peu plus de perspicacité. considère au lieu d'utiliser un objet de tableaux:

var arrays = { 
    Array1: [A,B,C,D], 
    Array2: [D,B,C,A], 
    Array3: [B,C,A,D], 
    Array4: [A,D,B,C] 
} 

for(var row = 1; row <=4; row++){ 
    for(var column = 0; column <=3; column++){ 
    if(row<4 && arrays["Array" + row][column] == arrays["Array" + (row + 1)][column]){ 
     console.log("Value of Array" + row + "column" + column + "is equal to" + "value of Array" + (row + 1) + "column" + column + "\n"); 
    } 
    } 
} 

Answer 2

Je ne dirais pas en utilisant eval comme ceci est une bonne idée. eval est très rarement utilisé car il est difficile de déboguer et peut être remplacé par quelque chose de plus facile à comprendre dans la plupart des cas. Il a des cas d'utilisation valides, mais ce n'est pas l'un d'entre eux.

Utilisez un tableau de tableaux à la place:

var A = 2, B = 2, C = 3, D = 4; 

var grid = [ 
    [A,B,C,D], 
    [D,B,C,A], 
    [B,C,A,D], 
    [A,D,B,C] 
] 

for (var row = 0; row < grid.length; row++) { 
    for (var column = 0; column < grid[0].length; column++) { 
     if (row + 1 < grid.length && grid[row][column] === grid[row + 1][column]) { 
      // they're equal 
     } 
    } 
}