Méthode contains() d'un Rectangle2D

Version imprimable

Bonsoir à tous,

je me pose certaines questions concernant la méthode contains applicables à des Rectangle2D car elle n'a jamais fonctionnée chez moi (alors que tout compile et s'exécute parfaitement).

Voici tout d'abord mes codes sources :

Classe Example.java
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 import java.awt.*; import javax.swing.*; import java.awt.event.*; import java.awt.geom.*; public class Example extends JFrame { private Container cont = new Container(); private PanRoundButton pan = new PanRoundButton(); private Menu lemenu = new Menu(); private Rectangle2D rect2d = new Rectangle2D.Double(); public Example() { super("Exemple"); setDefaultCloseOperation(EXIT_ON_CLOSE); setSize(800,600); setLocationRelativeTo(null); cont = getContentPane(); cont.setLayout(null); cont.add(lemenu); cont.add(pan); pan.setBounds(0,0,800,600); lemenu.setBounds(0,0,800,600); lemenu.addMouseListener(new MouseAdapter() { public void mouseClicked(MouseEvent e) { System.out.println(lemenu.getRect2D()); if(lemenu.getRect2D().contains(e.getX(),e.getY())) System.out.println("true"); else System.out.println("false"); } }); setVisible(true); } public static void main(String[] args) { new Example(); } }
Voici ensuite la classe Menu.java :
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 import java.awt.*; import javax.swing.*; import java.awt.geom.*; public class Menu extends JPanel { private String menuTitle = "Bubble Game"; private int buttonWidth1 = 120, buttonWidth2 = 120, buttonWidth3 = 120; private int buttonHeight = 35; private int interButton = 10; private Rectangle2D rect = new Rectangle2D.Double(); private Rectangle2D rect2d = new Rectangle2D.Double(); public Menu() { } public void paint(Graphics gg) { Graphics2D g = (Graphics2D)gg; Font f1 = new Font("Arial",Font.PLAIN,60); g.setFont(f1); FontMetrics fm = g.getFontMetrics(); g.drawString(menuTitle,(getWidth()-fm.stringWidth(menuTitle))/2,fm.getAscent()+10); g.drawLine(70,fm.getHeight()+10,getWidth()-70,fm.getHeight()+10); g.translate(0,fm.getHeight()+10); Font f2 = new Font("Arial",Font.PLAIN,15); g.setFont(f2); FontMetrics fm2 = g.getFontMetrics(); g.drawRoundRect((getWidth()-buttonWidth1)/2,20,buttonWidth1,buttonHeight,10,30); g.drawRoundRect((getWidth()-buttonWidth2)/2,20+buttonHeight+interButton,buttonWidth2,buttonHeight,10,30); g.drawString("Nouvelle partie",(getWidth()-fm2.stringWidth("Nouvelle partie"))/2,20+(buttonHeight+fm2.getAscent())/2); g.drawRoundRect((getWidth()-buttonWidth3)/2,20+buttonHeight+interButton+buttonHeight+interButton,buttonWidth3,buttonHeight,10,30); rect2d.setRect((getWidth()-buttonWidth1)/2,20,buttonWidth1,buttonHeight); g.draw(rect2d); } public Rectangle2D getRect2D() { return rect2d; } public void setRect2D(Rectangle2D rect2d) { this.rect2d = rect2d; } }
Lorsque j'exécute ce code, le résultat fourni par :
Code:

