Utilisation des final

**moomba** · 29/07/2008, 17h39

Bonjour

je cherche à compiler le code suivant :

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
import java.lang.*;
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.event.*;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.image.BufferedImage;
import java.awt.image.*;
import java.awt.geom.AffineTransform;
import java.io.File;
import java.io.IOException; 
import javax.imageio.ImageIO;
import java.text.*; 

class FenetreDonnees extends JFrame
{
	private JPanel pan_back = new JPanel();
		private JPanel pan_back_haut = new JPanel();
			private JPanel pan_back_haut_coupe = new JPanel();
		private JPanel pan_back_bas = new JPanel();
			private JPanel pan_consommation = new JPanel();
				private JScrollPane scp_consommation = new JScrollPane();
			private JPanel pan_benefices = new JPanel();
			private JPanel pan_stocks = new JPanel();
			private JPanel pan_buttons = new JPanel();
				
	//private JLabel [] lb_ressources_scroll;
	//private JLabel [] lb_benefices_scroll;
	//private JTextField [][] jtf_consommation_scroll;
	private JLabel lb_infos = new JLabel("Remplissez les zones de texte, puis cliquez sur valider.");
		
	private JButton bt_valider = new JButton("Valider");
	private JButton bt_reset = new JButton("Reset");
		
	private int nb_produits;
	private int nb_ressources;
		
	private JTable table_consommation;
		private Object[][] data_consommation;
	private JTable table_stocks;
		private Object[][] data_stocks;
	private JTable table_benefices;
		private Object[][] data_benefices;
			
	private JOptionPane question;
			
	FenetreDonnees(int nb_produits,int nb_ressources)
	{
		System.out.println(nb_produits);
		System.out.println(nb_ressources);
		setTitle("Insertion des donnees");
		
		
		//initialisation du dictionnaire
		//jtf_consommation_scroll = new JTextField [nb_produits][nb_ressources];
		
		//GridLayout dico = new GridLayout(nb_ressources+1,nb_produits+1);
		//GridLayout dico = new GridLayout(5,5);
		//scp_consommation.setLayout(dico);
		//scp_consommation.setLayout(new GridLayout(nb_produits,nb_ressources));
		//scp_consommation.applyComponentOrientation(ComponentOrientation.LEFT_TO_RIGHT);
		/*
		for(int y=1;y<=nb_ressources;y++)
		{
			for(int x=1;x<=nb_produits;x++)
			{
				scp_consommation.add(jtf_consommation_scroll[x][y]);
			}
		}*/

		
		//mise en place des grilles
		pan_back.setLayout(new BorderLayout());
		pan_back_haut.setLayout(new BorderLayout());
		pan_back_bas.setLayout(new BorderLayout());
		pan_consommation.setLayout(new BorderLayout());
		pan_benefices.setLayout(new BorderLayout());
		pan_stocks.setLayout(new BorderLayout());
		pan_buttons.setLayout(new BorderLayout());
		pan_back_haut_coupe.setLayout(new BorderLayout());
		
		pan_back.setBorder(BorderFactory.createEmptyBorder(10,10,10,10));
		pan_back_haut.setBorder(BorderFactory.createEmptyBorder(10,10,10,10));
		pan_back_bas.setBorder(BorderFactory.createEmptyBorder(10,10,10,10));
		
		pan_consommation.setBorder(BorderFactory.createCompoundBorder(BorderFactory.createTitledBorder("Consommations : "), BorderFactory.createEmptyBorder(5,5,5,5)));
		pan_benefices.setBorder(BorderFactory.createCompoundBorder(BorderFactory.createTitledBorder("Benefices des produit : "), BorderFactory.createEmptyBorder(5,5,5,5)));
		pan_stocks.setBorder(BorderFactory.createCompoundBorder(BorderFactory.createTitledBorder("Stocks des ressource : "), BorderFactory.createEmptyBorder(5,5,5,5)));
		
		data_consommation = new Object[nb_ressources][nb_produits];
		table_consommation = new JTable(nb_ressources+1,nb_produits+1);	
		
		for(int x=1;x<=nb_produits;x++)
		{
			table_consommation.setValueAt("Pr. "+x,0,x);
		}
		for(int y=1;y<=nb_ressources;y++)
		{
			table_consommation.setValueAt("Re. "+y,y,0);
		}
		
		data_stocks = new Object[nb_ressources][2];
		table_stocks = new JTable(nb_ressources,2);
		
		for(int y=0;y<=nb_ressources-1;y++)
		{
			table_stocks.setValueAt("St. Re. "+y,y,0);
		}
		
		data_benefices = new Object[nb_produits][2];
		table_benefices = new JTable(nb_produits,2);
		
		for(int x=0;x<=nb_produits-1;x++)
		{
			table_benefices.setValueAt("Be. Pro. "+x,x,0);
		}
		
		pan_consommation.add(table_consommation,BorderLayout.CENTER);
		pan_stocks.add(table_stocks,BorderLayout.CENTER);
		pan_benefices.add(table_benefices,BorderLayout.CENTER);
		
		pan_back_haut_coupe.add(pan_benefices,BorderLayout.EAST);
		pan_back_haut_coupe.add(pan_stocks,BorderLayout.WEST);
		
		pan_back.add(lb_infos,BorderLayout.NORTH);
		pan_back_haut.add(pan_consommation,BorderLayout.NORTH);
		pan_back_haut.add(pan_back_haut_coupe,BorderLayout.CENTER);
		pan_buttons.add(bt_valider,BorderLayout.WEST);
		pan_buttons.add(bt_reset,BorderLayout.EAST);
		pan_back_bas.add(pan_buttons,BorderLayout.CENTER);
		pan_back.add(pan_back_haut,BorderLayout.CENTER);
		pan_back.add(pan_back_bas,BorderLayout.SOUTH);
		
		setSize(400,542);
		
		setContentPane(pan_back);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setResizable(false);
		setVisible(true);
		
		bt_reset.addActionListener(new ActionListener()
		{
			public void actionPerformed(ActionEvent event)
			{
				question = new JOptionPane();
				if(JOptionPane.showConfirmDialog(null,"Etes vous sur ?", "Attention", JOptionPane.YES_NO_OPTION)==0)
				{
					for(int x=1;x<=nb_produits;x++)
					{
						for(int y=1;y<=nb_ressources;y++)
						{
							table_consommation.setValueAt("",y,x);
						}
					}
					for(int y=0;y<=nb_ressources-1;y++)
					{
						table_stocks.setValueAt("",y,1);
					}
					for(int x=0;x<=nb_produits-1;x++)
					{
						table_benefices.setValueAt("",x,1);
					}
				}
				
			}		});
		
	}
}

