Discussion :

[bouboule22]

Hello !
J'ai besoin de vos lumières! Quelqu'un pourrait m'aider pour convertir du js en jquery pour un bout de code, j'ai tenté plusieurs trucs mais il y a des bugs...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 for (let i = 1; i < 4; i++) {
    let el = document.querySelector(".square[data-x='" + (x + i) + "'][data-y='" + y + "']"); 
 
    if (el) { 
      if ((el.getAttribute("type") === "wall") || (el.getAttribute("type") === "player")) { 
        break;
      } else if ((el.getAttribute("type") === "") || (el.getAttribute("type") === "weapon")) { 
 
        if (el.getAttribute("type") === "") { 
          el.setAttribute("type", "move"); 
        } else if (el.getAttribute("type") === "weapon") { 
          el.setAttribute("type", "move+weapon");
        };
        el.classList.add("move" + player);
        el.addEventListener('click', clickMove, false); 
      };
    };
  };

Merci beaucoup!!

[Invité]

Bonjour,

1- montre ce que tu as fait.

2- Si ça fonctionne en JS pur, ça n'a pas d'intérêt d'être "traduit" en jQuery.

[bouboule22]

Merci de me répondre, en fait j'ai remplacé les el.getAttribute et el.setAttribute par des $(el).attr mais ça bug, puis les el.Attribute uniquement avec $(el).attr idem
Mon code n'avait pas d'erreur mais ne fonctionnait pas.. je dois absolument utiliser jquery pour cet ex.

[Invité]

1- montre ce que tu as fait.

Si en plus c'est un exercice, on ne va pas le faire à ta place.

[bouboule22]

En fait, c'est un projet de jeu... Fort sympathique ce forum..., bonjour l'entraide !!!

[SpaceFrog]

en fait j'ai remplacé les el.getAttribute et el.setAttribute par des $(el).attr mais ça bug

1- montre ce que tu as fait.

Si tu ne nous montres pas ce que tu as fait, on ne peux pas deviner ton erreur ...

[NoSmoking]

Bonjour,
il me semble qu'il nous manque quand même, outre ce qui t'a déjà été demandé, un exemple de ton code HTML et notamment pour connaitre la valeur que peut prendre ta variable x, car il semblerait que tes sélecteurs soient simplifiables.

D'un autre côté ton code n'est pas des plus optimisé, ce qui tend à montrer que tu n'a pas bien appréhendé ce que tu souhaites écrire, car quand on voit dans la même boucle écrit cinq fois el.getAttribute("type") il y aurait lieu de factoriser cela

Code :

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

var attrib = el.getAttribute("type");

[bouboule22]

Bonjour,
J'étais un peu dépité vu le dernier message...j'ai réussi à me dépatouiller ! Merci NoSmoking pour ta remarque!!
Heureusement qu'il y a des gens comme vous, de bon coeur !