tableau d'imagesen JAVA Résolu/Fermé

Question

Bonjour ! Je post afin de m'en remettre à vous car je n'arrive pas à parcourir un tableau et afficher les images situées dans un autre tableau en fonction de ce qui est lu dans le premier..

Peut-être est ce la fatigue mais je ne vois pas ou est mon erreur ? Pourriez vous m'aider s'il vous plait ?

Voici le dit-code :
public static void draw(int[][] level, BufferedImage[] spriteSheet){ 		int readLevel = 0, readSprite = 0, q = 0;		int x = 0, y = 0;		JPanel levelPanel = new JPanel();		JLabel [] levelLabel = new JLabel [500];		Image [] tileLevel = new Image [500];		ImageIcon [] iconLevel = new ImageIcon [500];  		levelPanel.setLayout(null);   		for(int i = 0; i < 25; i++){			for(int j = 0; j < 20;j++){ 				readLevel = level[i][j]; 				switch(readLevel){ 				case 0:					readSprite = 0;					break;				case 1:					readSprite = 1;					break;				default:					break;				} 				tileLevel [q] = Toolkit.getDefaultToolkit().createImage(spriteSheet[readSprite].getSource());				iconLevel [q] = new ImageIcon(tileLevel[q]);				levelLabel[q].setIcon(iconLevel[q]);				levelLabel[q].setBounds(x,y,32,32);				levelPanel.add(levelLabel[q]);				q++; 				if( x <= 768){					x += 32;				}				else{ 					x = 0;					y += 32;				} 			} 		}  		window.getMainFrame().setContentPane(levelPanel);		window.getMainFrame().validate(); 	}

Merci d'avance !

KX · Answer

Bonjour,

Déjà il faut éviter les méthodes qui font tout, il vaut mieux les diviser.
Là tu as à la fois du traitement de données, de l'affichage... ça ne devrait pas être au même endroit.

De plus, il y a des "valeurs magiques" (ex: new JLabel[500], pourquoi 500 ?), là encore c'est à éviter. Il faut préférer passer ces valeurs en paramètres, ou les enregistrer dans des constantes, voire les recalculer.
Par exemple, 500, c'est 25*20, les dimensions de level[i][j], donc autant récupérer ces valeurs directement depuis level.

Ensuite, ton switch(readLevel) est maladroit, si readLevel[i][j] ne peut contenir que 0 ou 1, alors autant faire un boolean[][] plutôt qu'un int[][].
S'il peut y avoir d'autres valeurs, alors tu as un problème avec ton default qui ne définit pas de valeur pour readSprite.

Pour ce code tileLevel [q] = Toolkit.getDefaultToolkit().createImage(spriteSheet[readSprite].getSource()); Notons déjà que la manière dont q est incrémenté est maladroite, le q++ est caché au milieu du code, il devrait être plus visible, par exemple au niveau du for. Ensuite tu fais 500 createImage alors que readSprite ne peut prendre que deux valeurs (0 ou 1), tu ferais mieux de ne créer que 2 fois ces valeurs. Même si en réalité c'est inutile puisque BufferedImage est déjà une Image donc tu peux l'utiliser tel quel sans avoir à passer par le Toolkit.

levelLabel[q].setBounds(x,y,32,32);

if( x <= 768){
 x += 32;
}
else{
 x = 0;
 y += 32;
}
Là il y a un problème, tu devrais te servir d'un GridLayout plutôt que de faire levelPanel.setLayout(null); ça simplifiera grandement les choses...

Enfin, tu n'as pas besoin de tous tes tableaux levelLabel, tileLevel, IconLevel, la seule chose qui t'intéresse c'est le level Panel final... du coup ça règle aussi le problème d'incrémentation de q.

Ce code devrait normalement faire plus ou moins la même chose :

public static void draw(int[][] level, BufferedImage[] spriteSheet) {

JPanel levelPanel = buildLevelPanel(level, spriteSheet);

window.getMainFrame().setContentPane(levelPanel);
    window.getMainFrame().validate();
}

public static JPanel buildLevelPanel(int[][] level, BufferedImage[] spriteSheet) {

int rows = level.length;
    int cols = level[0].length;

JPanel levelPanel = new JPanel(new GridLayout(rows, cols));

for (int row = 0; row < rows; row++) {
        for (int col = 0; col < cols; col++) {
            int readSprite = level[row][col];
            Image tileLevel = spriteSheet[readSprite];
            ImageIcon iconLevel = new ImageIcon(tileLevel);
            JLabel levelLabel = new JLabel(iconLevel);
            levelPanel.add(levelLabel);
        }
    }

return levelPanel;
}
La confiance n'exclut pas le contrôle

MrMahus · Answer

Est ce que lorsque l'on créer une fenêtre via JFrame la taille que l'on rentre ( 800*600 par exemple) inclue la fenêtre de son bord haut gauche extérieur ou du bord intérieur de la fenêtre ?

Tableau d'imagesen JAVA

2 réponses

Discussions similaires