Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

JavaScript Discussion :

Portée de variable


Sujet :

JavaScript

  1. #1
    Membre régulier
    Portée de variable
    Bonjour, je n'arrive pas à comprendre pourquoi je n'arrive pas à récupérer la bonne valeur de ma variable btnAbsent ?

    Voir ligne 13 et 17
    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
    19
    20
    21
    22
    23
    24
    25
    26
    27
    // LE MODAL EST APPELE EN ATTRIBUANT DATA-TOGGLE ET DATA-TARGET A CHAQUE BOUTON
    var btnAbsents = document.getElementsByClassName('Absent');     
    var select = document.querySelector('.custom-select'); 
     
    btnAbsents.forEach(function(btnAbsent){
     
     btnAbsent.setAttribute('data-toggle', 'modal'); 
     btnAbsent.setAttribute('data-target', '#modalAbsent');
     
     btnAbsent.addEventListener('click', function(){
     
      label = btnAbsent.nextElementSibling; 
      console.log(btnAbsent); // --> je récupère bien mon ELEMENT HTML en fonction du click (ELT 1 - ELT 2 - ELT 3 ..... ok)
     
      select.addEventListener("change", function(e){
     
       console.log(btnAbsent); // --> je récupère systématiquement et uniquement mon ELEMENT HTML 1 quel que soit le click sur les btnAbsent (Elt1 - Elt1 - Elt 1....) alors que je pensais récupérer (ELT 1 - ELT 2 - ELT 3) en fonction du click
       btnAbsent.value = motifAbsence;   
     
       var motifAbsence = select.value;
       label.innerHTML = (' Absent : ' + motifAbsence); 
     
       $('#modalAbsent').modal('hide');
       e.stopImmediatePropagation();
      })
     })
    });


    Si vous aviez une explication et une solution. Merci d'avance

  2. #2
    Rédacteur/Modérateur

    Code :Sélectionner tout -Visualiser dans une fenêtre à part
    var select = document.querySelector('.custom-select');


    c'est un selection ...
    il faut boucler sur ces éléments pour faire le addEventListener
    Ma page Developpez - Mon Blog Developpez
    Président du CCMPTP (Comité Contre le Mot "Problème" dans les Titres de Posts)
    Deux règles du succès: 1) Ne communiquez jamais à quelqu'un tout votre savoir...
    Votre post est résolu ? Alors n'oubliez pas le Tag

    Venez sur le Chat de Développez !

    réalisations :www.oxygen-translations.fr|www.saftair.fr| www.ouestisol.fr | www.sistac-alizay.fr | www.acoustishop.fr | www.litt.fr | www.ouestventil.fr
    Humour

  3. #3
    Modérateur

    Bonjour,
    ce n'est pas un soucis de scope mais plutôt un soucis de cohérence/logique de ton code.

    Tout d'abord la méthode document.getElementsByClassName retourne une HTMLCollection et un HTMLCollection ne possède pas de méthode forEach mais peut-être l'as tu prototypée avant dans ton code.

    Il faut être conscient qu'à chaque fois que tu cliques sur un élément « btnAbsents » tu ajoutes un événement onchange à ton <select>.

    Donc si tu cliques sur #1 puis sur #2 et enfin sur #3, lors du change tu vas avoir trois console.log qui vont s'afficher.
    J'ai donc du mal à voir ce que tu veux au final !

    Un autre point, dans ces lignes
    Code :Sélectionner tout -Visualiser dans une fenêtre à part
    1
    2
       btnAbsent.value = motifAbsence;   
       var motifAbsence = select.value;

    tu affectes une valeur non encore définie, le « hosting » à ses limites !

    Un dernier point, lorsque l'on utilise jQuery on l'utilise, on ne mixe pas les écritures, vanilla et jQuery

    Citation Envoyé par SpaceFrog
    c'est un selection ...
    il faut boucler sur ces éléments pour faire le addEventListener
    il manque All pour que cela retourne une NodeList.

  4. #4
    Invité
    Invité(e)
    Bonjour,

    déjà, avec jQuery, ça simplifie nettement l'écriture ici...
    Code :Sélectionner tout -Visualiser dans une fenêtre à part
    1
    2
    $('.Absent').on('click', function(){
    ...


    Enfin bon. Avoir un bout du code HTML associé ne serait pas du luxe.

###raw>template_hook.ano_emploi###