Utilisation de indexOf

Version imprimable

Bonjour à tous

je ne suis pas un grand seigneur du javascript et j'ai un souci que je n'arrive pas à comprendre

voici une fonction que j'ai faite
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 function check(numbers, stars){ let tir = { 5: [2,3,4,5,6,7,8,9,10,11,12], 6: [2,3,4,5,6,7,8,9,10,11,12], 7: [2,3,4,5,6], 8: [2,3,4], 9: [2,3], 10: [2] } let list = tir[numbers]; if(list.indexOf(stars) != -1){ return true; } else { return false; } }
cette fonction est faite pour verifier si je peux mettre X nombre et Y étoiles
par exemple sir je met numbers a 5 et stars a 2

Citation:

5: [2,3,4,5,6,7,8,9,10,11,12],

ici je sais que si numbers est égal à 5, stars peut avoir les valeurs 2,3,4,5,6,etc...

donc list = tir[5];

avec jsbin, codepen ect quand je fais

Code:

console.log(list.indexOf(stars));

ça me renvoit 0, normal puisque tir[0] = 2
là mon code fonctionne.

je le teste sur mon site ça me retourne -1, donc il ne le trouve pas

pour essayer de comprendre je fais ceci

Code:

console.log(numbers+'|'+stars+'|'+list+'|'+list.indexOf(stars));

en retour j'ai ceci

Citation:

5|2|2,3,4,5,6,7,8,9,10,11,12|-1

je ne comprends pas pourquoi il ne le trouve pas alors qu'avec jsbin ou autre, cela fonctionne.

Quelqu'un aurait il une réponse à me donner.

Merci à vous

10/12/2019, 10h46
javatwister

Bonjour,

