Discussion :

[je®ome]

Bonjour,

j'ai beau chercher dans la fac et dans les exemples de Valère VIANDIER , mais je ne comprend pas pourquoi mon Thread n'est pas pris en compte.

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
39
40
41
42
43
      Thread setTime;

          public void actionPerformed(ActionEvent e) 
    {
//         JFrame frame1 = new JFrame("---Snake---");
        frame1.setSize(400,400);
        setTime=new Thread();
        
        if(e.getActionCommand().equals("DEMARRER"))
        {
            frame1.addKeyListener(new KeyAdapter() 
            {
                public void keyPressed(KeyEvent e) 
                {
                    int keyCode = e.getKeyCode();
        
                    if ((keyCode == e.VK_LEFT) &&(!right)){left = true; up = false; down = false;if(!started)started=true;}
                    if ((keyCode == e.VK_RIGHT) && (!left)){right = true; up = false; down = false;if(!started)started=true;}
                    if ((keyCode == e.VK_UP) && (!down)){ up = true; right = false; left = false;if(!started)started=true;}
                    if ((keyCode == e.VK_DOWN) && (!up)){down = true; right = false; left = false;if(!started)started=true;}
                    if(compteur==0)
                    {
                        deplacement();
                        compteur++;
                        setTime.start();
                    }
                }
            });

            x[0]=91;
            y[0]=91;
            button.setEnabled(false);
            button1.setEnabled(true);
            frame1.setContentPane(new Affiche(2,0,true,x,y,0));
            frame1.setLocationRelativeTo(frame1.getParent());
            frame1.setBackground(Color.white);
        
            //Display the window.
            frame1.pack();
            frame1.setVisible(true);
        }
}

dans le méthode déplacement:

Code :

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

1
2
3
4
5
6
7
      while(stop==1)    { // j'éffectue différentes opérations

   //a la fin des opérations, je souhaite faire un sleep                     
try{setTime.sleep(level);}            
 catch(InterruptedException e){}
 }

Je reçois l'erreur suivant :

Code :

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

1
2
3
 
unreported exception java.lang.InterruptedException; must be caught or declared to be thrown
                                                setTime.sleep(level);

J'ai essayé sous d'autres formes, mais je ne trouve pas la bonne.
Qu'est-ce que je fais de faux ??

[adiGuba]

Salut,

1. Tu ne démarres pas ton Thread (je ne vois pas l'appel de la méthode start().
2. Ton Thread ne comporte aucun code : soit tu hérites de Thread et tu redéfinis la méthode run(), soit tu lui passes un objet Runnable en paramètre...
3. La méthode sleep() est statique, et tu ne dois pas l'utiliser comme cela mais l'appeller de manière statique ( Thread.sleep(2000); ).
4. La méthode sleep() renvoit une exception, tu dois donc la traiter (Voir la FAQ : Exception ... must be caught ... ?)

a++

[je®ome]

Je me suis gouré en faisant du copier coller, j'ai modifié mon code.
Je ne comprend pas vraiment ce que tu veux dire par mettre sleep en statique ??

Le code modifié fonctionne, sauf que j'ai l'erreur fréquente qu'il n' affiche pas les changements, mais seulement une fois que le pgm est terminé.
Jvais aller voir comment modifier ça.
N'est -il pas suggérer d'utiliser la classe Timer dans mon cas ??

Merci pour l'aide

[let_me_in]

N'est -il pas suggérer d'utiliser la classe Timer dans mon cas ??

pour faire quoi au juste? j'ai pas bien compris ton code.

[orelero]

C'est bizarre comment tu instancies ton thread .
Je pense que ta méthode dois gérer les exceptions de cette façon :

Code :

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

public void actionPerformed(ActionEvent e) throws Exception

.
Sinon je pense que ton code est bon, faudrait voir le reste.

[mavina]

Je ne comprend pas vraiment ce que tu veux dire par mettre sleep en statique ??

Non il ne suggere pas de mettre sleep en statique, il dit que dans la classe Thread, la methode sleep(int) est statique, c'est à dire que tu n'es pas obligé d'instancier la classe Thread pour l'utiliser...
un simple :

Code :

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

1
2
3
4
5
6
7
8
 
try
{
  Thread.sleep(30);
}catch (Exception e)
{
  System.out.println("fichtre, mon sleep n'a pas marché !");
}

note bien le try catch, qui est obligatoire, sleep generant des exceptions si il s'est mal passé..

Bien à toi

mavina