Probleme Activer/Desactiver thread

Version imprimable

Salut à tous,
J'ai développer un thread qui compare à chaque fois le nombre de ligne d'une table de ma base de données:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
public boolean verifier(int x) {
        String query2 = "select count(*) from client ";
      try{
             ResultSet rs=cn.getResultat(query2);
          while (rs.next()) {
           nblignes2 = rs.getInt(1);
                }
                rs.close();
          } catch (Exception e) {
      JOptionPane.showMessageDialog(null, "Erreur lors de l'exécution de la requête recherche du nouveau nombre enregistrement client");
            }
 
         if( nblignes2 != x) {
                    //System.exit(-1);
                    return false;
                }
         else
                return true;
            }

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
public void run(){
 
      String query = "select count(*) from client";
      try{
             ResultSet rs=cn.getResultat(query);
          while (rs.next()) {
           nblignes = rs.getInt(1);
                }
                rs.close();
          } catch (Exception e) {
      JOptionPane.showMessageDialog(null, "Erreur lors de l'exécution de la requête recherche nombre enregistrement client");
           }
    while (!isInterrupted()){
          if (verifier(nblignes)==false){
               JOptionPane.showMessageDialog(null, "Une modification a été effectué sur la table client");
               nblignes=nouveaunblignes();
               //System.out.println("=>"+ nblignes);
                //System.exit(-1);
          }
                   try {
                   Thread.sleep(7000);
                      } catch (InterruptedException ex) {
              Logger.getLogger(EspionProduit.class.getName()).log(Level.SEVERE, null, ex);
             }
 
        }
    }

****
et dans la page d'acceuil j'ai fait un bouton (soit activer, soit desactiver), et je veut que le thread soit activé ou bien activé en fonction du clic de l'utilisateur ... j'ai testé avec ce code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 EspionClient thread1 = new EspionClient();
 
 
        if (pageacceuil.comptclt % 2 ==0)
        {
 
            actclt.setText("Activer");
            thread1.interrupt();
 
        }
        else {
             actclt.setText("Desactiver");
            thread1.start();
            }
 
        pageacceuil.comptclt ++;

Avce actclt est le nom du bouton et comptclt est un compteur pour pouvoir basculer entre l'affichage de "activer" et "desactiver" sur le bouton ...
L'activation fonctionne parfaitement, mais quand je clique pour désactivé le thread, rien ne se passe et le thread continue a s'exécuté ... :calim2:
Que dois-je faire pour arrêter le thread en cliquant sur désactiver? :roll:

Merci ;)

Salut,

c'est l'une des subtilités de l'interruption des threads.
Un thread a un flag qui indique si il est interrompu. Quand on appelle la méthode interrupt, ce flag est positionné à true. Mais il peut changer suivant les opérations effectuées comme par ex un try catch. Il faut dans ce cas repositionner le flag.
Code:

1 2 3 4 5 6 7 try { Thread.sleep(7000); } catch (InterruptedException ex) { Logger.getLogger(EspionProduit.class.getName()).log(Level.SEVERE, null, ex); Thread.currentThread.interrupt(); //repositionne le flag }

19/11/2009, 11h32
tchize_
Citation:
Envoyé par L'aigle de Carthage

Code:

1 2 3 4 5 6 7 8 9 10 EspionClient thread1 = new EspionClient(); if (pageacceuil.comptclt % 2 ==0) { actclt.setText("Activer"); thread1.interrupt(); }

Avce actclt est le nom du bouton et comptclt est un compteur pour pouvoir basculer entre l'affichage de "activer" et "desactiver" sur le bouton ...
L'activation fonctionne parfaitement, mais quand je clique pour désactivé le thread, rien ne se passe et le thread continue a s'exécuté ... :calim2:
Tu balance là ton interrupt() sur un nouveau thread, alors que ce que tu dois faire, c'est l'envoyer sur le thread que t'as démarré à l'appel précédent. Pour ca, il faut maintenir une référence vers ce thread.
19/11/2009, 13h18
L'aigle de Carthage

Citation:

Envoyé par tchize_

Tu balance là ton interrupt() sur un nouveau thread, alors que ce que tu dois faire, c'est l'envoyer sur le thread que t'as démarré à l'appel précédent. Pour ca, il faut maintenir une référence vers ce thread.

Merci pour votre réponse, mais je ne sais pas comment maintenir une référence vers mon thread ... Est ce vous pouvez m'expliquez de plus? :roll:
Merci une autre fois ;)
19/11/2009, 13h47
tchize_

en le stockant, simplement, dans un champ, comme n'importe quelle donnée que tu devrais conserver entre chaque appel

Citation:
Envoyé par Kirua12
Code:

