JTextField tout le temps vide ?

Version imprimable

Bonsoir,
J'ai un programme qui demande à l'utilisateur son nom et son mot de passe dans un JTextField et des JPasswordField. Le problème est quand je veux récupérer le nom de l'utilisateur, la chaine est toujours vide...
Voilà des bouts de code :

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
public class Fenetre extends JFrame implements ActionListener {
	private Container container;
 
	private JLabel labelNom;
	private JLabel labelVieuxMdp;
	private JLabel labelNouveauMdp;
	private JLabel labelConfirmerMdp;
 
	private JTextField fieldNom;
	private JPasswordField fieldVieuxMdp;
	private JPasswordField fieldNouveauMdp;
	private JPasswordField fieldConfirmerMdp;
 
	private JButton buttonLancer;
 
[ ... ]
private Container construireFenetre() {
		container = new JPanel();
		container.setLayout(new GridBagLayout());
		GridBagConstraints contraintes = new GridBagConstraints();
 
		labelNom = new JLabel();
		labelNom.setText("Nom");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 0;
		contraintes.gridy = 0;
		container.add(labelNom, contraintes);
 
		fieldNom = new JTextField(25);
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 1;
		contraintes.gridy = 0;
		container.add(fieldNom, contraintes);
 [ ... ]
 
public String getNom() {
		return fieldNom.getText();
	}
public void actionPerformed(ActionEvent e) {
	System.out.println("nom : "+getNom()); // Marche pas !!
}

Merci de m'aider !

21/03/2009, 00h47
jiddou

Jusqu'à maintenant c'est correct mais le code n'est pas complet et le bug est ailleurs donc donne le reste de la classe.
21/03/2009, 05h30
erictoguem

Le bug doit se trouver un peu plus loin.

jiddou a raison, le bug doit se trouver un peu plus loin dans ton code.

Ok voilà le code de la classe :
Merci

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
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
package fr.unice.ducot.gui;
 
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
import javax.swing.*;
 
public class Fenetre extends JFrame implements ActionListener {
	private Container container;
 
	private JLabel labelNom;
	private JLabel labelVieuxMdp;
	private JLabel labelNouveauMdp;
	private JLabel labelConfirmerMdp;
 
	private JTextField fieldNom;
	private JPasswordField fieldVieuxMdp;
	private JPasswordField fieldNouveauMdp;
	private JPasswordField fieldConfirmerMdp;
 
	private JButton buttonLancer;
 
 
 
	public Fenetre() {
		super();
		setTitle("Changer mot de passe");
		setSize(350,150);
		setLocation(200,200);
		setVisible(true);
		setContentPane(construireFenetre());
	}
 
	public static void main(String[] args) {
		SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				Fenetre f = new Fenetre();
				f.construireFenetre();
			}
		});
	}
 
	public void actionPerformed(ActionEvent e) {
		System.out.println("nom : "+getNom());
	}
 
	private Container construireFenetre() {
		container = new JPanel();
		container.setLayout(new GridBagLayout());
		GridBagConstraints contraintes = new GridBagConstraints();
 
		labelNom = new JLabel();
		labelNom.setText("Nom");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 0;
		contraintes.gridy = 0;
		container.add(labelNom, contraintes);
 
		fieldNom = new JTextField(25);
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 1;
		contraintes.gridy = 0;
		container.add(fieldNom, contraintes);
 
		labelVieuxMdp = new JLabel();
		labelVieuxMdp.setText("Mot de passe");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 0;
		contraintes.gridy = 1;
		container.add(labelVieuxMdp, contraintes);
 
		fieldVieuxMdp = new JPasswordField();
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 1;
		contraintes.gridy = 1;
		container.add(fieldVieuxMdp, contraintes);
 
		labelNouveauMdp = new JLabel();
		labelNouveauMdp.setText("Nouveau mot de passe");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 0;
		contraintes.gridy = 2;
		container.add(labelNouveauMdp, contraintes);
 
		fieldNouveauMdp = new JPasswordField();
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 1;
		contraintes.gridy = 2;
		container.add(fieldNouveauMdp, contraintes);
 
		labelConfirmerMdp = new JLabel();
		labelConfirmerMdp.setText("Retapez le mot de passe");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 0;
		contraintes.gridy = 3;
		container.add(labelConfirmerMdp, contraintes);
 
		fieldConfirmerMdp = new JPasswordField();
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 1;
		contraintes.gridy = 3;
		container.add(fieldConfirmerMdp, contraintes);
 
		buttonLancer = new JButton();
		buttonLancer.setText("Lancer la modification");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.0;
		contraintes.gridwidth = 2;
		contraintes.gridx = 0;
		contraintes.gridy = 4;
		buttonLancer.addActionListener(this);
		container.add(buttonLancer, contraintes);
 
		return container;
	}
 
	public String getNom() {
		fieldNom.setText("salut"); 
		return fieldNom.getText();
	}
 
	public String getVieuxMdp() {
		return new String(fieldVieuxMdp.getPassword());
	}
 
	public String getNouveauMdp() {
		return new String(fieldNouveauMdp.getPassword());
	}
 
	public String getConfirmerMdp() {
		return new String(fieldConfirmerMdp.getPassword());
	}
 
 
}

