Modifier ma variable

Version imprimable

Bonjour

J'ai résolu mon problème mais je ne comprends pas mon erreur. Si vous pouvez m'aider à comprendre.
En fait, je n'arrivais pas à modifier une variable avec la fonction reset. Je ne pouvais pas mettre à zero la variable "numberoftimes".

Voila le code qui ne marchait pas :

Code:

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
import java.util.Timer;
import java.util.TimerTask;
 
import KiloboltGame.StartingClass;
 
public class Time extends TimerTask {
 
   public int numberoftimes;
 
   public Time(){
	numberoftimes = 0;
   }
 
//*************************** ça fonctionne pas ***************************
   public void reset() {
	numberoftimes = 0;
   }
//***********************************************************************
   @Override
    public void run() {
         numberoftimes++;
    }
 
   private Timer t;
 
   public void startTimer() {
        t = new Timer();
        t.scheduleAtFixedRate(new Time(), 0, 1000);
    }
 
 
}

Et le code qui fonctionne :

Code:

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
import java.util.Timer;
import java.util.TimerTask;
 
import KiloboltGame.StartingClass;
 
public class Time extends TimerTask {
 
   public int numberoftimes;
 
   public Time(){
	numberoftimes = 0;
   }
 
 
//*************************** ça fonctionne ***************************
 
   public void reset() {
        t.cancel();
	numberoftimes = 0;
        startTimer();
   }
 
//********************************************************************
 
   @Override
    public void run() {
         numberoftimes++;
    }
 
   private Timer t;
 
   public void startTimer() {
        t = new Timer();
        t.scheduleAtFixedRate(new Time(), 0, 1000);
    }
 
 
}

07/08/2015, 00h59
tchize_

vu que tu ne vérifie jamais numberoftimes, comment tu sais ce qui marche ou pas?

Ton code 2 est complètement différent. dans le premier tu crée deux fois Time(), celui sur lequel tu change les valeur et celui qui tourne dans le timertask
Dans le deuxième, tu crée 3 instances. Celle sur laquelle tu change les valeur, celle qui monte dans le timertask et celle qui va le remplacer après le cancel.

Retire toute trace de new Time() dans Time et tou ira bien, tu aura une seule instance.
07/08/2015, 01h54
gogoldujour

Citation:

Envoyé par tchize_

vu que tu ne vérifie jamais numberoftimes, comment tu sais ce qui marche ou pas?

Ton code 2 est complètement différent. dans le premier tu crée deux fois Time(), celui sur lequel tu change les valeur et celui qui tourne dans le timertask
Dans le deuxième, tu crée 3 instances. Celle sur laquelle tu change les valeur, celle qui monte dans le timertask et celle qui va le remplacer après le cancel.

Retire toute trace de new Time() dans Time et tou ira bien, tu aura une seule instance.

Pardon j'ai enlevé du code pour que cela soit plus lisible mais sinon j'avais une ligne pour afficher la variable.

Supprimer new m'a mis un message d'erreur comme quoi il n'existe pas de méthode Time(). ( The method Time() is undefined for the type Time )

Cancel a bien un rôle de destructeur ? En faisant t.cancel je détruis l'instance entre la variable t et la classe Time, et ensuite je la reconstruis en faisant startTimer(). Et comme je reconstruit l'instance il me relance le constructeur Time() qui remet à 0 la variable ? Est-ce acceptable comme programmation ?

Salut,

@tchize_ n'a pas dit de retirer le new, mais le new Time(), sous entendu ne pas créer plusieurs instances de Time, donc l'une serait celle manipulée par le Timer, et l'autre celle sur laquelle reset() serait invoquée. Il faut toujours que ça soit la même instance, sinon ce n'est pas le même compteur (numberoftimes) qui est utilisé. Il ne suffit pas d'ailleurs de supprimer le new Time(), il faut le remplacer au moins par this. Mais dans le premier code, ça poserait un problème : pouvoir lancer plusieurs Timer sur la même instance, ce qui n'est probablement pas voulu.

Pour le premier code, on pourrait faire également :

Code:

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
public class Time extends TimerTask {
 
   private volatile int numberoftimes;
 
   private Time(){
	numberoftimes = 0;
   }
 
   public int getNumberOfTimes() {
        return numberOfTimes;
   }
 
   public void reset() {
	numberoftimes = 0;
   }
 
   @Override
    public void run() {
         numberoftimes++;
    }
 
   public static Time startTimer() {
        Timer t = new Timer();
        Time time = new Time();
        t.scheduleAtFixedRate(time, 0, 1000);
        return time;
    }
 
}

Avec

Code:

1
2
Time chrono = Time.startTimer();
chrono.reset(); // c'est bien la même instance de Time que celle manipulée par Time

Mais avec ce code, impossible de terminer le chrono, sans fermer l'application.

Code:

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
39
40
41
42
43
44
45
46
import java.util.Timer;
import java.util.TimerTask;
 
import KiloboltGame.StartingClass;
 
public class Time extends TimerTask {
 
   private volatile int numberoftimes;
 
   public Time(){
	numberoftimes = 0;
   }
 
   public void reset() {
        numberoftimes = 0;         
   }
 
   public int getNumberOfTimes() {
        return numberOfTimes;
   }
 
   @Override
    public void run() {
         numberoftimes++;
    }
 
   private Timer t;
 
   public synchronized void startTimer() {
        if ( t==null ) {
           t = new Timer();
           t.scheduleAtFixedRate(this, 0, 1000);
        }
        else {
             reset();
        }
    }
 
    public synchronized void stopTimer() {
        if ( t!=null ) {
          t.cancel();
          t=null;
        }
   }
 
}

08/08/2015, 14h46
gogoldujour

Merci beaucoup pour les réponses, je comprends mieux.

Pour l'instant j'ai utilisé la solution de remplacer "new Time" par this. Je garde les autres solution sous le coude merci pour tout.