Appel d'une classe graphic

**lallana2re** · 13/04/2018, 15h47

Bonjour j'ai un petit blem , en fait je voulais afficher un rectangle (qui se trouve dans un autre classe ) si j'appui sur le bouton rectangle sa doit afficher le rectangle avec les parametres passés dans la methode rect , mais jusqua present rien du tout . Je veux votre aide SVP !!!!

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
class Graphic :
import java.awt.Color;
import java.awt.Component;
import java.awt.Graphics;
 
public class Graphic extends Component{
 
	int u=0,x=0,y=0,z=0;
	int u1=0,x2=0,y3=0,z4=0;
	Color couleur = Color.black;
	Color couleur1 = Color.black;
 
	public void paint(Graphics g) {				
 
		g.setColor(couleur);
		g.drawRect(u, x, y,z);
		g.setColor(couleur1);
		g.drawOval(u1, x2, y3,z4);
	}
 
	public void rect(int u, int x, int y, int z,Color couleur1) {
		this.u=u;
		this.x=x;
		this.y=y;
		this.z=z;
 
		this.couleur1= couleur1;
 
		repaint();
 
	}
 
	public void cer(int u1, int x2, int y3, int z4,Color couleur) {
		this.u1=u1;
		this.x2=x2;
		this.y3=y3;
		this.z4=z4;
 
		this.couleur= couleur;
 
		repaint();
 
	}
 
}

classe appelant le dessin AppPaint :

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
import java.awt.Color;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
 
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
 
public class AppPaint extends JFrame {
 
	public AppPaint() {
 
		Graphic cg = new Graphic();
		JPanel Panel = new JPanel();
 
		JButton bouton;
		JButton bouton1;
 
 
		bouton1 = new JButton("rectangle");
		bouton = new JButton("cercle");		
 
		this.add(Panel);
 
		Panel.add(bouton1);
		Panel.add(bouton);
 
		Panel.add(cg);
 
 
		bouton1.addMouseListener(new MouseAdapter() {
		    public void mouseClicked(MouseEvent arg0) {
		    	cg.rect(100, 100, 100, 100, Color.GREEN);
			}
		});
 
 
 
		this.setSize(400,400);
		this.setTitle("Application de paint");
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		this.setVisible(true);
	}
 
	public static void main (String arg[]) {
		 new AppPaint();
 
	}
 
}

**joel.drigo** · 13/04/2018, 16h40

Salut,

Trois problèmes principaux :

Panel est un JPanel avec un FlowLayout et que Graphic est un java.awt.Component sans taille.
tu ne dessines pas le fond du composant Graphic, ce qui cause des dysfonctionnements (à cause du recyclage du contexte graphique), que tu ne vois pas pour le moment à cause du premier problème

Donc déjà, ajouter l'appel au super.paint(g) pour que le fond soit correctement dessiné :

Code :

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

1
2
3
4
5
6
7
8
public void paint(Graphics g) {				
 
		super.paint(g);
		g.setColor(couleur);
		g.drawRect(u, x, y,z);
		g.setColor(couleur1);
		g.drawOval(u1, x2, y3,z4);
	}

Ensuite, ce serait mieux d'utiliser un composant Swing et pas un composant AWT. Donc un JPanel par exemple (ou un JComponent). Cela ne change pas grand chose à ton code public class Graphic extends JPanel { au lieu de public class Graphic extends Component { (et l'import bien sûr)

Dans un FlowLayout, un composant doit au moins avoir une taille préférentielle :

Code :

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

1
2
3
public Graphic() {
		setPreferredSize(new Dimension(300,300));
	}

Mais tant qu'à faire :

Code :

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

1
2
3
4
5
public Graphic() {
		setMinimumSize(new Dimension(300,300));
		setPreferredSize(new Dimension(300,300));
		setMaximumSize(new Dimension(300,300));
	}

Ici, dans cet exemple, j'ai mis la même taille pour les 3 (300x300), mais on pourrait avoir des tailles différentes pour le mini, le maxi et le préférentiel.

La deuxième solution serait de mettre le Graphic dans un JPanel avec BorderLayout, mais ça t'obligera à avoir un JPanel spécifique pour les boutons, à mettre au nord de Panel.

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
		Graphic cg = new Graphic();
		JPanel Panel = new JPanel(new BorderLayout());
 
		JButton bouton;
		JButton bouton1;
 
 
		bouton1 = new JButton("rectangle");
		bouton = new JButton("cercle");		
 
		this.add(Panel);
 
		JPanel Panel2 = new JPanel();
		Panel2.add(bouton1);
		Panel2.add(bouton);
 
		Panel.add(Panel2, BorderLayout.NORTH);
		Panel.add(cg);

A noter par ailleurs, qu'il y a une inversion entre couleur et couleur1 (tu t'en apercevras vite en testant).

PS. j'ai conservé les mêmes conventions que dans ton code pour rester cohérent, mais ce serait mieux de respecter les conventions Java pour le nommage des variables et l'indentation.