Premier problème :

Code:

1
2
3
4
5
 
public void run() {
	Fenetre f = new Fenetre();
	//f.construireFenetre(); 
	}

pas la peine de mettre f.construireFenetre(); puisque tu l'appelles dans le constructeur ainsi:

Code:

1
2
 
setContentPane(construireFenetre());

tu enlèves ton setText qui se trouve dans getNom et le code qui en resulte est le suivant.il n' y a plus de bug:

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
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
 
package fr.unice.ducot.gui;
 
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
import javax.swing.*;
 
public class Fenetre extends JFrame implements ActionListener {
	private Container container;
 
	private JLabel labelNom;
	private JLabel labelVieuxMdp;
	private JLabel labelNouveauMdp;
	private JLabel labelConfirmerMdp;
 
	private JTextField fieldNom;
	private JPasswordField fieldVieuxMdp;
	private JPasswordField fieldNouveauMdp;
	private JPasswordField fieldConfirmerMdp;
 
	private JButton buttonLancer;
 
 
 
	public Fenetre() {
		super();
		setTitle("Changer mot de passe");
		setSize(350,150);
		setLocation(200,200);
		setVisible(true);
		setContentPane(construireFenetre());
	}
 
	public static void main(String[] args) {
		SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				Fenetre f = new Fenetre();
			}
		});
	}
 
	public void actionPerformed(ActionEvent e) {
		System.out.println("nom : "+getNom());
	}
 
	private Container construireFenetre() {
		container = new JPanel();
		container.setLayout(new GridBagLayout());
		GridBagConstraints contraintes = new GridBagConstraints();
 
		labelNom = new JLabel();
		labelNom.setText("Nom");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 0;
		contraintes.gridy = 0;
		container.add(labelNom, contraintes);
 
		fieldNom = new JTextField(25);
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 1;
		contraintes.gridy = 0;
		container.add(fieldNom, contraintes);
 
		labelVieuxMdp = new JLabel();
		labelVieuxMdp.setText("Mot de passe");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 0;
		contraintes.gridy = 1;
		container.add(labelVieuxMdp, contraintes);
 
		fieldVieuxMdp = new JPasswordField();
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 1;
		contraintes.gridy = 1;
		container.add(fieldVieuxMdp, contraintes);
 
		labelNouveauMdp = new JLabel();
		labelNouveauMdp.setText("Nouveau mot de passe");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 0;
		contraintes.gridy = 2;
		container.add(labelNouveauMdp, contraintes);
 
		fieldNouveauMdp = new JPasswordField();
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 1;
		contraintes.gridy = 2;
		container.add(fieldNouveauMdp, contraintes);
 
		labelConfirmerMdp = new JLabel();
		labelConfirmerMdp.setText("Retapez le mot de passe");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 0;
		contraintes.gridy = 3;
		container.add(labelConfirmerMdp, contraintes);
 
		fieldConfirmerMdp = new JPasswordField();
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 1;
		contraintes.gridy = 3;
		container.add(fieldConfirmerMdp, contraintes);
 
		buttonLancer = new JButton();
		buttonLancer.setText("Lancer la modification");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.0;
		contraintes.gridwidth = 2;
		contraintes.gridx = 0;
		contraintes.gridy = 4;
		buttonLancer.addActionListener(this);
		container.add(buttonLancer, contraintes);
 
		return container;
	}
 
	public String getNom() {
		return fieldNom.getText();
	}
 
	public String getVieuxMdp() {
		return new String(fieldVieuxMdp.getPassword());
	}
 
	public String getNouveauMdp() {
		return new String(fieldNouveauMdp.getPassword());
	}
 
	public String getConfirmerMdp() {
		return new String(fieldConfirmerMdp.getPassword());
	}
 
 
}

