Discussion :

[krizenn]

Bonjour,

Je suis débutant en javascript et je rencontre mon premier problème.

J'ai une boucle forEach qui récupère les valeurs d'une colonne d'un tableau (une date) puis qui effectue une action si la condition If est remplit (date inférieure à la date actuelle).

Voici mon fichier javascript, la fonction concernée est en bas:

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
28
29
30
31
32
33
34
 
$(document).ready(function() {
  hideInactives()
 
  $('#show').on( 'click', function (e) {
    e.preventDefault()
 
      $('#dataTable tbody tr').each(function () {
        $(this).show()
      })
 
      $('#hide').show()
      $('#show').hide()
    });
 
  $('#hide').on( 'click', function (e) {
    e.preventDefault()
      hideInactives()
  });
});
 
function hideInactives(){
  $('#hide').hide()
  $('#show').show()
 
  $('#dataTable tbody tr').each(function () {
    var $rowValue = $(this).find('td').eq(5).html()
 
      if(moment($rowValue).isBefore()){
        $(this).hide()
        $(this).addClass("inactives")
      }
  })
}

Mon problème est que je ne rentre qu'une fois dans mon if (pour la première date qui remplie la condition) même si plusieurs dates remplissent la condition.

Est-ce un problème d'algorithme?

Merci de m'avoir lu.

[SpaceFrog]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
counter=0;
 $('#dataTable tbody tr').each(function () {
    var $rowValue = $(this).find('td').eq(5).html()
    counter++;
     if(moment($rowValue).isBefore()){
        $(this).hide()
        $(this).addClass("inactives")
      }
  })
console.log(counter)

que te retourne counter ?

sinon isBefore() n'attend pas un argument pour la comparaison ?

[krizenn]

Counter me renvoit le nombre de row contenu dans le tableau.

isBefore() sans argument prend comme valeur la date actuelle, ce qui est le comportement voulu.

[SpaceFrog]

Code :

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

1
2
3
4
5
6
7
8
9
10
counter="";
 $('#dataTable tbody tr').each(function () {
    var $rowValue = $(this).find('td').eq(5).html()
    counter+=$rowValue ;
     if(moment($rowValue).isBefore()){
        $(this).hide()
        $(this).addClass("inactives")
      }
  })
console.log(counter)

et là les dates elles correspondent ?

[krizenn]

Oui je récupère bien toutes les dates de mon tableau.

Visiblement le soucis vient des dates, de 01 à 12 jours ça fonctionne (jusqu'en 2016, par pour 2017), à partir du 13è jour ou plus ça ne fonctionne plus. J'ai du mal à comprendre pourquoi, je vais me renseigner sur la documentation moment.js

[krizenn]

erreur

[SpaceFrog]

Mais encore ?

[krizenn]

Problème résolu, le soucis vient du format de date, mes dates sont au format DD/MM/YYYY et la comparaison se faisait par rapport au format MM/DD/YYYY.

[SpaceFrog]

ok