Discussion :

[wirtouce]

Bonjour, je viens vous demandez conseils au sujet de ceci :


-class Conteneur

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
import java.awt.Color;
import java.awt.Graphics;
 
import javax.swing.*; 
public class Conteneur extends JPanel{
 
int[] combinaisonAtrouver = new int[4];
int[] combinaisonEntree = new int[4];
int i=0;
Graphics g = getGraphics();{
 
	combinaisonAtrouver[0]=1;
	combinaisonAtrouver[1]=5;
	combinaisonAtrouver[2]=6;
	combinaisonAtrouver[3]=2;
 
	combinaisonEntree[0]=7;
	combinaisonEntree[1]=5;
	combinaisonEntree[2]=3;
	combinaisonEntree[3]=2;
 
	for (i=0;i==3; i++){
 
		if (combinaisonEntree[i] == combinaisonAtrouver[i]){
			g.setColor(Color.blue);
			g.fillOval(100+50*i, 200, 50, 50);
 
 
		} else {
			g.setColor(Color.magenta);
			g.fillOval(100+50*i, 200, 50, 50);
			 }}}}

-class Fenetre

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
import javax.swing.JFrame;
 
public class Fenetre extends JFrame {
 
	private Conteneur conteneurPrincipal = new Conteneur();
 
	// Constructeur d'objet de la classe Fenetre
	public Fenetre() {
		setTitle("Mastermind");
		setSize(800, 800);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setLocationRelativeTo(null);
		setVisible(true);
		setResizable(false);
		setAlwaysOnTop(true);
		setContentPane(conteneurPrincipal);
	}
}

-class principale

Code :

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

1
2
3
4
5
6
7
public class Principale {
 
	public static void main(String[] args) {
		Fenetre fenetrePrincipale = new Fenetre();
	}
 
}

Or quand je lance le programme... la fenêtre est complètement grise :'( alors qu'il devrait se tracer de cercles colorés bleu quand les valeurs contenues au rang i sont égales et magenta quand elles ne le sont pas. Quelle est mon erreur?

[joel.drigo]

Salut,

1. Ce n'est pas du tout comme ça qu'on dessine dans un composant SWING. Là, tu dessines dans le bloc de préinitialisation, qu'on utilise rarement (préférer le constructeur de la classe) : même si on pouvait dessiner comme ça, ce bloc (ou le constructeur) ne s'exécutant qu'une seule fois au moment de la création de l'instance, donc bien avant d'être mis dans la fenêtre, ça ne risque pas de dessiner quelque chose de visible dans la fenêtre.
   On dessine dans la méthode paint(), ou l'une des méthodes qui est appelée indirectement par paint(), par exemple paintComponent() : pour ce faire, on redéfinit la méthode.
2. ensuite le programme ne rentrera jamais dans ta boucle for. Un for (initialisation; condition; expression) itère tant que condition est vraie. A l'initialisation, i vaut 0, et la condition est i==3, donc fausse, donc on n'entre pas dans la boucle. A noter qu'il vaut mieux toujours déclarer les variables au plus près de leur utilisation, et uniquement dans leur scope d'utilisation, ce qui évite des erreurs de confusion de variables par la suite.
   Donc au moins faire i<=3 ou i<4, quoiqu'il vaudra mieux utiliser ici une comparaison à la longueur des tableaux, ce qui fera que ça fonctionnera dans tous les cas (y compris si on change la taille du tableau, ou si on ajoutait un constructeur pour lequel la taille du tableau serait paramétrée, donc non fixe et connue par avance.
3. notons également qu'il vaut mieux toujours préparer une fenêtre entièrement avant de l'afficher (setVisible(true)). Ici ça fonctionne, parce que le setContentPane() fait implicitement ce qu'il faut pour mettre à jour la fenêtre. Mais il vaut mieux prendre l'habitude de procéder dans l'ordre : préparer, afficher (sauf en cas de modification dynamique de l'affichage, au quel cas on fera un revalidate() et un repaint() après la modification pour avertir SWING de faire ce qu'il faut pour mettre à jour l'affichage). Dans le cas contraire, tu pourras avoir le même effet de fenêtre grise, ou de parties non visibles (ou qui le deviennent seulement si on redimensionne la fenêtre par exemple).

Donc, en résumé, voici le code qui fonctionne :

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
public class Conteneur extends JPanel {
 
	private final int[] combinaisonAtrouver;
	private final int[] combinaisonEntree;
 
	public Conteneur() {
                combinaisonAtrouver = new int[4];
		combinaisonAtrouver[0] = 1;
		combinaisonAtrouver[1] = 5;
		combinaisonAtrouver[2] = 6;
		combinaisonAtrouver[3] = 2;
 
                combinaisonEntree = new int[4];
		combinaisonEntree[0] = 7;
		combinaisonEntree[1] = 5;
		combinaisonEntree[2] = 3;
		combinaisonEntree[3] = 2;
	}
 
	@Override
	protected void paintComponent(Graphics g) {
		super.paintComponent(g);
 
		for (int i = 0; i<combinaisonAtrouver.length && i<combinaisonEntree.length; i++) { // ou for (int i = 0; i <= 3; i++) ou for (int i = 0; i < 4; i++)
 
			if (combinaisonEntree[i] == combinaisonAtrouver[i]) {
				g.setColor(Color.blue);
				g.fillOval(100 + 50 * i, 200, 50, 50);
 
			} else {
				g.setColor(Color.magenta);
				g.fillOval(100 + 50 * i, 200, 50, 50);
			}
		}
 
	}
 
}

[wirtouce]

Merci ça marche nickel !