Discussion :

[olivier252]

Bonjour,

Je suis tout nouveau en java et je d�bute l'interface SWT depuis une semaine, et notre formateur nous demande de pouvoir dessiner des cercles � la vol�e dans un composant Canvas afin de reproduire de fa�on tr�s tr�s modeste les principes d'un logiciel de type Paint.

En cours nous avons utilis� le plugin org.eclipse.swt.graphics.Rectangle pour tracer des rectangles mais je ne sais pas comment m'y prendre pour les cercles

Voici ce que nous avons fait en cours pour cr�er un rectangle :

Nous avons d�fini une classe abstraite avec les crit�res communs � toutes les formes, puis nous avons �tendu cette classe avec une classe rectangle. Ensuite, dans une classe Canvas, nous avons utilis� l'interface MouseListener et PaintListener pour pouvoir dessiner les rectangles � la souris. J'aurais voulu r�ussir � faire une classe Cercle qui reprenne la logique de ma classe rectangle. Jen e vois pas trop comment aborder le probl�me

La classe abstraite

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
package paintrcp.biz;
 
import org.eclipse.swt.graphics.GC;
import org.eclipse.swt.graphics.Point;
import org.eclipse.swt.graphics.RGB;
 
 
public abstract class Shape {
	private String name;
	private RGB border = new RGB(0,0,0);
	private RGB fill = new RGB(255,255,255);
 
	public Shape(String name, Point begin, Point end, RGB border, RGB fill) {
		setName(name);
		setBounds(begin, end);
		setBorder(border);
		setFill(fill);	
	}
 
	public RGB getBorder() {
		return border;
	}
 
	public void setBorder(RGB border) {
		Colors.copy(this.border, border);
	}
 
	public RGB getFill() {
		return fill;
	}
 
	public void setFill(RGB fill) {
		Colors.copy(this.fill, fill);
	}
 
	public String getName() {
		return this.name;
	}
 
	public void setName(String name) {
		if(name.length()<2) {
			throw new IllegalArgumentException("Name must have at least 2 characters");
		}
		this.name=name;
	}
 
	public void setBounds(Point begin, Point end) {
		if(begin==null || end==null) {
			throw new NullPointerException("begin and end cannot be null");
		}
		if(begin.equals(end)) {
			throw new IllegalArgumentException("begin and end must be different");
		}
	}
 
	public abstract void draw(GC gc);
}

La classe m�tier Rectangle

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
package paintrcp.biz;
 
import org.eclipse.swt.graphics.Color;
import org.eclipse.swt.graphics.GC;
import org.eclipse.swt.graphics.Point;
import org.eclipse.swt.graphics.RGB;
 
public class Rectangle extends Shape {
 
private org.eclipse.swt.graphics.Rectangle rect;
 
	public Rectangle(String name, Point begin, Point end, RGB border, RGB fill) {
		super(name, begin, end, border, fill);
	}
 
	@Override
	public void draw(GC gc) {
		gc.setBackground(new Color(gc.getDevice(), getFill()));
		gc.setForeground(new Color(gc.getDevice(), getBorder()));
		gc.fillRectangle(rect);
		gc.drawRectangle(rect);
	}
 
	@Override
	public void setBounds(Point begin, Point end) {
		super.setBounds(begin, end);
		rect = new org.eclipse.swt.graphics.Rectangle(
				Math.min(begin.x,end.x),
				Math.min(begin.y, end.y),
				Math.abs(begin. x-end.x),
				Math.abs(begin. y-end.y)
			);
	}
}

Merci pour votre aide

[Mickael_Istria]

Spoiler: si tu cherches dans le code du bundle org.eclipse.swt.examples, qui tu peux trouver sur le repo Git de SWT, alors tu y verras un "EllipseFigure.java" qui s'apparente beaucoup a ce qui est demande.
Sinon, pour tracer des ovales/ellipses et cas particulier du cercle, tu peux utiliser gc.drawOval.

[olivier252]

Bonjour Mickael, merci pour ta r�ponse

En attendant, j'ai cr�e ceci mais le code est faux...mon probl�me est que ma m�thode draw ne prend qu'un param�tre lors de sa d�claration dans la classe abstraite.

Je suis au point mort...Merci pour l'info du git

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
package paintrcp.biz;
 
import org.eclipse.swt.graphics.Color;
import org.eclipse.swt.graphics.GC;
import org.eclipse.swt.graphics.Point;
import org.eclipse.swt.graphics.RGB;
 
public class Cercle extends Shape{
 
	public Cercle(String name, Point begin, Point end, RGB border, RGB fill) {
		super(name, begin, end, border, fill);
	}
 
