[swing] Petite question sur JColorChooser

Version imprimable

13/01/2006, 23h36
yupa

[swing] Petite question sur JColorChooser
bonjour
je me sert de showDialog de JColorChooser, pas de probleme jusque la, mais le soucis c'est que 9/10 ca me renvoie null a la place d'une couleur.
Normalement ca doit renvoyé null uniquement si l'utilisateur clique su annulé. Mais la j'ai beau cliqué sur choisir une nouvelle couleur et cliqué sur ok, ca me renvoie null.

est ce que qq a une idée...
je vous met un extrait du code
Code:

1 2 3 4 5 6 Color col = JColorChooser.showDialog(MainFrame.instance(), "color", node.getColorNode()); if(col != null ){ node.setColorNode(col); tree.repaint(); }
a bientot
13/01/2006, 23h54
MeDioN

Je ne comprends pas ton problème.

Moi en instanciant JColorChooser et en appliquant .getColor() dessus je récupère toujours une couleur. Mais comme je vois pas tout ton code je ne sais pas ce que tu cherches à faire (je m'en sers pour changer la couleur d'une courbe).

je cherche a changer la couleur du fond d'un neu d'un Jtree

je te mets le code conplet de l'action:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
public class SOActionSetNodeColor extends SOAction{
    	private SOTree tree;
    	private RoutingGraphTreeNode node;
    	public SOActionSetNodeColor(RoutingGraphTreeNode node, SOTree tree){
    		this.tree = tree;
    		this.node = node;
    		putValue(Action.NAME, "Color");
    	}
    	public void actionPerformed_Threaded(ActionEvent e) {
    		Color col = JColorChooser.showDialog(MainFrame.instance(), "color", node.getColorNode());
    		if(col != null ){
	    		node.setColorNode(col);
	    		tree.repaint();
    		}
    	}
    }

j'ai trouvé dans plusieur exemple l'utilisation de JColorChooser de cette facon...

14/01/2006, 00h31
MeDioN

Color col = JColorChooser.showDialog(MainFrame.instance(), "color", node.getColorNode());

Tu essaies de récupérer la couleur sélectionnée avec ça? Si c'est le cas ça ne risque pas de marcher...
14/01/2006, 00h33
yupa

oui c'est ca
pour quoi ca risque de ne pas marcher?
14/01/2006, 00h41
MeDioN

Citation:

static Color showDialog(Component component, String title, Color initialColor)
Shows a modal color-chooser dialog and blocks until the dialog is hidden.

Ca vient peut être du static
Pour récupérer la couleur sélectionnée, utilise getColor(), moi ça toujours marché avec.
Mais en théorie ton code devrait marcher... Dans quel cas il te renvoie pas null?
14/01/2006, 00h50
yupa

je ne trouve pas un cas particulier...
c'est vraiment alléatoir
tu peu me montrer un exemple de ton 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
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
/**
 * @author Mateo & JS
 */
 
package compGraph.compOngl;
 
//---------------------------------------------------------------------------
//Imports
import java.awt.BorderLayout;
import java.awt.Image;
import java.awt.Toolkit;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.*;
import compGraph.panel.*;
import compGraph.*;
import java.awt.*;
 
//---------------------------------------------------------------------------
//Class
public class ChoixCouleur extends JFrame implements ActionListener{
 
	//Declarations
	JButton ok = new JButton("ok");
	BorderPanel panneau = new BorderPanel();
	GridPanel gp = new GridPanel(1,5);
	JPanel pan1 = new JPanel();
	JPanel pan2 = new JPanel();
	JPanel pan3 = new JPanel();
	JPanel pan4 = new JPanel();
	JColorChooser tcc = new JColorChooser();
	Graphique graphic = new Graphique();
	Color couleurGraph /*= Color.BLACK*/;
	static final long serialVersionUID= 1;
 
	//Constructeur
	public ChoixCouleur(Graphique graph){
		couleurGraph = graph.getColor();
		Image icone = Toolkit.getDefaultToolkit().getImage("./logo.jpg");
		setTitle("Sélection de la couleur");
		setIconImage(icone);
		setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
		getContentPane().add(panneau);
		panneau.add(tcc, BorderLayout.CENTER);
		panneau.add(gp, BorderLayout.SOUTH);
		gp.add(pan1);
		gp.add(pan2);
		gp.add(ok);
		gp.add(pan3);
		gp.add(pan4);
		setSize(450,450);
		ok.addActionListener(this);
		setVisible(false);//Au debut on masque la fenetre
		this.graphic=graph;
	}
 
	// Gestionnaire des actions
	public void actionPerformed(ActionEvent e){
		if(e.getSource()==ok){
			changerCouleur();
		}
	}
 
	// Methodes 
	private void changerCouleur(){
		if(tcc.getColor()!= Color.WHITE){
			graphic.setColor(tcc.getColor());
			graphic.reinitTout();
			graphic.repaint();
			setVisible(false);
		}else{setVisible(false);}
	}
}

Le mien ressemble à ça