1 2 if(lemenu.getRect2D().contains(e.getX(),e.getY())) System.out.println("true"); else System.out.println("false");
me retourne toujours 'false' (d'où le problème, car même quand je clique à l'intérieur du Rectangle2D correspondant, j'ai false)

D'autre part, je pense qu'il s'agit (peut-être) de l'accesseur et du mutateur suivant qui sont mal programmés.
Code:

1 2 3 4 5 6 7 public Rectangle2D getRect2D() { return rect2d; } public void setRect2D(Rectangle2D rect2d) { this.rect2d = rect2d; }
En effet, la ligne

Code:

System.out.println(lemenu.getRect2D());

de la classe Example me retourne :

Code:

java.awt.geom.Rectangle2D$Double[x=340.0,y=20.0,w=120.0,h=35.0]

(ce qui correspond parfaitement à mon Rectangle2D !)

Or, quand je change le nom de rect2d en rect par exemple, au sein de l'accesseur et du mutateur, je me retrouve avec :

Code:

java.awt.geom.Rectangle2D$Double[x=0.0,y=0.0,w=0.0,h=0.0]

Je pense donc que j'utilise mal cet accesseur et ce mutateur, mais je ne vois pas comment m'y prendre autrement.

Si quelqu'un pouvait donc éclairer ma lanterne... :mrgreen:

Une dernière petite question (à part) :

Pourquoi l'accesseur n'a pas besoin de prendre d'argument, alors que la méthode retourne l'objet en question?
Par exemple, ici, pourquoi n'écrit-on pas
Code:

1 2 3 public Rectangle2D getRect2D(Rectangle2D rect2d) { return rect2d; }
(je pause cette question car il n'y a pas que l'accesseur qui fonctionne comme cela. Pas mal de méthodes utilisent des variables pour 'travailler dessus' alors que ces dernières ne passent pas spécialement en argument, et ça, je ne vois pas du tout pourquoi)

Merci d'avance et bonne soirée à tous :ccool:

04/02/2010, 08h52
Jack Sparrow
Citation:

Envoyé par greasf

Or, quand je change le nom de rect2d en rect par exemple, au sein de l'accesseur et du mutateur, je me retrouve avec :

Code:

java.awt.geom.Rectangle2D$Double[x=0.0,y=0.0,w=0.0,h=0.0]

Bah, si tu fais ça :
Code:

1 2 3 4 public void setRect2D(Rectangle2D rect2d) { this.rect = rect; }
C'est clair que ça va pas faire grand chose.

On peut voir ton e.getX() et e.getY().
La position de la souris correspond à la position sur le panel.

Non, en fait :
Code:

1 2 3 public void setRect2D(Rectangle2D rect2d) { this.rect2d = rect2d; }
et
Code:

1 2 3 public void setRect2D(Rectangle2D rect) { this.rect = rect; }
me retournent deux résultats différents, ce qui n'est pas normal (?).

Le premier me retourne

Code:

java.awt.geom.Rectangle2D$Double[x=340.0,y=20.0,w=120.0,h=35.0]

et le deuxième me retourne :

Code:

java.awt.geom.Rectangle2D$Double[x=0.0,y=0.0,w=0.0,h=0.0]

(étant donné la déclaration suivante : )
Code:

1 2 3 4 5 private Rectangle2D rect = new Rectangle2D.Double(); private Rectangle2D rect2d = new Rectangle2D.Double(); rect2d.setRect((getWidth()-buttonWidth1)/2,20,buttonWidth1,buttonHeight);
Or, l'accesseur et le mutateur sont censés fonctionner quelque soit l'objet, donc, je ne vois pas pourquoi, dès que je change l'argument, ça change le résultat. Pour moi, c'est selon l'appel qu'on y fait qui doit changer les résultats, non?

Voici mes getX() et getY() :
Code:

1 2 3 4 5 6 7 cont.addMouseListener(new MouseAdapter() { public void mouseClicked(MouseEvent e) { System.out.println(myMenu.getRect2D()); if(leMenu.getRect2D().contains(e.getX(),e.getY())) System.out.println("true"); else System.out.println("false"); } });
(J'ai essayé d'appliquer l'écouteur au cont, à this, à lemenu, mais quelque soit le cas, il me retourne false)

Merci d'avance ;)

04/02/2010, 13h36
Jack Sparrow
Citation:
Envoyé par greasf

Non, en fait :

Code:

1 2 3 public void setRect2D(Rectangle2D rect2d) { this.rect2d = rect2d; }

et

Code:

1 2 3 public void setRect2D(Rectangle2D rect) { this.rect = rect; }

me retournent deux résultats différents, ce qui n'est pas normal (?).
Ca me parait normal. Tu as une variable private rect et une variable private rect2d. Si tu affiches l'autre, forcement ça sera (0,0,0,0)

Quand je demandais tes getX et getY, c'était leur contenu ;)
A mon avis, tu as une incohérence entre ta notion de rectangle et la position réelle de la souris (la souris ne se fie qu'à la position dans le panel)
04/02/2010, 14h37
greasf

J'ai affiché getX et getY, et il me retourne correctement la valeur de la position sur le panneau (dans l'angle supérieur gauche, j'ai bien (0,0), et dans l'angle inférieur droit, j'ai mon (800,600) définit).

Comment faire alors pour que la ligne suivante

Code:

leMenu.getRect2D().contains(e.getX(),e.getY())

s'applique quelque soit le rectangle2D?

C'est-à-dire pour que l'accesseur et le mutateur s'exécute quelque soit le Rectangle2D?

Merci d'avance et bonne journée:ccool:
04/02/2010, 14h42
tchize_

Citation:

Envoyé par greasf

Comment faire alors pour que la ligne suivante

Code:

leMenu.getRect2D().contains(e.getX(),e.getY())

s'applique quelque soit le rectangle2D?

C'est-à-dire pour que l'accesseur et le mutateur s'exécute quelque soit le Rectangle2D?

Ta question n'a ni queue ni tete. Soyons clair

1) quels sont les limites de ton REctangle2D au moment du test?
2) quelles sont les valeurs de getX et getY (car je note que tu ne les a toujours pas données) quand ca foire?

Quand à ton rectangle2D: nettoie ton code déjà pour virer le rectangle2D que tu utilise pas, histoire de pas confondre les rectangels que tu utilise!
04/02/2010, 15h04
wax78

Citation:

Envoyé par tchize_

2) quelles sont les valeurs de getX et getY (car je note que tu ne les a toujours pas données) quand ca foire?