	@Override
	public void draw(GC gc, Point begin, Point end) {
		gc.setBackground(new Color(gc.getDevice(), getFill()));
		gc.setForeground(new Color(gc.getDevice(), getBorder()));
		drawOval(begin.x,begin.y,end.x,end.y);
		fillOval(begin.x,begin.y,end.x,end.y);
	}
 
	@Override
	public void setBounds(Point begin, Point end) {
		super.setBounds(begin, end);
		cercle = new org.eclipse.swt.graphics.Rectangle(
				Math.min(begin.x,end.x),
				Math.min(begin.y, end.y),
				Math.abs(begin. x-end.x),
				Math.abs(begin. y-end.y)
			);
	}
}

Bonne soir�e

[joel.drigo]

Salut,

En effet, la m�thode draw() de la classe abstraite ne prend que le contexte graphique en param�tre, parce que la classe repr�sente une forme qui connait sa position et sa taille (les bounds). Pas besoin donc de pr�ciser en param�tre autre chose pour la dessiner. Et la d�claration de la m�thode abstraite coupl� au @override t'�vite justement de faire autre chose.

La m�thode public void setBounds(Point begin, Point end) { est cens� permettre justement de pr�ciser ces bounds. Seulement, pour pouvoir les utiliser dans une autre m�thode, en l�occurrence la m�thode draw(), il faut les stocker dans la classe.

Dans la classe Rectangle, c'est la variable rect qui stocke les bounds du rectangle. Dans la m�thode setBounds() de Cercle tu tentes de stocker ces bounds dans la variable cercle, mais tu ne la d�clares pas. Il suffit de le faire et de l'utiliser avec gc.fillOval() et gc.drawOval(). Attention � la fa�on d'utiliser les informations d'un rectangle (la variable cercle est du type rectangle) pour d�finir les coordonn�es d'un cercle (ou ellipse). D'ailleurs, par ailleurs, il existe une api draw2d qui te donnera la possibilit� de dessiner directement des formes (donc de stocker une forme directement adapt�e (comme une ellipse) plut�t que de dessiner une ellipse � partir de donn�es d'un rectangle).

Attention � dessiner le contour apr�s la forme "remplie", surtout si les couleurs sont diff�rentes, sinon cette derni�re va recouvrir en partie le contour.

Attention �galement � disposer correctement les ressources en SWT. La classe Color est une ressource et doit �tre lib�r�e par l'appel de la m�thode dispose(). A chaque redessin, la m�thode draw() va �tre appel�e et deux nouvelles instances de Color vont �tre cr��e, ce qui r�servera deux handle syst�me. Ces handles �tant limit�s, tu pourras avoir des soucis � un moment, surtout si ton programme plante. La classe ColorRegistry peut t'aider � g�rer cette probl�matique.

[olivier252]

Bonjour Jo�l, merci d'avoir pris le temps de me donner une r�ponse aussi compl�te.

Ta contribution a r�ellement �t� tr�s pr�cieuse...hier soir � force de bidouiller dans tous les sens, j'ai post� le bout de code sans la variable cercle en effet.
Voici l'extrait de code retravaill� mais qui contient encore des erreurs :

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
public class Cercle extends Shape {
 
	private org.eclipse.swt.graphics.Rectangle cercle;
	private Point begin;
	private Point end;
 
	public Cercle(String name, Point begin, Point end, RGB border, RGB fill) {
		super(name, begin, end, border, fill);
	}
 
	@Override
	public void draw(GC gc) {
		gc.setBackground(new Color(gc.getDevice(), getFill()));
		gc.setForeground(new Color(gc.getDevice(), getBorder()));
		gc.fillOval(begin.x, begin.y, end.x, end.y);
		gc.drawOval(begin.x, begin.y, end.x, end.y);
                //J'ai essayé ça aussi :
                gc.fillOval(cercle);
		gc.drawOval(cercle);
	}
 
	@Override
	public void setBounds(Point begin, Point end) {
		super.setBounds(begin, end);
		cercle = new org.eclipse.swt.graphics.Rectangle(
				Math.min(begin.x, end.x),
				Math.min(begin.y, end.y),
				Math.abs(begin.x-end.x),
				Math.abs(begin. y-end.y)
			);
	}
}
 
	}

Je viens de comprendre en effet que la classe Rectangle permet aussi de cr�er des cercles. Je me m�lange aussi les pinceaux dans les param�tres

Quand tu dis de faire attention � dessiner d'abord le contour, tu veux dire qu'il faut d'abord executer drawOval() avant fillOval() ?

Merci enfin pour l'info concernant l'api. Je pense que, comme nous sommes des d�butants qui d�butent, le formateur a d� juger utile de passer par la classe Rectangle dans un premier temps, mais je mets de c�t�.

[joel.drigo]

Voici l'extrait de code retravaill� mais qui contient encore des erreurs :

Oui, parce que tu utilises les deux attributs begin et end de la classe Cercle pour dessiner. Alors que ces variables ne sont pas affect�es donc de valeur null.

C'est la variable cercle qui sert � stocker les coordonn�es du cercle donc c'est cette variable que tu dois r�f�rencer dans fillOval et drawOval. Tu peux bien s�r stocker les coordonn�es dans begin et end au lieu de les stocker dans cercle, mais il faut �tre juste coh�rent : utiliser la bonne variable et stocker dans la bonne variable.

Tu �tais un peu sur la voie en tentant :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
gc.fillOval(cercle);
gc.drawOval(cercle);

Mais ces deux m�thodes ne prennent pas des org.eclipse.swt.graphics.Rectangle en param�tre, donc �a ne peut pas fonctionner. Ce qui est trompeur c'est que la m�thode fillRectangle(Rectangle) existe effectivement en plus de fillRectangle(x,y,width,height). Normalement, si tu utilises un EDI, il devrait t'aider � voir que la m�thode n'existe pas pour tel ou tel type de param�tre.

Regarde la doc : fillOval (et drawOval) prend 4 int en param�tre, le x et y du point sup�rieur gauche, et les largeur et hauteur. Informations stock�es dans cercle :

• cercle.x stocke le x du point sup�rieur gauche du rectangle
• cercle.y stocke le y du point sup�rieur gauche du rectangle
• cercle.width stocke le width (largeur) du rectangle
• cercle.height stocke le height (hauteur) du rectangle

Il suffit donc de simplement utiliser ces informations comme param�tre de fillOval (et drawOval) :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

gc.fillOval(cercle.x, cercle.y, cercle.width, cercle.height);

Quand tu dis de faire attention � dessiner d'abord le contour, tu veux dire qu'il faut d'abord executer drawOval() avant fillOval() ?

En fait je disais plut�t l'inverse : dessiner d'abord la forme "remplie" puis le contour, donc ex�cuter fillOval() avant drawOval() comme dans la classe paintrcp.biz.Rectangle d'ailleurs.

[olivier252]

Bonjour Joel,

Merci (encore) pour ta r�ponse et surtout ta patience ! Je suis en cours ce matin, mais je vais travailler sur ta r�ponse ce soir et te faire un retour plus d�taill�
Bonne journ�e

[olivier252]

Du coup un petit retour comme pr�vu.

J'ai pu travailler ta r�ponse (pas dur me diras-tu vu que tu m'as fourni la r�ponse). Je suis d��u de ne pas avoir trouv� tout seul alors que tu 'avais mis sur la voie...J'aurais au moins la capacit� � comprendre mes erreurs gr�ce � toi :
- Initialiser des objets de la classe Point � null
- j'ai voulu utiliser les param�tres de la classe Point dans ma m�thode draw() sans tenir compte des valeurs allou�es par la m�thode setBounds(), et toujours dans le chapitre de la confusion de param�tres, je n'avais pas fait attention que la classe rectangle prenait une hauteur et une largeur.
Beaucoup d'incompr�hensions au total...

