getComponents non fonctionnelle

**SpeedOverflow** · 07/10/2010, 02h15

Cette boucle devrait parcourire tout les composants d'un JInternalFrame,
puis modifier le contenu (text) après avoir trouvé le composant via son nom.

Lors de l'execution du code, aucun des controls sont mis à jour avec les nouvelles données.

J'attend vos suggestions.

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
private void findControl(String name, String text){
 
	//label1.setText("r");
 
	for (Component composant : getCreateClient().getComponents()) {
		label1.setText("r");
		composant.setEnabled(false);
 
		if (composant.getName() != null) {
 
			if (composant.getName() == name.trim()) {
				System.out.println(composant.getName());
 
				if (composant instanceof JFormattedTextField) {
					System.out.println(text.trim());	
 
					((JFormattedTextField) composant).setText(text);
				}
					break;
			}
		}
	}
}

**adiGuba** · 07/10/2010, 10h09

Salut,

Tu appelles getComponents() sur quoi précisément ? Tu as loggués les composants retournée pour voir ?

a++

**SpeedOverflow** · 07/10/2010, 10h16

Envoyé par adiGuba

Salut,

Tu appelles getComponents() sur quoi précisément ? Tu as loggués les composants retournée pour voir ?

a++

J'appel la méthode du JPanel (conteneur). Pour y lire les objets, afin de les manipuler.

AdiGuba, aurais tu une solution ?

**tchize_** · 07/10/2010, 10h18

Code :

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

composant.getName() == name.trim()

On ne test jamais l'égalité de deux strings comme ça. Il y a une différence fondamentale entre
string1 == string2
et
string1.equals(string2)

!!!

**adiGuba** · 07/10/2010, 10h20

Et ? Cela te retournes quels composants exactement ?

a++

[edit] Bien vu tchize_

**SpeedOverflow** · 07/10/2010, 10h45

Envoyé par tchize_

Code :

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

composant.getName() == name.trim()

On ne test jamais l'égalité de deux strings comme ça. Il y a une différence fondamentale entre
string1 == string2
et
string1.equals(string2)

!!!

Le première compare des valeurs dite primitives et le second compare les valeurs renvoyé par la méthode toString de l'instance indiquée. non

Composant.getName => String
name.trim() => String en supprimant les espaces des deux côtés.

