Discussion :

[nadong94]

Bonjour à tous,

J'ai une classe qui implémente ChangeListener.
Je ne sais pas trop quoi faire pour récupérer un SelectedIndex de façon générique, je m'explique :

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
 
package EVENT;
 
import javax.swing.JTabbedPane;
import javax.swing.event.ChangeEvent;
import javax.swing.event.ChangeListener;
 
public class ActionChange implements ChangeListener {
	//Un jtabbedPane génrique
	private JTabbedPane tabbedPane;
 
	//Le constructeur va donc initialiser le tabbedPane dont j'ai besoin d'un changeListener
	public ActionChange(final JTabbedPane tabbedPane) {
		this.tabbedPane = tabbedPane;
	}
 
	//le getSelectedIndex() doit être rempli génériquement au préalable mais comment ?
	private int lastSelected = tabbedPane.getSelectedIndex();
	{
		tabbedPane.setForegroundAt(lastSelected,
				Final.getSelectionColor());
	}
 
	@Override
	public void stateChanged(ChangeEvent arg0) 
	{
		// Color selection tab
		if (lastSelected >= 0) {
			tabbedPane.setForegroundAt(lastSelected,
					tabbedPane.getForeground()); // default
													// color
			tabbedPane.setBackgroundAt(lastSelected,
					tabbedPane.getBackground());
		}
		lastSelected = tabbedPane.getSelectedIndex();
		tabbedPane.setForegroundAt(lastSelected,Final.getSelectionColor());
	}
}

Merci à tous

[nadong94]

J'ai essayer quelque chose, quand pensez-vous ?

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
 
package EVENT;
 
import java.awt.Component;
import java.util.List;
 
import javax.swing.JComponent;
import javax.swing.JTabbedPane;
import javax.swing.event.ChangeEvent;
import javax.swing.event.ChangeListener;
 
public class ActionChange implements ChangeListener {
	private JTabbedPane tabbedPane;
	private String[] names;
	private Component[] components;
 
	public ActionChange(final JTabbedPane tabbedPane, String[] names, Component[] components) {
 
		for (String name : names) {
			for (Component component : components) {
				tabbedPane.add(name,component);
			}
		}
		this.names = names;
		this.components = components;
		this.tabbedPane = tabbedPane;
	}
 
	private int lastSelected = tabbedPane.getSelectedIndex();
	{
		tabbedPane.setForegroundAt(lastSelected,
				Final.getSelectionColor());
	}
 
	@Override
	public void stateChanged(ChangeEvent arg0) 
	{
		// Color selection tab
		if (lastSelected >= 0) {
			tabbedPane.setForegroundAt(lastSelected,
					tabbedPane.getForeground()); // default
													// color
			tabbedPane.setBackgroundAt(lastSelected,
					tabbedPane.getBackground());
		}
		lastSelected = tabbedPane.getSelectedIndex();
		tabbedPane.setForegroundAt(lastSelected,Final.getSelectionColor());
	}
}

Toujours un NPE au même endroit. Donc il n'a pas initialiser le tabbedPane, pourquoi ?

[joel.drigo]

Salut,

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
public class ActionChange implements ChangeListener {
	//Un jtabbedPane génrique
	private JTabbedPane tabbedPane;
 
	//Le constructeur va donc initialiser le tabbedPane dont j'ai besoin d'un changeListener
	public ActionChange(final JTabbedPane tabbedPane) {
		(2)this.tabbedPane = tabbedPane;
	}
 
	//le getSelectedIndex() doit être rempli génériquement au préalable mais comment ?
	(1)private int lastSelected = tabbedPane.getSelectedIndex();
	{
		tabbedPane.setForegroundAt(lastSelected,
				Final.getSelectionColor());
	}
 
	@Override
	public void stateChanged(ChangeEvent arg0) 
	{
		// Color selection tab
		if (lastSelected >= 0) {
			tabbedPane.setForegroundAt(lastSelected,
					tabbedPane.getForeground()); // default
													// color
			tabbedPane.setBackgroundAt(lastSelected,
					tabbedPane.getBackground());
		}
		lastSelected = tabbedPane.getSelectedIndex();
		tabbedPane.setForegroundAt(lastSelected,Final.getSelectionColor());
	}
}

La ligne 1 est exécutée avant la ligne 2 (toujours les déclarations et leur initialisation associée d'abord, puis le code dans le constructeur), donc quand tabbedPane contient encore null...

Solution, tout simplement :

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
ublic class ActionChange implements ChangeListener {
	//Un jtabbedPane génrique
	private JTabbedPane tabbedPane;
	private int lastSelected;
 
	//Le constructeur va donc initialiser le tabbedPane dont j'ai besoin d'un changeListener
	public ActionChange(final JTabbedPane tabbedPane) {
		this.tabbedPane = tabbedPane;
                this.lastSelected = tabbedPane.getSelectedIndex()
                tabbedPane.setForegroundAt(lastSelected, Final.getSelectionColor());
	}
 
	@Override
	public void stateChanged(ChangeEvent arg0) 
	{
		// Color selection tab
		if (lastSelected >= 0) {
			tabbedPane.setForegroundAt(lastSelected,
					tabbedPane.getForeground()); // default
													// color
			tabbedPane.setBackgroundAt(lastSelected,
					tabbedPane.getBackground());
		}
		lastSelected = tabbedPane.getSelectedIndex();
		tabbedPane.setForegroundAt(lastSelected,Final.getSelectionColor());
	}
}

sinon, c'est quoi ce Final.getSelectionColor() ? Tu peux pas mettre la constante dans la classe ActionChange, tout simplement, ou faire un import static ?

[nadong94]

Waouh ça ma fait gagner un nombre incalculable de ligne MERCI.
Tu as raison pour le Final.getSelectColor. Désormais je l'ai déclarer dans la classe en final.