Pas besoin de chercher plus loin c'est ca son problème je pense pour avoir lancé son code et avoir vu les valeurs :)

Le rectangle vaut (xywh : 340,20, 120,35) dans le panel, or la souris au moment du clique sur "le rectangle" affiche en realité 340,100 ce qui effectivement va lui envoyer false.

Greafs si tu clique X pixels plus haut tu verras que ca te retourne true.
Je pense qu'il faudrait aussi eviter de changers les bounds du rectangle a l'interieur du paint (mais bon ...).
04/02/2010, 22h24
greasf

Merci pour vos réponses.
Effectivement, wax78 a totalement raison, merci pour avoir testé le code (et répondre par la même occasion à tchize_:mrgreen:).

Cependant, je ne comprend pas pourquoi il y a un tel décalage.
Quelqu'un pourrait-il m'expliquer?

@tchize_ :
je vais essayé de reformuler ma question qui te semble si incohérente ^^.
En fait, je dessine plusieurs "boutons" avec drawRoundRect (tous les 3 dessinés dans la variable 'lemenu').
Au final, j'aurai plusieurs tests à faire avec les Rectangle2D, c'est-à-dire que je pensais faire par exemple 3 drawRoundRect, et donc 3 Rectangle2D.
Etant donné cela, j'aurai donc 3 tests, du type :

Code:

leMenu.getRect2D().contains(e.getX(),e.getY())

Cependant, cette ligne fait appel à getRect2D(), et la valeur renvoyée ne varie pas selon le rectangle2D comme on a pu le constater précédemment, et c'est ça que j'aimerai rectifier.
En espérant avoir été un peu plus clair.

Merci à tous et bonne soirée
04/02/2010, 23h35
tchize_

si t'as besoin de trois rectangle, tu crée trois champs, c'est tout.

A noter que tes trois "boutons" sont dessinés décalés de 10pixels vers le bas par rapport aux coordonnées dans rectangle.
05/02/2010, 10h06
wax78

De la façon don tu fais, tu peux même utiliser une liste de rectangles. Et au moment du clique tu parcours la liste et tu regardes si un des éléments (rectangle) contient le point de ta souris. Ainsi tu pourras mettre autant de boutons que tu veux et n'importe ou de n'importe quel taille (sans recouvrements).
05/02/2010, 13h55
greasf

@ tchize_ :
En faisant cela, je devrai faire autant de setter et getter qu'il y a de "boutons", non?
C'est là mon problème, je pensais qu'un seul et unique setter et getter pouvais gérer autant d'objet que l'on désire passer en argument.

@wax78 :
C'est en effet un peu comme cela que j'avais vu la chose, jusqu'à ce que je me heurte à ce problème. Du coup, je ne sais plus vraiment comment je dois faire.

En ce qui concerne le décalage, avez-vous une idée pour régler ce dernier?

Merci :)
05/02/2010, 14h09
tchize_

un getter/setter, c'est pour une propriété. Maintenant, ta propriété peut très bien être une List<Rectangle2D>

Alllais, c'est bien parce qu'ajourd'hui c'est mon anniversaire :

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
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
import java.awt.*;
import javax.swing.*;
 
import java.awt.event.ComponentEvent;
import java.awt.event.ComponentListener;
import java.awt.geom.*;
import java.awt.geom.Rectangle2D.Double;
import java.util.ArrayList;
 
public class Menu extends JPanel implements ComponentListener {
 
	private String menuTitle = "Bubble Game";
	private int buttonWidth1 = 120;
	private int buttonHeight = 35;
	private int interButton = 10;
 
