Discussion :

[taosikua]

Yop !

Bon , j'ai un truc vraiment mais vraiment bizarre qui m'arrive , je m'explique

A l'exécution de mon script , je reçois ce message d'erreur :

Uncaught SyntaxError: Identifier 'boutonfavoris' has already been declared at fav.js:1

Voila qui semble être une erreur vachement d'inattention basique et réglée en deux secondes , j'me rends donc sur mon script regarder ou sa cloche...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
const boutonfavoris = document.getElementsByClassName('btn-fav');
let myFetchArray = [];
for ( let i = 0 ; i < boutonfavoris.length ; i++ )*{
  boutonfavoris[i].addEventListener('click',  (e) => {
    e.preventDefault();
    for ( let x = 0 ; x < myFetchArray.length ; x++) {
      if (myFetchArray[x] === boutonfavoris[i].dataset.id ) {
        return;
      }
    }
    myFetchArray.push(boutonfavoris[i].dataset.id);
    console.log(myFetchArray);
  })
}

Et en fait bah... J'suis pas fou hein , ce message n'a pas lieu d'être ?

Merci d'avoir lu

[SpaceFrog]

Code :

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

for ( let i = 0 ; i < boutonfavoris.length ; i++ )*{

que fait l'astérisque à cet endroit ?

[taosikua]

Hey Grenouille !!! ;p

P'tit ajout lors du copiage collage apparemment ,elle n'existe pas dans mon script ;p

[SpaceFrog]

Dans ton addEventListener ...

remplace boutonfavoris[i] par this

[taosikua]

Malheureusement c'est pas ca :/ Mais c'est trop bizarre non ?

[SpaceFrog]

pourquoi une constante ?

Code :

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

let boutonfavoris = document.getElementsByClassName('btn-fav');

[taosikua]

Avant de posté mon soucis j'avais déja essayé de transformer cette ligne en var ou en let , aucun changement non plus! :/

[javatwister]

Je ne comprends pas bien non plus;
Fais tout de même attention à ne pas laisser se balader des "*" dans ton code...

Et essaie de faire comme ça:

Code javascript :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
const boutonfavoris = document.getElementsByClassName('btn-fav');
let myFetchArray = [];
for ( let i = 0 ; i < boutonfavoris.length ; i++ ){
	boutonfavoris[i].addEventListener('click',  (e) => {
	   e.preventDefault();
	   const d_id=e.target.dataset.id;
 
	   if(!myFetchArray.includes(d_id)){myFetchArray.push(d_id)}
	   console.log(myFetchArray);
	})
}

[taosikua]

Salut Javatwister ! j'ai essayé avec ton code , toujours la meme erreurs , j'me demande si j'appel pas deux fois le script et que c'est pour ça qu'une fois il me dit que c'est déjà déclaré , c'est la seule possibilité logique non ?

EDIT : J'crois bien que c'est ca , j'ai ajouté une autre déclaration de variable au dessus pour testé et ca m'retourne l'erreur... J'dois avoir réussi a appelé mon script 2x quelque part

[javatwister]

Ben oui, je n'ai eu aucune erreur avec ton code;
Tu as peut-être un appel de script externe plus haut dans ta page, où boutonfavoris est déclarée;
N'empêche, ta boucle pour vérifier la présence du data était alambiquée!

[taosikua]

Héhé , je débute encore ! ;p

Mais j'avoue que

Code :

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

if(!myFetchArray.includes(d_id)){myFetchArray.push(d_id)}

C'est pas mal , et plus claire ! :p