N'oublies pas le TAG RESOLU

21/03/2009, 12h54
vincbobor

Autant pour moi j'ai pas relu ce que j'ai posté
f.construireFenetre est en commentaire donc pas utilisé
et le setText("salut") est là juste pour tester. Dans ce cas le getNom() renvoie toujours "salut" même si je rajoute du texte (d'ailleurs "salut" ne s'affiche pas dans le JTextField), sans le setText() j'ai la chaine vide...

Le code marche bien chez moi (sous mon netbeans :D). J'ai juste mis setContentPane(construireFenetre()); avant le this .setVisible() dans le constructeur.

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
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
 
package fr.unice.ducot.gui;
 
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
import javax.swing.*;
 
public class Fenetre extends JFrame implements ActionListener {
	private Container container;
 
	private JLabel labelNom;
	private JLabel labelVieuxMdp;
	private JLabel labelNouveauMdp;
	private JLabel labelConfirmerMdp;
 
	private JTextField fieldNom;
	private JPasswordField fieldVieuxMdp;
	private JPasswordField fieldNouveauMdp;
	private JPasswordField fieldConfirmerMdp;
 
	private JButton buttonLancer;
 
 
 
	public Fenetre() {
		super();
		setTitle("Changer mot de passe");
		setSize(350,150);
		setLocation(200,200);
        setContentPane(construireFenetre());
		setVisible(true);
 
	}
 
	public static void main(String[] args) {
		SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				Fenetre f = new Fenetre();
				f.construireFenetre();
			}
		});
	}
 
	public void actionPerformed(ActionEvent e) {
		System.out.println("nom : "+getNom());
	}
 
	private Container construireFenetre() {
		container = new JPanel();
		container.setLayout(new GridBagLayout());
		GridBagConstraints contraintes = new GridBagConstraints();
 
		labelNom = new JLabel();
		labelNom.setText("Nom");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 0;
		contraintes.gridy = 0;
		container.add(labelNom, contraintes);
 
		fieldNom = new JTextField(25);
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 1;
		contraintes.gridy = 0;
		container.add(fieldNom, contraintes);
 
		labelVieuxMdp = new JLabel();
		labelVieuxMdp.setText("Mot de passe");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 0;
		contraintes.gridy = 1;
		container.add(labelVieuxMdp, contraintes);
 
		fieldVieuxMdp = new JPasswordField();
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 1;
		contraintes.gridy = 1;
		container.add(fieldVieuxMdp, contraintes);
 
		labelNouveauMdp = new JLabel();
		labelNouveauMdp.setText("Nouveau mot de passe");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 0;
		contraintes.gridy = 2;
		container.add(labelNouveauMdp, contraintes);
 
		fieldNouveauMdp = new JPasswordField();
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 1;
		contraintes.gridy = 2;
		container.add(fieldNouveauMdp, contraintes);
 
		labelConfirmerMdp = new JLabel();
		labelConfirmerMdp.setText("Retapez le mot de passe");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 0;
		contraintes.gridy = 3;
		container.add(labelConfirmerMdp, contraintes);
 
		fieldConfirmerMdp = new JPasswordField();
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.5;
		contraintes.gridx = 1;
		contraintes.gridy = 3;
		container.add(fieldConfirmerMdp, contraintes);
 
		buttonLancer = new JButton();
		buttonLancer.setText("Lancer la modification");
		contraintes.fill = GridBagConstraints.HORIZONTAL;
		contraintes.weightx = 0.0;
		contraintes.gridwidth = 2;
		contraintes.gridx = 0;
		contraintes.gridy = 4;
		buttonLancer.addActionListener(this);
		container.add(buttonLancer, contraintes);
 
		return container;
	}
 
	public String getNom() {
		return fieldNom.getText();
	}
 
	public String getVieuxMdp() {
		return new String(fieldVieuxMdp.getPassword());
	}
 
	public String getNouveauMdp() {
		return new String(fieldNouveauMdp.getPassword());
	}
 
	public String getConfirmerMdp() {
		return new String(fieldConfirmerMdp.getPassword());
	}
 
 
}

