Discussion :

[Magicmax]

Bonjour à tous,

Voila, dans mon appli je crée un Thread qui doit effectuer une tache en fond entre deux heures définies par l'utilisateur.

Dans le constructeur du thread je récupère la date courante de cette maniere :

Code :

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

1
2
3
 
GregorianCalendar cal = new GregorianCalendar();
d = cal.getTime();

Ensuite voila ma méthode run() :

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
35
36
37
38
 
public void run(){
        while(true){
         if(beginH<endH){
                if((d.getHours()>=beginH) && (d.getHours()<endH)){
                 System.out.println(d.toLocaleString());
                 try {
         Thread.sleep(5000);
        } catch (InterruptedException e) {
         // TODO Auto-generated catch block
         e.printStackTrace();
        }
                }
                else try {
                 //sleeps for 15 minutes
                    Thread.sleep(900000);                    
                }catch (InterruptedException e){
                    System.out.println(e.getMessage());
                }
         }
         if(beginH>endH){
          if((d.getHours()>=beginH) || (d.getHours()<endH)){
           System.out.println(d.toLocaleString());
           try {
      Thread.sleep(5000);
     } catch (InterruptedException e) {
      // TODO Auto-generated catch block
      e.printStackTrace();
     }
          }
          else try {
           //sleeps for 15 minutes
           Thread.sleep(900000);
          } catch (InterruptedException e) {
           e.printStackTrace();
          }
         }
        }

Pour l'instant la tache à réaliser est un simple println suivit d'un court sleep().

Le problème est que le d.getHours() retourne toujours l'heure à laquelle l'appli a été lancé donc si l'appli est lancée en dehors de la plage de fonctionnement la tache n'est jamais réalisée et si l'appli est lancée dans la plage horaire définie par l'utilisateur alors la tache s'effectue tout le temps même lorsque l'heure de fin a été dépassée.

J'ai un peu de mal à comprendre d'où vient ce problème.
Merci

[Baptiste Wicht]

Je m'avance peut-être un peu, mais je vais te dire que c'est logique, tu initialise d une seule fois et au début de ton programme, donc à chaque fois que tu fais un getHours sur d, c'est normal que ca te renvoie l'heure de départ

[g_rare]

et si l'appli est lancée dans la plage horaire définie par l'utilisateur alors la tache s'effectue tout le temps même lorsque l'heure de fin a été dépassée.

Penser à la gestion d'un flag "boolean" indiquant si l'exécution s'est déjà déroulée pendant la plage horaire courante...

[Magicmax]

Je m'avance peut-être un peu, mais je vais te dire que c'est logique, tu initialise d une seule fois et au début de ton programme, donc à chaque fois que tu fais un getHours sur d, c'est normal que ca te renvoie l'heure de départ

Alors là autant pour moi, je savais pas qu'il fallait remettre à jour, mais maintenant que tu pointes le doigt la dessus, j'ai un peu honte de ne pas y avoir pensé
Merci en tout cas, je teste tout de suite...

[Magicmax]

J'ai rajouté

Code :

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

1
2
 
d = cal.getTime();

après mon while(true) sans plus de succès. Je suppose donc que ce n'est pas la bonne manière de mettre à jour l'objet d...

[g_rare]

Le tout ce n'est pas de mettre-à-jour une heure (ici representée par un "long"), mais de vérifier ta position horaire par rapport à l'heure courante : donc penser à faire...

Code :

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

1
2
3
4
5
 
Date maintenant = new Date(); // construite à partir de l'heure systeme (12H18)
// meme si tu fesais un traitement de 20 minutes ici
// la valeur de "maintenant" ne serait pas changee
long dateHeure = maintenat.getTime(); // valeur = 12H18 (pas 12H38)

NB: Pour info, la valeur d'une instance de "Date" et "Calendar" renvoyée par les méthodes "getTime" correspond à la valeur insérée (en création ou modification) dans ces objets !

[Magicmax]

D'accord je comprend mieux, la solution donc dans mon cas étant de créer un nouvel objet GregorianCalendar au début de chaque traitement.
Merci beaucoup pour ces éclaircissements

[g_rare]

Précision : en général on utlise un "Calendar" uniquement pour calculer des dates et heures (comme ajouter 3 semaines à une date ou décaler de 15 minutes une heure...), sinon on utilise une "java.util.Date" si cette date&heure ne doit pas être changée !