Pas de raison que tu aies une différence en ligne; les 2 versions de ta page sont forcément différentes (imbrication de fonctions, problème de portée de variables, appel de check...

tu peux aussi gérer ton test comme ça

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
function check(numbers, stars){
  let tir = {
    5: [2,3,4,5,6,7,8,9,10,11,12], 
    6: [2,3,4,5,6,7,8,9,10,11,12],
    7: [2,3,4,5,6],
    8: [2,3,4],
    9: [2,3],
    10: [2]
  };
/* true = nombre et étoiles existent
   false = étoiles n'existe pas
   invalide = nombre n'existe pas */
  return tir[numbers] ? tir[numbers].includes(stars): "invalide";

10/12/2019, 11h38
colem

Citation:

Envoyé par javatwister

Bonjour,

Pas de raison que tu aies une différence en ligne; les 2 versions de ta page sont forcément différentes (imbrication de fonctions, problème de portée de variables, appel de check...

je ne comprends pas car en voyant les résultats que me donnais mon site, j'ai fais un copier coller sur jsbin et codepen

pour ce que tu m'a donner je regarde ça ce soir et te redis.

Merci à toi javatwister

Les implémentation de javascript ne se valent pas toutes.

quand tu écrit let tir = { 5: [ 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12] ....

il faut avoir en tête qu'ici la clé est considérée comme un string et non comme un entier ayant pour valeur 5

ce qui est équivalent à let tir = { '5': [ 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12] ....

en principe les interpréteurs JS font la conversion des entiers en string ( et inversement) selon les besoin du code.
donc écrire tir[numbers est automatiquement converti en tir[numbers.toString(10)], mais dans ton cas , j'ai un doute...

l'autre truc qui me chiffonne dans ton code (mais c'est une question de puriste) c'est la fait d' utiliser "numbers" comme nom pour une variable parce que c'est très proche de la classe d'objet Number de JS. faut avoir une pensée pour ceux font beaucoup de maintenance de code, donner des noms de variables synonymes à celle des éléments syntaxiques d'un langage apporte plus de confusion que de clarté; alors pitié pour ceux qui te relisent.

autre probleme, ton code ne vérifie pas que la valeur de num existe bien comme clé dans l'objet tir

Donc dans ton code une solution pourrait être :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 function check(num, stars) { let tir = { 5: [ 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12] , 6: [ 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12] , 7: [ 2, 3, 4, 5, 6] , 8: [ 2, 3, 4] , 9: [ 2, 3] , 10: [ 2] } if (!tir[num.toString(10)]) return false // vérifie l'existence de la clé return (tir[num.toString(10)].indexOf(stars) != -1) }
PS:

Citation:

Envoyé par colem

quand je fais

Code:

console.log(list.indexOf(stars));

ça me renvoi 0, normal puisque tir[0] = 2

c'est faux: tir[0] n'existe pas.

tu veux dire tir['5'][0] === 2 ? :?

10/12/2019, 15h16
javatwister
Citation:

Envoyé par psychadelic

en principe les interpréteurs JS font la conversion des entiers en string ( et inversement) selon les besoin du code.
donc écrire tir[numbers est automatiquement converti en tir[numbers.toString(10)], mais dans ton cas , j'ai un doute...

Pour un nom de propriété de l'objet "tir", ça paraît logique, oui;
N'empêche que si tu demandes
Code:

1 2 for(i in tir){ console.log(isNaN(i))}
tu obtiens false à tous les coups avec la configuration actuelle;

Sinon, je proposais le même test que toi avec

Code:

return tir[numbers] ? tir[numbers].includes(stars): "invalide";
10/12/2019, 17h28
psychadelic
Citation:
Envoyé par javatwister

N'empêche que si tu demandes

Code:

1 2 for(i in tir){ console.log(isNaN(i))}

tu obtiens false à tous les coups avec la configuration actuelle;
non cela ne révèle rien du tout
la fonction isNaN permet de savoir si une valeur peut ou non être traitée comme numérique

la preuve
Code:

1 2 3 const Str25 = '25' console.log('Str25: isNaN?=',isNaN(Str25), ' type=', typeof(Str25)) // -> Str25 Nan?= false type= string
plus significatif:

Code:

for(elmKey in tir) { console.log( elmKey, '->', typeof(elmKey) ) }

renvoie:

Citation:

5 -> string
6 -> string
7 -> string
8 -> string
9 -> string
10 -> string

sinon, pour la petite histoire
Code:

1 2 console.log("Str25 + 3 =>", Str25 + 3, typeof(Str25 + 3) ) // -> Str25 + 3 => 253 string console.log("Str25 * 3 =>", Str25 * 3, typeof(Str25 * 3) ) // -> Str25 * 3 => 75 number
10/12/2019, 17h42
javatwister

Ah oui, rigolo.
Je ne pense pas que ça fasse bugger un interpréteur, quand même; le transtypage est un avantage...

Citation:
Envoyé par psychadelic
sinon, pour la petite histoire
Code:

1 2 console.log("Str25 + 3 =>", Str25 + 3, typeof(Str25 + 3) ) // -> Str25 + 3 => 253 string console.log("Str25 * 3 =>", Str25 * 3, typeof(Str25 * 3) ) // -> Str25 * 3 => 75 number
:ptdr:

D'où la force d'evaluation de JS 8-)

10/12/2019, 18h03
NoSmoking
Bonjour,
pour commencer le choix d'un objet est plus que douteux dans ton cas puisque visiblement tu utilises un index 5, 6, 7, 8, 9 ...

Les propriétés des objets sont de préférence de type String pour des raisons de manipulation et d'écriture conforme au format JSON.

Dans ce cas autant utiliser un tableau de tableau
Code:

1 2 3 4 5 6 7 const tir = []; tir[5] = [2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12]; tir[6] = [2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12]; tir[7] = [2, 3, 4, 5, 6]; tir[8] = [2, 3, 4]; tir[9] = [2, 3]; tir[10] = [2];
ou autre forme d'écriture.

Les tests deviennent plus simple :
Code:

1 2 3 4 5 6 let list = tir[numbers] || []; if (list.indexOf(stars) != -1) { return true; } else { return false; }
l'appel hors indice ne lèvera pas d'erreur et renverra -1 ce qui est le but recherché, donc pas besoin de contorsion hasardeuse ;)

Citation:

Envoyé par psychadelic

l'autre truc qui me chiffonne dans ton code (mais c'est une question de puriste) c'est la fait d' utiliser "numbers" comme nom pour une variable parce que c'est très proche de la classe d'objet Number de JS.

Je ne vois pas où il y a problème tant que ce ne sont pas des mots clés qui sont utilisés, pour les francophones il y a effectivement moyen de faire autrement mais demande à un codeur anglophone, ou utilisant l'anglais dans le codage, de faire autrement !

alors je vous remercie tous pour m'avoir aiguillé.

voilà ce que j'ai fais et maintenant ça fonctionne

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
function check(numbers, stars){
  let tir = {
    '5': ['2','3','4','5','6','7','8','9','10','11','12'], 
    '6': ['2','3','4','5','6','7','8','9','10','11','12'],
    '7': ['2','3','4','5','6'],
    '8': ['2','3','4'],
    '9': ['2','3'],
    '10': ['2']
  }
 
  let list = tir[numbers];
  return list.includes(stars);
}

12/12/2019, 11h01
NoSmoking

@colem :
Tu ne fais toujours pas de vérification sur l'existence de la clé de ton objet tir que tu passes en paramètre.

En utilisant la méthode includes il faut être conscient que tu ne sera plus compatible IE même si cela devient de moins en moins nécessaire :aie:

@psychadelic :
Pourquoi avoir édité ton post pour mettre en gras ce passage :

Citation:

Envoyé par psychadelic

. faut avoir une pensée pour ceux font beaucoup de maintenance de code, donner des noms de variables synonymes à celle des éléments syntaxiques d'un langage apporte plus de confusion que de clarté; alors pitié pour ceux qui te relisent.

que, j'avoue ne pas mieux comprendre !