Je récupère l'erreur suivante :

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
.\FenetreDonnees.java:149: local variable nb_produits is accessed from within in
ner class; needs to be declared final
                                        for(int x=1;x<=nb_produits;x++)
                                                       ^
.\FenetreDonnees.java:151: local variable nb_ressources is accessed from within
inner class; needs to be declared final
                                                for(int y=1;y<=nb_ressources;y++
)
                                                               ^
.\FenetreDonnees.java:156: local variable nb_ressources is accessed from within
inner class; needs to be declared final
                                        for(int y=0;y<=nb_ressources-1;y++)
                                                       ^
.\FenetreDonnees.java:160: local variable nb_produits is accessed from within in
ner class; needs to be declared final
                                        for(int x=0;x<=nb_produits-1;x++)

Ilf aut donc que j'utilise des variables constantes, c'est ça ? mais dans ce cas, comment puis je les récupérer dans le constructeur si elles doievnt être fixées avant la création de la classe ?

J'avoue ne pas trop comprendre le problème...

Quelqu'un pourrait-il maider ?

**verbose** · 29/07/2008, 17h58

Tu as deux variables nb_produits, l'une est une variable locale correspondant à un paramètre de ton constructeur, l'autre un attribut de la classe.

Dans une classe anonyme, tu ne peux pas accéder à une variable locale, seulement à des constantes (final). Cela vient du fait qu'une classe sert à créer des objets dont la durée de vie dépasse la porté de la méthode dans laquelle elles sont crées. Il faut donc créer une closure avec le mot-clé final pour que la constante soit rattachée aux classes qui y accèdent.

Mais dans ton cas, je pense que tu devrais plutôt accéder à la variable d'instance, je suppose que c'est son rôle. Tu dois donc faire deux choses :
*initialiser cet attribut dans le constructeur :

Code :

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

1
2
3
4
5
6
 
private int nb_produits;
FenetreDonnees(int nb_produits,int nb_ressources) {
	this.nb_produits = nb_produits
	[...]
}

