classe interne à declarer final

Version imprimable

Bonjour, je suis en train de voir ce que l'on peut faire avec les Threads
et j'ai ajouté rapidement une interface graphique pour voir ce qui se passait lorsque je lancais un Thread à la main au milieu d'autre que j'avais lancé par programme.

Donc j'ai vite fait, une interface graphique avec un bouton qui appel un objet (en l'occurence un Thread), mais là ca bloque à la compilation sur un probleme de classe interne. Alors que ma classe unObjet est une classe à l'exterieur de la premiere et qui est compilée.
Code:

1 2 3 4 5 6 7 8 9 10 C:\Java\ObjetThread>javac unObjet.java unObjet.java:58: local variable obj2 is accessed from within inner class; needs to be declared final obj4 = new unObjet("obj4"); ^ unObjet.java:59: local variable obj2 is accessed from within inner class; needs to be declared final obj4.start(); ^ 2 errors
Je mets le code du main()
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 public static void main(String args[]) { unObjet obj1; unObjet obj2; monObjet obj4; obj1 = new unObjet("obj1"); obj2 = new unObjet("obj2"); obj1.start(); obj2.start(); JFrame uneFrame = new JFrame("test"); JPanel unJpanel = new JPanel(); uneFrame.setContentPane(unJpanel); JButton lebouton = new JButton("interruption"); unJpanel.add(lebouton); uneFrame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE); uneFrame.pack(); uneFrame.setVisible(true); lebouton.addActionListener(new ActionListener(){ public void actionPerformed(ActionEvent e) { obj4 = new monObjet("obj4"); obj4.start(); }}); } }
D'avance merci de vos idées

14/03/2006, 17h20
BugFactory

Placez public void actionPerformed(ActionEvent e) à l'extérieur de main.
14/03/2006, 17h24
sinok

Citation:

Envoyé par BugFactory

Placez public void actionPerformed(ActionEvent e) à l'extérieur de main.

Pour faire propre oui(pas bien de coder quoi que ce soit dans un main....), mais dans le cas présent la solution se trouvera dans le fait de déclarer obj4 en final

Code:

final monObjet obj4;

, mais bon ce n'est pas la panacée non plus

Bon du coup j'ai mis mon code du Bouton à l'exterieur dans une nouvelle classe, mais je ne parviens pas à l'instancier. Je dois faire quelque chose de tres mal encore. Mais bon l'autoformation c'est pas le top...
le message d'erreur :

Citation:

C:\Java\ObjetThread>javac unObjet.java
unObjet.java:60: non-static variable this cannot be referenced from a static context obj5 =new threadBouton();
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 public class threadBouton { monObjet obj4; threadBouton () { JFrame uneFrame = new JFrame("test"); JPanel unJpanel = new JPanel(); uneFrame.setContentPane(unJpanel); JButton lebouton = new JButton("interruption"); unJpanel.add(lebouton); uneFrame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE); uneFrame.pack(); uneFrame.setVisible(true); lebouton.addActionListener(new ActionListener(){ public void actionPerformed(ActionEvent e) { obj4 = new monObjet("obj4"); obj4.start(); }}); } } public static void main(String args[]) { unObjet obj1; unObjet obj2; threadBouton obj5; [b]obj5 =new threadBouton();[/b] obj1 = new unObjet("obj1"); obj2 = new unObjet("obj2"); obj1.start(); obj2.start(); } }

14/03/2006, 18h22
sinok

un fichier java ne peut contenir qu'un seule classe: celle qui porte le même nom que le fichier, ensuite dans ta classe tu pourras créer des inner class, mais pas accéder à celles cis dans le main

Fais voir l'intégralité de ton code, car il semble que tu ais besoin d'un peu de révisions en java

Comme demandé l'intégralité du code.
D'avance merci de vos réponses.

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
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
 
 
// modelisation d un objet avec des Threads
public class unObjet extends Thread
{
private int var1, var2, var3;
private String obj;
public unObjet(String obj)
{
this.obj=obj;
var1=0;
System.out.println ("je suis un objet"+this.obj);
}
 
public void run()
{
try{
  for (int i=0;i<10;i++)
  {
  this.var1++;
  sleep(1000);
  System.out.println("objet="+this.var1+this.obj);
  }
}
catch (InterruptedException e) {}
 
}
 
public class threadBouton
{
  monObjet obj4;
  threadBouton ()
  {
  JFrame uneFrame = new JFrame("test");
  JPanel unJpanel = new JPanel();
  uneFrame.setContentPane(unJpanel);
  JButton lebouton = new JButton("interruption");
  unJpanel.add(lebouton);
 
  uneFrame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE); 
  uneFrame.pack();
  uneFrame.setVisible(true);
  lebouton.addActionListener(new ActionListener(){
  public void actionPerformed(ActionEvent e) 
  {
  obj4 = new monObjet("obj4");
  obj4.start();
  }});
  }
}
public static void main(String args[])
{
  unObjet obj1;
  unObjet obj2;
  threadBouton obj5;
  obj5 =new threadBouton();
  obj1 = new unObjet("obj1");
  obj2 = new unObjet("obj2");
  obj1.start();
  obj2.start();
}
}

et dans un autre fichier...

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
 
public class monObjet extends Thread
{
private int var1, var2, var3;
private String obj;
public monObjet(String obj)
{
this.obj=obj;
var1=0;
System.out.println ("je suis un objet"+this.obj);
}
public void run()
{
try{
  for (int i=0;i<10;i++)
  {
  this.var1++;
  sleep(5);
  System.out.println("objet="+this.var1+this.obj);
  }
}
catch (InterruptedException e) {}
 
}
}

14/03/2006, 18h44
Satch

Citation:

Envoyé par sinok

un fichier java ne peut contenir qu'un seule classe: celle qui porte le même nom que le fichier, ensuite dans ta classe tu pourras créer des inner class, mais pas accéder à celles cis dans le main

Fais voir l'intégralité de ton code, car il semble que tu ais besoin d'un peu de révisions en java

désolé sinok mais tu as aussi besoin de révision :p
Il peut y avoir plusieurs classes dans un fichier .java. Mais une seule doit être publique et porter le même nom que le fichier.
14/03/2006, 18h55
Satch

Le problème vient du fait que tu veux instancier ta classe interne threadButton.

Pour instancier une classe interne il faut avoir une instance de la classe englobante.
Etant dans un contexte static (main est static), tu n'as pas d'instance de unObjet.

il faudrait donc faire ça :

Code:

obj5 = new unObjet().new threadButton();

PS : par convention, les noms des classes commencent par une majuscule.
14/03/2006, 19h10
comme de bien entendu

Merci ca marche, quel talent. :lol:
15/03/2006, 11h30
sinok

Désolé pour la faille, mais bon, mieux vaut éviter la surcharge de code dans un fichier (enfin c'est que je pense donc je morcelle au plus possible)
Bon sur ce je vais retourner me planquer