Voici après modification :

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
for (Component composant : getCreateClient().getComponents()) {
			label1.setText("r");
			composant.setEnabled(false);
 
			if (!composant.getName().equals(null)) {
 
				if (composant.getName().equals(name)) {
					System.out.println(composant.getName());
 
					if (composant instanceof JFormattedTextField) {
						System.out.println(text.trim());	
 
						((JFormattedTextField) composant).setText(text);
					}
 
					break;
				}

les erreur :

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
java.lang.NullPointerException
	at view.frmPatient.findControl(frmPatient.java:278)
	at view.frmPatient.RemplireFiche(frmPatient.java:249)
	at view.frmPatient.update(frmPatient.java:300)
	at java.util.Observable.notifyObservers(Unknown Source)
	at model.Facade.setCurrentPatient(Facade.java:39)
	at controller.FieldFocusListener.focusLost(FieldFocusListener.java:32)
	at java.awt.AWTEventMulticaster.focusLost(Unknown Source)
	at java.awt.Component.processFocusEvent(Unknown Source)
	at javax.swing.JFormattedTextField.processFocusEvent(Unknown Source)
	at java.awt.Component.processEvent(Unknown Source)
	at java.awt.Container.processEvent(Unknown Source)
	at java.awt.Component.dispatchEventImpl(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.KeyboardFocusManager.redispatchEvent(Unknown Source)
	at java.awt.DefaultKeyboardFocusManager.typeAheadAssertions(Unknown Source)
	at java.awt.DefaultKeyboardFocusManager.dispatchEvent(Unknown Source)
	at java.awt.Component.dispatchEventImpl(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

**tchize_** · 07/10/2010, 10h55

Envoyé par SpeedOverflow

Le première compare des valeurs dite primitives et le second compare les valeurs renvoyé par la méthode toString de l'instance indiquée. non

Non, la premier compare les instance, la deuxième les valeurs. En l'occurrence

Code :

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

1
2
3
String s1 = new String("1234");
String s2 = new String(s1);
s1==s2; // false

Code :

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

1
2
 
			if (!composant.getName().equals(null)) {

C'est

Code :

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

if (!composant.getName()==null)

**SpeedOverflow** · 07/10/2010, 11h10

Envoyé par tchize_

Non, la premier compare les instance, la deuxième les valeurs. En l'occurrence

Code :

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

1
2
3
String s1 = new String("1234");
String s2 = new String(s1);
s1==s2; // false

C'est

Code :

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

if (!composant.getName()==null)

Le mystère reste entier !

Biensure, on compare leurs références en memoire et du faite qu'ils ne sont pas aux même endroit ils sont inégals.

malgré cela, j'ai changé la première condition.

ils n'effectue pas l'assignation

pourquoiiiiiiiiiiiiiiiiiiii !!!

Code :

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

1
2
3
4
if (composant instanceof JFormattedTextField) {
System.out.println(text.trim());	
((JFormattedTextField) composant).setText(text);
}

**sinok** · 07/10/2010, 11h45

Le fait d'appeler getComponents montre directement qu'il y a un problème d'architecture.

En effet si tu as besoin d'utiliser des composant spécifiques, tu devrais plutôt conserver des références dessus.

Utiliser getComponents pour retrouver un composant, c'est juste über crade comme façon de faire.

**SpeedOverflow** · 07/10/2010, 11h51

Envoyé par sinok

Le fait d'appeler getComponents montre directement qu'il y a un problème d'architecture.

En effet si tu as besoin d'utiliser des composant spécifiques, tu devrais plutôt conserver des références dessus.

Utiliser getComponents pour retrouver un composant, c'est juste über crade comme façon de faire.

, Donne moi une façon plus simple de desactive toutes ou parties des controls présents sur un formulaire.

ou encore de changé d'autres propriétés

en un minimun de lignes.

**tchize_** · 07/10/2010, 12h01

je suis d'accord, getComponents est aussi là pour servir. Pour trouver 1 composant -> on le garde en attribut. Pour faire un traitement généric (genre passer tout ls chamsp en lecture seul), un scan me semble raisonnable.

Maintenant, affiche chaque composnat que tu teste pour voir son nom et son type, ca te donnera une idée de quoi tu scanne. Aussi, getComponents n'est pas récursif, si t'a utilisé des panel poru aranger ton layout, tu devra aller voir dans ces sous panneaux!

**SpeedOverflow** · 07/10/2010, 12h21

Envoyé par tchize_

je suis d'accord, getComponents est aussi là pour servir. Pour trouver 1 composant -> on le garde en attribut. Pour faire un traitement généric (genre passer tout ls chamsp en lecture seul), un scan me semble raisonnable.

Maintenant, affiche chaque composnat que tu teste pour voir son nom et son type, ca te donnera une idée de quoi tu scanne. Aussi, getComponents n'est pas récursif, si t'a utilisé des panel poru aranger ton layout, tu devra aller voir dans ces sous panneaux!

Voici le résultat :

Code :

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

1
2
3
4
5
6
if (composant instanceof JFormattedTextField) {
						System.out.println(text.trim());	
						System.out.println("Name : " + composant.getName() + " CN : " + composant.getClass().getCanonicalName());
 
						((JFormattedTextField) composant).setText(text);
					}

la console :

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
PatientExistant
Nom
Bruggeman
Name : Nom CN : javax.swing.JFormattedTextField
Prenom
Raphaël
Name : Prenom CN : javax.swing.JFormattedTextField
Adresse
XXXXXXXXXXXXXXXX
Name : Adresse CN : javax.swing.JFormattedTextField
eMail
bruggeman.raphael@hotmail.com
Name : eMail CN : javax.swing.JFormattedTextField
Conjoint
XXXXXXXXXX
Name : Conjoint CN : javax.swing.JFormattedTextField

**tchize_** · 07/10/2010, 13h18

je pense qu'il effectue bien l'assignation, mais que tu scanne des composant qui ne sont pas ceux affiché à l'écran. Fait des getText() pour t'en convaincre. Je suppose que getCreateClient() renvoie une instance différente...

**SpeedOverflow** · 07/10/2010, 15h02

Envoyé par tchize_

je pense qu'il effectue bien l'assignation, mais que tu scanne des composant qui ne sont pas ceux affiché à l'écran. Fait des getText() pour t'en convaincre. Je suppose que getCreateClient() renvoie une instance différente...

Effectivement, je créai une seconde instance à l'appel de la méthode getCreateClient() .

En très grand merci, tchize.

getComponents non fonctionnelle

Java

Discussions similaires

Partager

Partager