Discussion :

[Hervé Saladin]

Bonjour,
j'ai un bug surprennant qui se produit aussi bien sous Firefox que Chrome :

dans un module ES6, je remplis un tableau associatif avec des données issues d'une requête HTTP vers une API REST (donc je ne code pas en dur ni les clés ni les valeurs de ce tableau, ce sont les données retournées par l'API).

J'ai un bout de code de la forme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
let monTableau = new Array();
 
for(let i = 0; i < donneesSources.length; i++) {
    let cle = donneesSources[i].toto;
    let valeur = donneesSources[i].titi;
    monTableau[cle] = valeur;
}

Tout se passe bien, jusqu'à ce que la boucle for tombe sur un cas particulier ou - manque de pot - donneesSources[i].toto vaut la chaîne "length" (ce qui est permis du point de vue de mon API REST).

=> Et là, c'est le drame !

Uncaught RangeError: Invalid array length

On dirait que l'interprêteur considère que j'essaie de faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

monTableau.length = valeur;

avec valeur qui n'est pas un entier.

=> Alors qu'en réalité, j'essaie de faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

monTableau["length"] = valeur;

ce qui n'est pas du tout la même chose de mon point de vue, mais on dirait que JS ne fait pas la différence.

Comble du bonheur, si j'essaie de faire une recherche Google avec les mots-clés "Javascript", "Array", "length" ou "RangeError" je n'obtiens que des résultats sans aucun rapport avec mon problème.

=> Avez-vous déjà rencontré ce bug ?
=> Est-ce normal que JS ne fasse pas la différence entre monTableau.length et monTableau["length"] ?
=> Avez-vous des astuces pour contourner "proprement" ce problème sachant que je ne choisis pas le contenu de cle ni valeur ?

Merci d'avance pour votre aide

[SpaceFrog]

tu as un tableau avec un seul élément

=> length=1

premier élément indexe [0]

ta boucle doit se faire jusqu'à length - 1

[Hervé Saladin]

Ben, c'est pour ça que dans mon for je mets la condition :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

i < donneesSources.length

avec un "<" pour "strictement inférieur", donc l'interpréteur sort de la boucle lorsque i atteint donneesSources.length.

Mais de toutes façons ce n'est pas le pb, c'est bien la ligne

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

monTableau[cle] = valeur;

qui déclenche l'erreur, et encore : uniquement lorsque cle vaut "length", comme décrit dans mon message (si tu le relis autrement qu'en diagonale )

[SpaceFrog]

Arf pas vu le strictement inférieur

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 cle = donneesSources[i].toto;

vaut quoi ?

[Hervé Saladin]

Ben justement : "length" ! Et c'est bien le problème !!!

[SpaceFrog]

Oui donc tu essayes de mettre dans une "clef" length d'un array une valeur string ...

Or la "clef" "length" est en fait une propriété qui existe déjà sur un array et attend un integer ...


Je ne sais pas si cela peut convenir à ton besoin mais si tu changes la case de ta "clef"

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
let donneesSources=[{"toto": "Length","titi":"name"}]
let monTableau = [];
 
for(let i = 0; i < donneesSources.length; i++) {
    let cle = donneesSources[i].toto;
    console.log(cle);
    let valeur = donneesSources[i].titi;
    console.log(valeur);
   monTableau[cle] = valeur;
}
console.log(monTableau);

[javatwister]

Salut,

Et tu peux inverser clé / valeur dans ton tableau de sortie? Si les données sont compatibles, ça va "résoudre" le problème (en fait, il faudrait que chaque valeur de titi soit unique... peu probable;

Sinon, tu reproduis la structure du tableau d'objets:

Code javascript :

Sélectionner tout - Visualiser dans une fenêtre à part

monTableau.push({cle,valeur});

[SpaceFrog]

Oui en plus JT a raison...
ta boucle ne fera que modifier à chaque fois une clef existante donc tu n'auras jamais que les dernières valeurs dans ton tableau

https://jsfiddle.net/smLdy7x5/

il est aussi possible de modifier la clef afin qu'elle ne vienne pas interférer avec les propriété existantes de l'array ...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 let cle = "_"+donneesSources[i].toto;