Discussion :

[_KB_]

Hello,

J'ai une classe "UI" qui dérive de JPanel. Je souhaite que toutes mes classes filles possèdent un JCheckBox qui permet d'activer ou désactiver tous les composants qui seront insérés par les constructeurs respectifs.

Pour se faire j'ai eu l'idée de redéfinir les méthodes "add" et "setLayout" entre autre afin de pouvoir continuer à utiliser ces méthodes directement (this) sans passer par un getter qui me permettrait de reccupérer le JPanel à modifier. (configPanel ici en l'occurence).

Le problème est dans la gestion des layouts, j'ai un nullpointer exception...

Voici le code :

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
public class UI extends JPanel implements Constants{
	private JPanel configPanel;
	
	public UI(String  panelName) {
		this.setBorder(BorderFactory.createTitledBorder(Interface.blackline, ""));
		JCheckBox enableConfigPanel = new JCheckBox("Configure "+panelName);
		configPanel = new JPanel();
		super.setLayout(new BorderLayout());		
super.add(enableConfigPanel, BorderLayout.NORTH);
		super.add(configPanel, BorderLayout.SOUTH);
		
	
		
	}
	public void enableAllComponents() {
		Component[] components = configPanel.getComponents();
		for (int i = 0; i < components.length; i++)
			components[i].setEnabled(true);
	}
	public void disableAllComponents() {
		Component[] components = configPanel.getComponents();
		for (int i = 0; i < components.length ; i++)
			components[i].setEnabled(false);
	}
	public void setTitleBorder (JPanel panel, String title) {
		panel.setBorder(BorderFactory.createTitledBorder(
				Interface.blackline, title));
	}
	public JPanel getPanel() {
		return configPanel;
	}
	public Component add(Component comp) {
		return configPanel.add(comp);
		
		
	}
	public void setLayout(LayoutManager layout){
		configPanel.setLayout(layout);
	}	
}

Une classe fille :

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
public class Settings extends UI{
	private String family;
	
	private HashMapSettings hashMap;
	
	private JTextField sourceNameField;
	private JComboBox familyBox;
	private JLabel familyLabel ,sourceNameLabel, pathLabel, pathBrowseLabel ;
	private JPanel showPanel;
	public Settings() {
		//panel:
		super("Settings");
		this.setLayout(new BoxLayout(this.getPanel(),BoxLayout.Y_AXIS));
		this.setVisible(false);
		hashMap = new HashMapSettings();
		showPanel();
		configPanel();
		hashMap.read();
		
	
	}

en rouge les lignes qui posent problème...
SI qqn voit plus simple je suis également preneur

[_KB_]

oops fausse manip

[xavlours]

oops fausse manip

C'est-à-dire ?? Résolu ? Faux message ?

Sinon, concernant ton post, je ne vois pas trop en quoi ces lignes posent problème. Je dirais juste qu'il y a plusieurs méthodes add dans JPanel, et que tu n'en surcharges qu'une seule. Il faut peut-être surcharger d'autres méthodes (notamment addImlp, mais je ne suis pas sûr).

Quant au BoxLayout, je te renvoie à la Javadoc qui déconseille de l'utiliser sur des JPanels, mais plutôt sur des Box.

J'ai une question, c'est pourquoi ne proposes-tu pas de passer le composant configPanel en paramètre du contructeur ? Ca te permettrait d'y mettre des JPanel ou des Box indifféremment.

[_KB_]

C'est-à-dire ?? Résolu ? Faux message ?

fausse manip lors d'une édition de mon post, le problème est toujours d'actualité !

Sinon, concernant ton post, je ne vois pas trop en quoi ces lignes posent problème. Je dirais juste qu'il y a plusieurs méthodes add dans JPanel, et que tu n'en surcharges qu'une seule. Il faut peut-être surcharger d'autres méthodes (notamment addImlp, mais je ne suis pas sûr).

je ne connais pas cette méthode je vais aller voir, mais sachant que la méthode que je souhaite redéfinir est add est pas addImlp je ne vois pas pourquoi je devrais redéfinir celle ci

Quant au BoxLayout, je te renvoie à la Javadoc qui déconseille de l'utiliser sur des JPanels, mais plutôt sur des Box.

Je ne savais pas

J'ai une question, c'est pourquoi ne proposes-tu pas de passer le composant configPanel en paramètre du contructeur ? Ca te permettrait d'y mettre des JPanel ou des Box indifféremment.

je ne veux pas passer configPanel en paramètre du constructeur. Je veux que ce paramètre soit transparent pour l'utilisateur des classes filles (d'ou la déclaration en private)

C'est un peu le but que je recherche en redéfinissant les méthodes add et setLayout.

L'utilisateur qui manipule les classes filles doit pouvoir faire un

Code :

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

classeFille.add(Component)

sans savoir que le composant sera ajouté au configPanel et pas au JPanel de la classe mère qui dérive de JPanel (this)

Mais au final je me demande si je me complique pasun peu la vie

[xavlours]

Pour addImpl, c'est peut être pas nécessaire (je ne sais pas trop), mais tu devras surcharger le add à 2 paramètres. Moi la première chose que je ferais avec ton composant, c'est d'y mettre un BorderLayout et de faire :

Code :

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

add(machin, BorderLayout.CENTER);

Pour la transparence, tu peux aussi utiliser un constructeur par défaut, ou des constantes pour choisir le type de contenu. C'est vrai que le fait de ne pas pouvoir utiliser de BoxLayout est dommage.