Discussion :

[kluck3000]

Bonjour,

J'affiche une date sous la forme :
debutSemaine(lundi) -> finSemaine(vendredi)
Exemple : 3/10/2008 -> 7/10/2008

Je possède un boutonPrecedent et un boutonSuivant pour faire varier la date.
Mon problème se situe quand je veux changer d'année en cliquant sur le boutonSuivant. J'arrive à passer en 2009 correctement, mais pas en 2010.

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
 
 
//j'appelle la méthode suivante dans l'actionPerformed de mon boutonSuivant
 
int compteur = 0;
public void afficherSemaineSuivante(){
 
    Calendar c = Calendar.getInstance(Locale.FRENCH);
    int anneeEnCours = c.get(Calendar.YEAR);
 
   if(nouveauNumSemaine > nombreDeSemaineDansLannee(Locale.FRENCH, anneeEnCours)) //je teste si le numéro de la semaine que j'affiche est supérieur au nombre de semaine dans l'année en cours
   {
     compteur++;
     nouveauNumSemaine = compteur;
     anneeEnCours = anneeEnCours + 1;
   }
 
   semaine(nouveauNumSemaine,anneeEnCours); //méthode qui permet d'afficher la date en fonction du numéro de la semaine et de l'année en cours
 
}

[lunatix]

euh, tu as une raison de pas utiliser les methodes de l'api calendar pour ajouter ?

genre calendar.add(1, Calendar.CE_QUE_TU_VEUX)
me semble que ca fait tout seul ce que tu veux faire

[kluck3000]

J'ai modifié mon code. Je fais directement le test avec le if dans ma fonction semaine auquel je passe maintenant uniquement le numéro de semaine.
Si je clique uniquement sur le boutonSuivant je peux effectivement aller aux années suivantes mais je n'arrive pas à conserver des numéros de semaines correctes pour toutes les années. J'ai essayé de mettre un compteur mais cela numérote bien les semaines uniquement pour l'année qui suit l'année en cours.
Exemple : on est la semaine 52 du 10/11/2008 -> 14/11/2008, je vais jusqu'à la semaine 1 du 29/12/2008 -> 02/01/2009, aucun probleme. Par contre si je tente d'aller en 2010, le numéro de la semaine est 54.

De plus, si je clique sur le boutonPrecedent je ne peux pas revenir en arrière, ce bouton fonctionne seulement si c'est l'année en cours.
Exemple :
on est la semaine 45 du 3/10/2008 -> 7/10/2008, je vais jusqu'à la semaine 2 du 7/01/2009 -> 11/01/2009 . Si je tente de revenir en arrière en 2008, je ne peux pas.

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
 
 
int compteur = 0;
boolean etatSem = false;
 
public void semaine(int numSemaine){
 Calendar c = Calendar.getInstance();
 int anneeEnCours = c.get(Calendar.YEAR);
 
 if(numSemaine> nombreDeSemaineDansLannee(Locale.FRENCH,anneeEnCours)) //je teste si le numéro de la semaine que j'affiche est supérieur au nombre de semaine dans l'année en cours
   {
     c.add(Calendar.YEAR, 1);
     etatSem = true;
   }
 
 if((etatSem == true) && (numSemaine != 53))
 {
  compteur++;
  numSemaine = compteur;
  etatSem=false;
 } 
 
 if(etatSem == true)
{
  compteur++;
  numSemaine = compteur;
}
 
c.set(Calendar.WEEK_OF_YEAR,numSemaine);

[dingoth]

Je crois qu'il faut que tu repartes sur de bases saines dans ton raisonnement.

Tu veux pouvoir naviguer dans les semaines, non ? Pourquoi ne pas faire un wrapper autour d'un objet Calendar ?

Ton code force l'année en cours et compare le nombre de semaines par rapport à soit l'année en cours, soit l'année suivante. D'où le fait que tu ne puisses aller au delà de l'année 2009 sans problème.

De plus tu testes la chose suivante : si le numéro de la semaine courante > nombre de semaines dans l'année, alors ajouter 1 à l'année. Et si par hasard on voulait la même semaine dans deux ans, on reste bloqué à l'année suivante.

Bref... repenser cette portion de code ne paraît pas un luxe.