**lallana2re** · 13/04/2018, 17h49

Jai mm essayé d'afficher le rectangle sans cliquer un bouton mais ça marche pas regarder le code des deux class:
class Graphic :

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
import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import javax.swing.JPanel;
 
public class Graphic extends JPanel{
 
	int u=0,x=0,y=0,z=0;
	int u1=0,x2=0,y3=0,z4=0;
	Color couleur = Color.black;
	Color couleur1 = Color.black;
 
	public void paint(Graphics g) {
 
		super.paint(g);
		g.setColor(couleur);
		g.drawRect(100, 100, 100,100);
		g.setColor(couleur1);
		g.drawOval(u1, x2, y3,z4);
 
		this.setPreferredSize(new Dimension(300,300));
		this.setMinimumSize(new Dimension(300,300));
		this.setPreferredSize(new Dimension(300,300));
		this.setMaximumSize(new Dimension(300,300));
	}
 
	public void rect(int u, int x, int y, int z,Color couleur) {
		this.u=u;
		this.x=x;
		this.y=y;
		this.z=z;
		this.couleur= couleur;
 
 
		repaint();
 
	}
 
	public void cer(int u1, int x2, int y3, int z4,Color couleur1) {
		this.u1=u1;
		this.x2=x2;
		this.y3=y3;
		this.z4=z4;
 
		this.couleur1= couleur1;
		;
 
		repaint();
 
	}
 
}

class AppPaint :

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
 
 
import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
 
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
 
public class AppPaint extends JFrame {
 
	public AppPaint() {
 
		Graphic cg = new Graphic();
		JPanel Panel = new JPanel();
		JPanel Pan = new JPanel(new BorderLayout());
 
		JButton bouton;
		JButton bouton1;
 
 
		bouton1 = new JButton("rectangle");
		bouton = new JButton("cercle");	
 
		this.add(Pan);
		this.add(Panel);
 
 
		Panel.add(bouton1);
		Panel.add(bouton);
 
		Pan.add(cg,BorderLayout.SOUTH);
		Pan.add(cg);
		cg.rect(100, 100, 100, 100, Color.GREEN);
 
		bouton1.addMouseListener(new MouseAdapter() {
		    public void mouseClicked(MouseEvent arg0) {
		    	cg.rect(100, 100, 100, 100, Color.GREEN);
			}
		});
 
 
 
		this.setSize(400,400);
		this.setTitle("Application de paint");
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		this.setVisible(true);
	}
 
	public static void main (String arg[]) {
		 new AppPaint();
 
	}
 
}

**joel.drigo** · 13/04/2018, 17h52

Regarde mieux ce que j'ai écrit : je n'ai pas mis les setXXXXSize dans la méthode paint, mais dans le constructeur de Graphic.

**lallana2re** · 13/04/2018, 18h04

Oui Oui je l'ai mis dans le constructeur de la class Graphic mais ça n'a pas afficher .
Essai de vérifier ce que j'ai écrit dans mon code précédent sachant que le constructeur a été bien modifier

**joel.drigo** · 13/04/2018, 18h15

Tout ça non plus ne correspond à ce que j'ai écrit, et plus non plus à ce que tu avais écrit (Panel est devenu le composant des boutons, et c'est Pan le composant principal) :

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
 
Graphic cg = new Graphic();
		JPanel Panel = new JPanel();
		JPanel Pan = new JPanel(new BorderLayout());
 
		JButton bouton;
		JButton bouton1;
 
 
		bouton1 = new JButton("rectangle");
		bouton = new JButton("cercle");	
 
		this.add(Pan);
		this.add(Panel);
 
 
		Panel.add(bouton1);
		Panel.add(bouton);
 
		Pan.add(cg,BorderLayout.SOUTH);
		Pan.add(cg);