* et accéder à la variable d'instance à la place de la variable locale dans ta classe anonyme :

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
bt_reset.addActionListener(new ActionListener()
{
	public void actionPerformed(ActionEvent event)
	{
		[...]
		for(int x=1;x<=FenetreDonnees.this.nb_produits;x++)
		{
			for(int y=1;y<=FenetreDonnees.this.nb_ressources;y++)
			{
				table_consommation.setValueAt("",y,x);
			}
		}
		[...]
});

[/CODE]

**moomba** · 29/07/2008, 19h09

Merci pour ta réponse si complète verbose.

Effectivement, comme ça, ça marche.

J'avoue être étonné (je suis débutant

), si j'initialise une variable avant le constructeur de la classe, c'est à dire par exemple :

Code :

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

private int nb_produits;

Et qu'ensuite je passe dans le constructeur une variable du même nom :

Code :

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

1
2
public class FenetreDonnee(int nb_produits)
{...}

Ce ne sera pas la même variable au final ?? Je pensai que le compilateur donnerai la valeur donnée dans le constructeur à la variable initialisée avant. Mais je n'ai peut être rien compris en fait

Autre question, comment est-il possible de poser dans l'action listener ceci:

Code :

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

FenetreDonnees.this.nb_produits

Je pensai qu'il n'était possible que de faire appel à ce qui était "en dessous" dans l'ordre de création, c'est à dire par exemple que ici, la classe FenetreDonnee à été créée par la classe FenetreAcceuil, puis le action listener à été créé par FenetreDonnee, donc FenetreAccueil peut acceder aux deux, FenetreDonnee ne peut accéder qu'au action listener, et ce dernier, bas il n'accède qu'à lui même

J'apprend en auto-didact, il est donc possible que je n'ai rien compris...

**welcome_59** · 29/07/2008, 22h53

Juste un petit élément de langage:

Code :

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

private int nb_produits;

n'est pas une initialisation, mais une déclaration (private int nb_produits = 10; serait une initialisation); c'était juste une parenthèse.

Code :

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

1
2
public class FenetreDonnee(int nb_produits)
{...}

Ce ne sera pas la même variable au final ??

Non ce ne sera pas la même variable.

L'attribut private int nb_produits est une variable d'instance et vit sur le tas (heap), alors que celui que tu déclares et manipules dans le constructeur est une variable locale qui elle vit dans la pile (stack).

Je pensai qu'il n'était possible que de faire appel à ce qui était "en dessous" dans l'ordre de création

Tu veux probablement dire dans l'ordre de déclaration.

En Java, l'utilisation des seules variables locales (et dans une certaine mesure les blocs static) obéit à l'ordre de déclaration (c'est à dire que tu ne peux pas utiliser une variable locale avant de l'avoir déclarée et initialisée).

Une variable d'instance (attribut) est encapsulée dans la classe qui est compilée. Peu importe donc l'ordre de déclaration puisque la classe sera compilée et chargée lorsque la méthode se retrouvera en mémoire.

**moomba** · 29/07/2008, 23h29

C'est une réponse ma foi trés claire

J'ai regardé sur le web les stack et heap et comprend mieux le principe. Je devrai moins me perdre maintenant

Veuillez accepter tous les deux mes sincères remerciements

**tchize_** · 30/07/2008, 00h05

Envoyé par welcome_59

Juste un petit élément de langage:

Code :

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

private int nb_produits;

n'est pas une initialisation, mais une déclaration (private int nb_produits = 10; serait une initialisation); c'était juste une parenthèse.

Je vais être vachement chien là mais, en fait, techniquement, c'est aussi une initialisation, les champs d'une classes, contrairement au variable locale, ont une valeur par défaut (false pour les boolean, 0 pour les int, null pour les object, etc :p) çà évite de se coltiner une tartine de = dans les déclarations ou d'etre forcé de tout initialiser dans le constructeur

Donc çà:

Code :

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

int x;

dans un bloc de code, c'est une déclaration.
çà, dans une classe

Code :

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

private int x;

c'est une déclaration avec initialisation à la valeur par défaut et çà

Code :

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

private int x=69;

C'est une déclaration avec initialisation à l'intégrale calculée entre 13 et 10 de 2xdx.
Voilà, ct ma parenthèse à moi :p

**moomba** · 30/07/2008, 09h35

C'est vrai que le Java est pas mal pour ça, pas besoin de mettre tout le monde à zéro.

Merci pour ces précisions

**welcome_59** · 30/07/2008, 09h52

techniquement, c'est aussi une initialisation, les champs d'une classes, contrairement au variable locale, ont une valeur par défaut (false pour les boolean, 0 pour les int, null pour les object, etc :p)

Entièrement d'accord.

Utilisation des final

avec Java

Discussions similaires

Partager

Partager