	ArrayList<Rectangle2D> boutons = new ArrayList<Rectangle2D>();
 
	public Menu()
	{
		setLayout(null);
		setSize(800,600);
		initboutons();
		this.addComponentListener(this);
	}
 
	// Place les rectangle dans l'espace tels qu'il devraient etre.
	public void initboutons()
	{
		int w = getWidth();
		int h = getHeight();
		boutons.clear();
		for (int i = 0; i < 4; i++)
		{
			Rectangle2D rect = new Rectangle2D.Double();
			rect.setRect(w/2-buttonWidth1/2, 20+(buttonHeight+interButton)*i, buttonWidth1, buttonHeight);
			boutons.add(rect);
		}
	}
 
	public void paint(Graphics gg) {
		Graphics2D g = (Graphics2D)gg;
 
		Font f1 = new Font("Arial",Font.PLAIN,60); 
		g.setFont(f1);			   
		FontMetrics fm = g.getFontMetrics();
 
		Font f2 = new Font("Arial",Font.PLAIN,15);
		g.setFont(f2);
		FontMetrics fm2 = g.getFontMetrics();
		for (int i = 0; i < boutons.size(); i++)
		{
			Rectangle2D rect =boutons.get(i); 
			float x = (float) rect.getX();
			float  y = (float)rect.getY();
			float  w = (float)rect.getWidth();
			float  h = (float)rect.getHeight();
			g.drawRoundRect((int)x, (int)y, (int)w, (int)h, 10, 30);
			g.drawString("Button_"+i,(int)x+w/2, (int)y+h/2);
		}
		g.drawString("Nouvelle partie",(getWidth()-fm2.stringWidth("Nouvelle partie"))/2,20+(buttonHeight+fm2.getAscent())/2);
 
	}
 
	@Override
	public void componentHidden(ComponentEvent e)
	{
		// TODO Auto-generated method stub
 
	}
 
	@Override
	public void componentMoved(ComponentEvent e)
	{
		// TODO Auto-generated method stub
 
	}
 
	@Override
	public void componentResized(ComponentEvent e)
	{
		initboutons();
 
	}
 
	@Override
	public void componentShown(ComponentEvent e)
	{
		// TODO Auto-generated method stub
 
	}
 
	public synchronized ArrayList<Rectangle2D> getBoutons()
	{
		return boutons;
	}
 
	public synchronized void setBoutons(ArrayList<Rectangle2D> boutons)
	{
		this.boutons = boutons;
	}
 
 
}

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
import java.awt.*;
import javax.swing.*;
import java.awt.event.*;
import java.awt.geom.*;
import java.util.ArrayList;
 
public class Example extends JFrame {
 
	//private Container cont = new Container();
	private Menu lemenu = new Menu();
 
	public Example() {
		super("Exemple");
		setDefaultCloseOperation(EXIT_ON_CLOSE);
 
	//	setLocationRelativeTo(null);
 
		//lemenu.setPreferredSize(new Dimension(800,600)); // A rajouter.
		//lemenu.setMinimumSize(new Dimension(800,600)); // A rajouter.
 
		getContentPane();
		setLayout(new BorderLayout());
		add(lemenu, BorderLayout.CENTER);lemenu.addMouseListener(new MouseAdapter() {
			public void mouseClicked(MouseEvent e) { 
				System.err.println("Point cliqué "+e.getX()+","+e.getY());
				ArrayList<Rectangle2D> rects = lemenu.getBoutons();
				for (int i = 0; i < rects.size(); i++)
				{
					Rectangle2D r = rects.get(i);
					if(r.contains(e.getX(),e.getY())) 	System.out.println("Je clique sur bouton "+i);
				}
 
			}
		});
		setSize(800,600);
		setVisible(true);
	}
 
	public static void main(String[] args) {
		new Example();
	}
}

Ceci devrait te permettre de repartir vers de nouveau horizon (contenant moultes porcheries certes, car le midi fut boissonique ... j'ai fait ca rapidos).

Si tu t'en inspire tu pourras faire un beau petit menu d'après moi ;)

PS : Veille bien a fair eattention quand tu utilise ton null layout, j'suis passé a un borderlayout dans mon exemple car ton exemple etait un peu etrange. (meme si j'ai utilisé null layout pdt tout un temp).

09/02/2010, 16h44
greasf

Merci pour la portion de code ! Je vais regarder ça attentivement, et bon anniversaire ! :ccool::mouarf: (en retard, certes^^).