21/03/2009, 17h07
jiddou
C'est normal qu'il t'affiche salut puisque tu fais
Code:

1 2 setText('salut'); return textField.gettext();
donc le setText écrase ce que tu as tappé dans ton textfield.
21/03/2009, 18h21
vincbobor

darrylsite > j'ai testé sous netbeans ça ne marche pas non plus, tu as le texte que tu as rentré dans "Nom" qui s'affiche ?

jiddou > ok on enlève le setText(), mon JTextField est toujours vide...
21/03/2009, 19h48
jiddou

Alors là je ne comprends plus: Ton problème c'etait.

Citation:

Le problème est quand je veux récupérer le nom de l'utilisateur, la chaine est toujours vide...

Et maintenant que getNom te retourne ce que tu as tappé dans Nom

Citation:

darrylsite > j'ai testé sous netbeans ça ne marche pas non plus, tu as le texte que tu as rentré dans "Nom" qui s'affiche ?

Et tu dis qu'il y a toujours problème.Je ne sais pas quoi te dire mon ami mais je suis perdu.
21/03/2009, 20h11
vincbobor

je n'ai jamais dit que getNom() me renvoyait le nom, il me renvoie toujours la chaine vide... (je demandais juste à darrylsite si ça marchait bien chez lui)
21/03/2009, 20h38
darrylsite

Chez moi, ça marche super bien avec le code que j'ai posté. Je ne vois pas de raison pour que ça ne marche pas. A moins que tu n'as pas tapé le nom dans la case reservée
21/03/2009, 20h55
jiddou

Chez moi ça marche très bien aussi.
Je te propose de nous envoyer à nouveau le code que tu utilises et de nous dire qu'est ce que tu attends de lui exactement.
22/03/2009, 00h09
vincbobor

Mon but que si on écrit "toto" dans le champ Nom et que l'on clique sur le bouton "Lancer la modification", il s'affiche sur la console "Nom : toto".
Ce n'est pas le cas pour moi, il ne s'affiche que "Nom : " (même avec le code de darrylsite).
Merci de votre patience ;)
22/03/2009, 03h14
jiddou

mais avec le setText("salut") il affiche "Nom : salut" ?
22/03/2009, 11h48
vincbobor

Oui c'est ça
22/03/2009, 13h58
jiddou

Essaies de créer une interface à part dans la quelle il y a une zone de texte dont tu affiches le contenu en cliquant sur un bouton.Je sais c'est pas logique parceque c'est la même chose que tu avais faite mais bon c'est illogique tout ça
22/03/2009, 20h14
vincbobor

Ok j'ai eu ma réponse.
Ma fenêtre est en fait construite 2 fois, j'appelle construireFenetre() dans le main et dans le constructeur de la classe Fenetre...
Tss...
Merci en tout cas
22/03/2009, 23h12
jiddou

mais on pensez que c'était régler puisque quand on t'as dis :

Citation:

pas la peine de mettre f.construireFenetre(); puisque tu l'appelles dans le constructeur ainsi:

Code:

setContentPane(construireFenetre());

t'as repondu:

Citation:

Autant pour moi j'ai pas relu ce que j'ai posté
f.construireFenetre est en commentaire donc pas utilisé

En plus on t'avais remis le code corrigé.
Content pour toi en tout cas mais faut faire attention prochainement.