Discussion :

[yupa]

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 :

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

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

[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).

[yupa]

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

je te mets le code conplet de l'action:

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
 
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...

[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...

[yupa]

oui c'est ca
pour quoi ca risque de ne pas marcher?

[MeDioN]

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?

[yupa]

je ne trouve pas un cas particulier...
c'est vraiment alléatoir
tu peu me montrer un exemple de ton code?

[MeDioN]

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
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