Pour finir la discussion, quelqu'un a une réponse à ce fameux décalage d'ordonnée?

Merci =)
09/02/2010, 16h46
wax78

Bah mieux vaut tard que jamais comme on dit ;) Merci a toi.

Pour le décalage je pense me souvenir (mais plus certains) que c'etait a cause du null layout et des setbounds (enfin un truc dans ce gout la je me rapelle plus trop).
14/02/2010, 18h53
greasf
Bonsoir à tous,

j'ai encore deux petites questions :

au niveau des arguments :
Citation:
Pourquoi l'accesseur n'a pas besoin de prendre d'argument, alors que la méthode retourne l'objet en question?
Par exemple, ici, pourquoi n'écrit-on pas

Code:

1 2 3 4 public Rectangle2D getRect2D(Rectangle2D rect2d) { return rect2d; }

(je pause cette question car il n'y a pas que l'accesseur qui fonctionne comme cela. Pas mal de méthodes utilisent des variables pour 'travailler dessus' alors que ces dernières ne passent pas spécialement en argument, et ça, je ne vois pas du tout pourquoi)
D'autre part, à quoi est dû le décalage?

Merci et bonne soirée à tous
14/02/2010, 21h32
wax78

Elle m'a l'air plus que suspecte cette fonction, voire totalement inutile ;)
14/02/2010, 22h05
plegat

Salut

Citation:

Envoyé par greasf

Pourquoi l'accesseur n'a pas besoin de prendre d'argument, alors que la méthode retourne l'objet en question?

(je pause cette question car il n'y a pas que l'accesseur qui fonctionne comme cela. Pas mal de méthodes utilisent des variables pour 'travailler dessus' alors que ces dernières ne passent pas spécialement en argument, et ça, je ne vois pas du tout pourquoi)

Il va falloir revoir tes cours de programmation objet en java... comment marche une classe, qu'est-ce qu'un accesseur/mutateur, les données membres, les méthodes, etc etc...

Pourquoi passer une donnée membre en paramètre alors que la méthode travaille directement sur l'objet?
15/02/2010, 12h20
greasf

Citation:

Envoyé par plegat

Pourquoi passer une donnée membre en paramètre alors que la méthode travaille directement sur l'objet?

Peux-tu développer cette phrase? Elle m'a (peut-être) permis de comprendre, mais je ne suis pas bien sur.

Merci d'avance
15/02/2010, 13h53
tchize_

la question est: pourquoi passer en paramètre l'objet à récupérer. Le but est justement de le récupérer, donc a priori on ne l'a pas et je vois pas ce qu'on pourrais passer en paramètre à ton getter!

En fait, c'est la question qui est plutot stupide (no offense) !

Pour eviter de devoir creer un objet intermediaire temporaire inutile bien sur ! Apres d'aucuns vont se pleindre que les libs 2D* ou 3D rament de la mort et ont une grosse empreinte memoire. Faire des getters et des setters ne dispense pas de devoir parfois optimiser les performances et l'usage de la memoire aux endroits ou cela peut s'averer necessaire.

Allez on recapepette : on ne peut pas retourner l'objet interne car il est mutable donc il faudrait en faire une copie. Or faire une copie veut dire allocation puis plus tard liberation ce qui peut etre un trou de perfomance. La solution est donc simple : faire que l'appelant puisse fournir un conteneur reutilisable. Bien sur des precautions doivent etre prises pour s'assurer que le conteneur soit dans un etat approprie (non-null) et l'initialiser le cas echeant si necessaire.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 public Rectangle2D getRect2D() { return getRect2D(null) ; } public Rectangle2D getRect2D(Rectangle2D rect2d) { if (rect2D == null) { rect2D = new Rectangle2D.Float(); } rect2D.setRect(...); return rect2d; }
Allouer et liberer de la memoire restent des operations couteuses, quand il est necessaire d'optimiser on doit se tourner vers la reutilisation d'objets, etc.
Code:

1 2 3 4 5 Rectangle2D bounds = null; for (Trucs t : trucs) { bounds = t.getRect2D(bounds); [...] }
*dans ces genres de trous de performance on peut entre autre noter l'utiliser des Dimension et Insets dans AWT/Swing ou encore des Boundaries en JavaFX ou a chaque calcul de layout/affichage des tonnes et des tonnes d'objets temporaires sont crees.

20/02/2010, 16h06
greasf

Ah !

Merci pour les réponses :ccool::mrgreen: