Discussion :

[haraelendil]

Bonjour,

Voila, mon appli a besoin d'une zone (classe Centre qui hérite de JPanel) ou l'on peut ajouter des autres JPanel (ici une classe Piste qui hérite donc de JPanel aussi) en cliquant sur un bouton, mais quand je clique, rien de ne passe:

voila le code de la classe, j'ai du faire une bétise quelquepart

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public class Centre extends JPanel //implements Scrollable
{	
	public Centre()
	{
		setBackground(java.awt.Color.LIGHT_GRAY);
		setLayout(new BoxLayout(this,BoxLayout.Y_AXIS));
	}
 
	public void nouvellePiste(File fichier)
	{
		Log.ecrire("Création d'une nouvelle piste avec fichier "+fichier);
		add(new Piste(fichier));
	}
}

J'ai ici utilisé un BoxLayout pour que les composants ajoutés soient les uns au dessus des autres.

Si quelqu'un entrevoit une solution...

Merci d'avance

[tchize_]

Déjà, as-tu le message, pourrais-tu nous montre le code qui fait appel à nouvellePiste?

[haraelendil]

oui oui, j'ai bien le message du log, je l'ai mis la exprès pour ça, je fais ça comme ça:

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
public void actionPerformed(ActionEvent e) 
	{
		if(e.getSource()==chargerAudio)
		{
			Log.ecrire("Demande de chargement de fichier audio");
			JFileChooser fc = new JFileChooser();
			int returnVal = fc.showOpenDialog(this);
			if (returnVal == JFileChooser.APPROVE_OPTION) 
			{
                File file = fc.getSelectedFile();
                scrollZone.nouvellePiste(file);
			}
			else return;
		}
	}

Ou chargerAudio est le nom du bouton qui génère l'évènement

Mais j'ai bien les deux messages dans le log, ce qui porte à croire que la fonction nouvellePiste est bien appelée.

[tchize_]

scrollzone est-il bien présent à l'écran (ajoute lui un bord rouge ou autre pour etre sur), même chose pour l'objet Piste (couleur différente). Montre nous aussi le code de Piste. Probablement que Piste a une taille de 0

[haraelendil]

J'ai testé la meme chose sans la scrollZone, ça change rien, donc ça ne vient pas de la (du moins pas pour l'instant lol).

Je vais mettres des bordures voir ce que ça donne

[haraelendil]

Bon ba ça vient bien du fait que la piste ne se rajoute pas au Centre, car quand je met des bordures, on voit celle du centre, mais j'ai beau cliquer mille fois, aucune bordure de piste.

Et le pistes ne sont pas vide, puisque quand je met juste une piste dans un JFrame, ça marche nikel....

[tchize_]

est-tu sur d'ajoute la piste dans la bonne instance de centre? Change la couleur de bordure de centre quand t'ajoute la piste ...?

[haraelendil]

Ah ça y est j'ai trouvé, c'est trés bete, je m'explique.
Avec le changement de couleur de la bordure du Centre (c'était la bonne instance, et pour cause, y en a qu'une^^), ça marche comme par magie, et dés que je l'enlève le changement, ça remarche pu, sauf si je redimensionne la fenêtre

donc je suppose que ça doit venir du fait que le centre n'est pas redéssiné à l'ajout d'une piste, ce qui me mène à une autre question, comment qu'on peut faire ça?^^ si je fais un centre.update(centre.getGraphics()); ???

//edit
Bon ba réponse à moi même, nan ça marche pas :p

[tchize_]

normalement, c'est repaint(), mais normalement, un add() provoque le redessinage. Probablement que ton Piste n'est pas très orthodoxe, on pourrait le voir?

[haraelendil]

Oh ba oui (et le repaint() un peu partout change rien^^):

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
74
75
76
77
78
79
80
81
82
83
84
85
public class Piste extends JPanel 
{	
	private CommandePiste commande;
	private GraphiquePiste graphique;
 
	private boolean muet;
 
	public Piste(File fichier)
	{
		super();
		graphique = new GraphiquePiste(this);
		commande = new CommandePiste(this);
		muet = false;
		setSize(1440,200);
		setLayout(new BoxLayout(this,BoxLayout.X_AXIS));
		setBorder(new LineBorder(java.awt.Color.black));
		add(commande);
		add(graphique);
	}
 
	public void setMuet(boolean b)
	{
		muet = b;
		graphique.repaint();
	}
 
	public boolean getMuet()
	{
		return muet;
	}
}
 
public class CommandePiste extends JPanel implements ItemListener
{	
	private JCheckBox muetCheck;
	private Piste piste;
 
	public CommandePiste(Piste p)
	{
		piste = p;
		setSize(new Dimension(200,200));
		setBackground(java.awt.Color.LIGHT_GRAY);
		muetCheck = new JCheckBox("Muet");
		muetCheck.addItemListener(this);
		add(muetCheck);
	}
 
	public void itemStateChanged(ItemEvent e) 
	{
		Object source = e.getItemSelectable();
		if(source==muetCheck)
		{
			if(e.getStateChange()==ItemEvent.SELECTED) piste.setMuet(true);
			if(e.getStateChange()==ItemEvent.DESELECTED) piste.setMuet(false);
		}
	}
 
}
 
public class GraphiquePiste extends JPanel 
{	
	private Piste piste;
 
	public GraphiquePiste(Piste p)
	{
		super();
		piste = p;
		setPreferredSize(new Dimension(1240,200));
	}
 
	protected void paintComponent(Graphics g)
	{
		Dimension d = getSize();
		g.setColor(java.awt.Color.WHITE);
		g.drawRect(0, 0, d.height, d.width);
		if(piste.getMuet()) g.setColor(java.awt.Color.BLUE);
		else g.setColor(java.awt.Color.DARK_GRAY);
		dessinerSon(g, d);
	}
 
	private void dessinerSon(Graphics g, Dimension d)
	{
		g.drawLine(0,d.height/2, d.width, d.height/2);
	}
}

Le problème pourrait venir de GraphiquePiste??

[ark99]

quand on ajoute un composant, normalement il faut faire appel à validate()

C'est d'ailleurs ce qui est préconisé dans la javadoc, à la méthode add() de Container;

If a component has been added to a container that has been displayed, validate must be called on that container to display the new component

Peut-être une piste??? souvent il faut d'ailleurs faire:
component.repaint();
component.validate();

[haraelendil]

Ah ba ouai, avec le validate() tout de suite après sur mon Centre, ça marche nickel, merci bien