Cela �tant, toujours dans mon esprit de d�butant bidouilleur, j'ai tent� de remplacer la variable "cercle" par le mot cl� "this" comme ceci.

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

gc.drawOval(this.x, this.y, this.width, this.height);

Dans mon raisonnement, l'objet this fait en effet r�f�rence � l'objet courant, et donc aurait pu se substituer � "cercle". J'ai �videment tout faux, peut-�tre sauras tu me dire pourquoi. Merci encore pour tes interventions sur mon sujet et du temps que tu y as consacr�. Je mets le sujet en r�solu

[joel.drigo]

Cela �tant, toujours dans mon esprit de d�butant bidouilleur, j'ai tent� de remplacer la variable "cercle" par le mot cl� "this" comme ceci.

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

gc.drawOval(this.x, this.y, this.width, this.height);

this d�signe l'instance courante, donc un objet de classe Cercle. Il n'y a pas d'attribut x, y, width ou height dans cette classe, donc this.x, this.y, this.width et this.height n'ont aucun sens.

Dans mon raisonnement, l'objet this fait en effet r�f�rence � l'objet courant

,
oui

et donc aurait pu se substituer � "cercle"

donc non

puisque cercle, c'est en fait this.cercle, donc �a ne peut pas �tre this, sauf si � un moment on avait fait this.cercle = this, ce qui n'est pas possible puisque this est du type Cercle alors que cercle du type Rectangle qui ne sont absolument pas compatibles (l'un n'est pas la sp�cialisation ou la g�n�ralisation de l'autre). En plus � quoi �a servirait qu'on stocke this dans cercle ?

[olivier252]

Ok, voil� mati�re � r�viser sur ce script pour le week end. Je vais tenter de g�n�rer d'autre formes en guise d'entrainement. Bravo et merci pour ta p�dagogie.
Bon we