Discussion :

[bigram]

Bonsoir,

Je suis débutant en Java,et je m'intéresse entre autre aux composants graphiques avec swing...

je vous explique mon probleme.
J'ai créé une fenetre avec un objet JFrame, ensuite j'ai créé un objet Panneau de JPanel et je voudrais afficher 32 carrés gris(4 X 8) dans ce panneau.

Je ne sais pas du tout si je suis pres de la solution ou pas.

Pouvez vous m'aider?

Merci d'avance.

PS: ci-dessous la classe Panneau :

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
public class Panneau extends JPanel{
	Spot tab[][]= new Spot[4][8];
	Spot[] sp = new Spot[4];	
	public void paintComponent(Graphics g){		
		g.setColor(Color.black);
		g.fillRect(100, 115, 200, 350);			
		for (int j = 0;j<8;j++)
		{
			for (int i=0;i<4;i++)
			{							
				sp[i].setPosX(120+(i*50));
				sp[i].setPosY(135+(40*j));
				sp[i].setCouleur(Color.gray);
 
				this.getGraphics().setColor(Color.gray);
				this.getGraphics().fillRect(sp[i].getPosX(),sp[i].getPosY(), 40, 40);
				tab[j][i]=sp[i];
				//this.repaint();
			}			
		}		
	}
}

et ici la classe Spot ne contient que les getters et setters (posX et posY pour les coordonnées en x et y.

[Ivelios]

je suis pres de la solution

Petit complément d'information :
Pour afficher 32 carrés gris il faut effectivement redéfinir paintComponent.(Ou alors utiliser un GridLayout avec des JLabels ou des JButtons)
Dans ton exemple il y a plusieurs erreurs/trucs bizarres/lacune qui sautent au yeux.
Pourquoi utiliser 2 tableaux?

Code :

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

1
2
Spot tab[][]= new Spot[4][8];
Spot[] sp = new Spot[4];

sp est inutile selon moi.
Tu n'initialise pas les éléments de tab[][].
Tu définie la taille des cases = 40. Et si on redimensionne la fenêtre?
Tu devrais prévoir cela, il n'y a quasiment rien à ajouter

Au final, pourquoi ne pas faire un simple et grand carré gris
Quelle est la finalité? pouvoir cliquer dessus par la suite?

[bigram]

Merci Ivelios pour la réponse rapide.

En fait, la finalité est de pouvoir changer la couleur de ces carrés gris en fonction d'un timer(je n'ai pas encore fait cette partie, chaque chose en son temps ).

Pour le tableau sp, vous avez raison, après avoir relu la théorie,en effet, 1 tableau a 2 dimensions suffit.

Avec quelles données initialiser le tableau tab[][]?

et si je remplace dans cette ligne =>this.getGraphics().fillRect(sp[i].getPosX(),sp[i].getPosY(), 40, 40);

les sp[i] par tab [i][j], ca ne va pas beaucoup mieux.

Et la taille des carrés (40) peut effectivement etre déclarée avec une variable dépendante de la taille de la fenetre

Merci d'avance pour les eclaircissements

[Ivelios]

Si le but final est de modifie les couleurs toutes les secondes par exemple alors :
sp doit être un tableau de couleurs
Et tous les "clock" d'horloge il faudra modifier le tableau et repaint();
Un peu 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
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
import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.*;
/**
 *
 * @author Ivelios
 */
public class Panneau extends JPanel{
 
    private Color[][] couleurs;
 
    /**Constructeur*/
    public Panneau(){
        couleurs = new Color[4][8];
        initialisationCouleurs();
 
        // Changement des couleurs
        ActionListener action = new ActionListener(){
            public void actionPerformed(ActionEvent e) {
                initialisationCouleurs();
                repaint();
            }
        };
 
        Timer t = new Timer(1000,action);//Toutes les secondes (1000 ms) déclenche l'action
        t.start();//Démarre le timer
    }
 
 
    public void initialisationCouleurs(){
        for(int i=0;i<4;i++){
            for(int j=0;j<8;j++){
                //On choisit une couleur au hazard pour remplir le tableau
                int random = (int)(Math.random()*5);
                Color c = null;
                switch(random){
                    case 0: c = Color.YELLOW; break;
                    case 1: c = Color.RED; break;
                    case 2: c = Color.BLACK; break;
                    case 3: c = Color.BLUE; break;
                    case 4: c = Color.GREEN; break;
                }
                //Initialisation du tableau de couleur
                couleurs[i][j] = c;
            }
        }
    }
 
    public void paintComponent(Graphics g){
        //Grille des couleurs
        for (int i = 0;i<4;i++){
            for (int j=0;j<8;j++){
                 g.setColor(couleurs[i][j]);
                 g.fillRect(i*40,j*40, 40, 40);
            }
        }
 
    }
 
    public static void main(String[] arrrrrrrgs){
        JFrame fenetre = new JFrame();
        fenetre.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        fenetre.setContentPane(new Panneau());
        fenetre.setSize(200, 400);
        fenetre.setVisible(true);
    }
}

[bigram]

ce n'est pas exactement ce que je veux faire, mais les bases y sont pour pouvoir m'améliorer.

J'ai encore une question : la méthode paintComponent est-elle appelée à l'exécution de repaint()?

j'ai encore un peu de mal a savoir quand elle s'exécute précisément.

Le tableau de couleurs est intéressant mais il faudrait que je le modifie car seules 2 couleurs m'interessent, par contre la position de chaque carré est importante, d'où mon tableau.
il faut que je sache la position de chaque carré et sa couleur.

Merci encore.

[bouye]

J'ai encore une question : la méthode paintComponent est-elle appelée à l'exécution de repaint()?

j'ai encore un peu de mal a savoir quand elle s'exécute précisément.

Non, repaint() met juste un flag indiquant que ce composant a besoin d'être repeint et et retourne immédiatement.

Le composant sera réaffiché (et donc paintComponent() appelé) au prochain (ou à l'un des prochains) cycle de réaffichage.

[bigram]

Merci bouye pour la réponse.

Peux tu me dire pourquoi cette méthode ne fonctionne pas?

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 initCarre(){
        //System.out.println("Dans la methode initCarre");
        for(int j=0;j<8;j++)
        {
            for(int i=0;i<4;i++)
            {        
 
                tab[i][j].setCouleur(Color.gray);
                tab[i][j].setPosX(120+(i*50));
                tab[i][j].setPosY(135+(j*50));                
            }            
        }    
        this.repaint();
    }

tab est déclarée comme il le faut pourtant (voir mon 1er post).

[Ivelios]

pourquoi cette méthode ne fonctionne pas?

Avec le message d'erreur s'aurait été mieux mais à première vu je dirais un NullPointerException.

Faisons simple :
Tu initialise bien ton tableau : Spot tab[][]= new Spot[4][8];
Mais ton tableau est actuellement VIDE. Il faut donc initialiser chaque case de ton tableau. Et créer 4*8 instance de Spot que tu stockeras dans le tableau.

Code :

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

1
2
3
4
tab[i][j] = new Spot();//<- Ajouté la ligne pour créer le spot
tab[i][j].setCouleur(Color.gray);
tab[i][j].setPosX(120+(i*50));
tab[i][j].setPosY(135+(j*50));

[bigram]

Encore merci Ivelios,

tu as tout a fait raison, l'erreur etait un NullPointerException.
et en rajoutant la ligne que tu m'as donnée, ca fonctionne très bien.
Je vais donc essayer de continuer seul.
En tout cas,ce forum est super et plein de gens sympas et pro.

A très vite...