1 2 3 4 5 6 7 try { Thread.sleep(7000); } catch (InterruptedException ex) { Logger.getLogger(EspionProduit.class.getName()).log(Level.SEVERE, null, ex); Thread.currentThread.interrupt(); //repositionne le flag }
J'ai fait exactement comme vous m'avez signaler mais pas de résultat, le thread continue a travaillé quand je re-clic sur désactiver ... :calim2:
------------------
Remarque:
Le code de développement du thread et le code d'initialisation du thread sont dans 2 pages différentes ... c'est à dire je fais la classe thread EspionClient dans une page et je l'instantie dans une autre(EspionProduit thread1 = new EspionThread();)

J'ai fait aussi comme ça dans la page ou j'initialise le thread:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
 private void actcltActionPerformed(java.awt.event.ActionEvent evt) {                                       
 
        EspionClient thread1 = new EspionClient();
       thread1.start();
 
 
        if (pageacceuil.comptclt % 2 ==0)
        {
 
            actclt.setText("Activer");
            EspionClient.currentThread().interrupt();
 
        }
        else {
             actclt.setText("Desactiver");
 
            }
 
        pageacceuil.comptclt ++; 
    }

Mais toujours pas de résultat ... :(

******
J'ai procédé autrement comme ça ... mais en vin ... :( :(

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
private void actcltActionPerformed(java.awt.event.ActionEvent evt) {                                       
 
        EspionClient thread1 = new EspionClient();
       thread1.start();
       Thread t= EspionClient.currentThread();
 
 
        if (pageacceuil.comptclt % 2 ==0)
        {
 
            actclt.setText("Activer");
            t.interrupt();
 
        }
        else {
             actclt.setText("Desactiver");
            t.resume();
            }
 
        pageacceuil.comptclt ++; 
    }

19/11/2009, 20h11
tchize_

fais sortir ta variable de ta méthode! Tu dois quand même bien te rendre compte qu'à chaque appel de ta méthode tu crée un nouvel objet thread. Je dis ca puisque, visiblement, t'a déjà compris qu'il fallait mettre comptclt ailleurs, alors pourquoi tu le fais pas avec le Thread?

Mais si je fais sortir EspionClient thread1 = new EspionClient(); de ma méthode ... le thread1 ne sera pas connu au sein de la méthode et je ne peut pas faire, par la suite, thread1.start() :calim2:... et si je fais sortir thread1.start() de ma méthode aussi ... alors le thread va se lancer dés le chargement de la page .... :calim2:
Vous avez compris mon problème ....
Sinon est ce que vous pouvez m'éclaircir un peu :( ... merci

Autre chose: Est ce que je dois mettre
Code:

1 2 3 4 5 6 7 try { Thread.sleep(7000); } catch (InterruptedException ex) { Logger.getLogger(EspionProduit.class.getName()).log(Level.SEVERE, null, ex); Thread.currentThread.interrupt(); //repositionne le flag }
?
:calim2:

19/11/2009, 21h56
tchize_

Citation:

Envoyé par L'aigle de Carthage

Mais si je fais sortir EspionClient thread1 = new EspionClient(); de ma méthode ... le thread1 ne sera pas connu au sein de la méthode

Y a pas de raison que non, ta méthode à accès a ses variable locale, mais aussi aux champs de sa classes, et au champs statiques de toute classe.
19/11/2009, 22h07
L'aigle de Carthage

Regardez ce que j'ai voulu vous dire

http://img695.imageshack.us/img695/7149/thre.jpg

lien : http://img695.imageshack.us/img695/7149/thre.jpg
...
:calim2: :calim2:
19/11/2009, 23h05
L'aigle de Carthage

Une autre question: Que dois-je mettre :

1- thread1.interrupt()
oubien

2- thread1.suspend()

?

et dans le bouton activer
pourquoi la méthode resume() est barré? :aie:
20/11/2009, 01h36
tchize_

Citation:

Envoyé par L'aigle de Carthage

Regardez ce que j'ai voulu vous dire

Ton thread1 n'est pas visible de ta méthode là où tu l'a mis. Si tu ne maitrise pas le concept de visibilité des variables en java, je te conseille de commencer par les tutoriaux de bases de java, avant de jouer avec les interfaces graphiques et les threads.

inteerupt sert à réveiller immédiatement le thread, le faire sortir de ses opérations bloquantes et positionner le flag 'isInterrupted' à true. C'est donc biens ca qu'il faut utiliser pour signaler à un thread de s'interrompre.
23/11/2009, 08h45
Uther

Citation:

pourquoi la méthode resume() est barré?

Car c'est une méthode dépréciée (deprecated). Il s'agit d'une vielle méthode qu'il est déconseillé d'utiliser car elle risque de provoquer des plantage imprévisibles.
tout comme suspend(), et stop()

D'ailleurs dans ton cas, resume() est inutile car il sert a relancer un thread qui a été suspendu via suspend().