Pan va dans la JFrame
Panel va dans Pan, au nord ou au sud ou ailleurs, sauf au centre
cg (Graphic) va au centre de Pan (pas la peine de le préciser, c'est par défaut).
Les boutons vont dans Panel

donc this.add(Panel); à supprimer et Pan.add(cg,BorderLayout.SOUTH); à remplacer par Pan.add(Panel,BorderLayout.SOUTH);

**lallana2re** · 13/04/2018, 20h36

Oui je vois ça a passé génial un grand merci

!!!!!!!!

**lallana2re** · 15/04/2018, 16h02

Salut,
J'ai encore un problème : si je clique sur le rectangle puis le cercle et que je veux changer la couleur, au lieu de changer la couleur du cercle , il redessine un rectangle en fonction des couleurs choisies et pourtant j'ai bien reinitialiser les valeurs, à chaque fois que je clique sur rectangle ou cercle mais toujours pas de solution.

class Graphic :

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
 
import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import javax.swing.JPanel;
 
public class Graphic extends JPanel{
 
int u=0,x=0,y=0,z=0;
int u1=0,x2=0,y3=0,z4=0;
Color couleur = Color.black;
Color couleur1 = Color.black;
 
public Graphic(){
setPreferredSize(new Dimension(300,300));
}
 
public void paint(Graphics g) {
 
super.paint(g);
g.setColor(couleur);
g.drawRect(u, x, y,z);
g.setColor(couleur1);
g.drawOval(u1, x2, y3,z4);
 
}
 
 
 
public void rect(int u, int x, int y, int z,Color couleur) {
this.u=u;
this.x=x;
this.y=y;
this.z=z; 
this.couleur= couleur;
 
repaint();
 
}
 
public void cer(int u1, int x2, int y3, int z4,Color couleur1) {
this.u1=u1;
this.x2=x2;
this.y3=y3;
this.z4=z4; 
this.couleur1= couleur1;
 
repaint();
 
}
 
 
}

class AppPaint :

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
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
 
import java.awt.Color;
import java.awt.Event;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
 
import javax.swing.JButton;
import javax.swing.JComboBox;
import javax.swing.JFrame;
import javax.swing.JPanel;
 
public class AppPaint extends JFrame {
 
public AppPaint() {
 
Graphic cg = new Graphic(); 
JPanel Pan = new JPanel();
JPanel jp = new JPanel();
 
JButton bouton1 = new JButton("rectangle");
JButton bouton = new JButton("cercle");
 
String[] tableau = {"bleu","rouge","jaune"};
JComboBox c1 = new JComboBox(tableau); 
 
 
this.add(Pan);
Pan.add(jp);
Pan.add(c1);
Pan.add(bouton1);
Pan.add(bouton);
Pan.add(cg);
 
 
bouton1.addMouseListener(new MouseAdapter() {
public void mouseClicked(MouseEvent arg0) {
 
cg.rect(100, 100, 100, 100, Color.black);
cg.cer(0, 0, 0, 0, Color.blue);
 
 
class EcouteurBox implements ItemListener{
 
@Override
public void itemStateChanged(ItemEvent e) {
 
if(e.getSource()==c1) {
if(c1.getSelectedItem().equals("bleu")) {
 
cg.rect(100, 100, 100, 100, Color.blue);
cg.cer(0, 0, 0, 0, Color.blue);
 
}else if(c1.getSelectedItem().equals("rouge")) {
 
cg.rect(100, 100, 100, 100, Color.red);
cg.cer(0, 0, 0, 0, Color.blue);
 
}else if(c1.getSelectedItem().equals("jaune")){
 
cg.rect(100, 100, 100, 100, Color.yellow);
cg.cer(0, 0, 0, 0, Color.blue);
 
}
 
}
}
 
}
 
c1.addItemListener(new EcouteurBox());
 
}
});
 
bouton.addMouseListener(new MouseAdapter() {
public void mouseClicked(MouseEvent arg0) {
 
cg.cer(100, 100, 100, 100, Color.black);
cg.rect(0, 0, 0, 0, Color.black);
 
class EcouteurBox implements ItemListener{
 
@Override
public void itemStateChanged(ItemEvent e) {
 
if(e.getSource()==c1) {
if(c1.getSelectedItem().equals("bleu")) {
 
cg.cer(100, 100, 100, 100, Color.blue);
cg.rect(0, 0, 0, 0, Color.black);
 
}else if(c1.getSelectedItem().equals("rouge")) {
 
cg.cer(100, 100, 100, 100, Color.red);
cg.rect(0, 0, 0, 0, Color.black);
 
}else if(c1.getSelectedItem().equals("jaune")){
 
cg.cer(100, 100, 100, 100, Color.yellow);
cg.rect(0, 0, 0, 0, Color.black);
 
}
 
}
}
 
}
 
c1.addItemListener(new EcouteurBox());
 
}
});
 
 
 
this.setSize(400,400);
this.setTitle("Application de paint");
this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
this.setVisible(true);
}
 
public static void main (String arg[]) {
new AppPaint();
 
}
 
}

**joel.drigo** · 16/04/2018, 01h07

Chaque fois que tu cliques sur un bouton, tu enregistres un écouteur sur la combo. Il reste enregistré tant que tu l'enlèves pas. Donc quand tu cliques sur le bouton cercle, l'écouteur enregistré lors du clique sur le bouton rectangle est encore effectif et sera exécuté. Plus tu cliques sur l'un des boutons, plus tu auras d'écouteurs sur les combo et ils seront toujours exécutés.

Soit tu utilises un seul écouteur de combo, avec un booléen qui permet de savoir le dernier bouton cliqué (donc savoir si on doit changer la couleur du rectangle ou celle du cercle,)
Soit il te faut retirer l'écouteur précédemment enregistré sur la combo avant de mettre le nouveau.

Par ailleurs, utilise de préférence des ActionListener plutôt que des écouteurs bas niveau comme MouseAdapter et ItemListener.

**lallana2re** · 18/04/2018, 17h25

ahhhh okk Frere tout est resolu , merci!!!!!!!!!

Appel d'une classe graphic

Java

Discussions similaires

Partager

Partager