sokoban java Fermé

Question

Bonjour, j'essaye de faire un jeu de sokoban (le jeu ou il faut ranger les caisse) mais j'ai un problème. si vous trouvez l'erreur ce serrait sympas svp. 

//terrain.java 

import java.awt.Graphics; 
import java.awt.Image; 
import java.io.File; 
import java.io.IOException; 

import javax.imageio.ImageIO; 
import javax.swing.JPanel; 


public class terrain extends JPanel{ 
 int posX; 
 int posY; 
 int Xmax= 500; 
 int Ymax= 400; 
 Image img; 
 public terrain(){ 
   
 } 
 public void paintComponent(Graphics g){ 
  setX(0); 
  setY(0); 
  for(int i=getX();i==Xmax;setX(posX+50)){ 
   for(int i2=getY();i2==Ymax;setY(posY+50)){ 
    try{ 
     img = ImageIO.read(new File("rien.png")); 
    g.drawImage(img,posX,posY,this); 
    }catch(IOException e){ 
     e.printStackTrace(); 
    } 
    i2=posY; 
    i=posX; 
   } 
  } 
 } 
 public void setCase(int x, int y){ 
  setX(x); 
  setY(y); 
 } 
 public int getY(){ 
  return posY*50; 
 } 
 public void setY(int y){ 
  posY = y; 
 } 
 public int getX(){ 
  return posX*50; 
 } 
 public void setX(int x){ 
  posX = x; 
 } 
} 

//Sokoban.java 

import javax.swing.JFrame; 


public class Sokoban extends JFrame{ 
static int x = 500; 
static int y =400; 
 public Sokoban(){ 
  setTitle("Sokoban"); 
  setSize(x,y); 
  setLocationRelativeTo(null); 
  setDefaultCloseOperation(EXIT_ON_CLOSE); 
  setContentPane(new terrain()); 
  setVisible(true); 
 } 
 public static void main(String[] args) { 
  new Sokoban(); 

 } 

} 
Configuration: Mac OS X / Firefox 5.0.1

KX · Answer

Et... qu'est-ce que c'est censé faire ? 

Ta boucle dans PaintComponent est absolument horrible !!! 

for(int i=getX();i==Xmax;setX(posX+50))
Comme tu as fais setX(0) avant, on sait que getX()==0 d'où i==0 
La condition i==Xmax ne sera donc jamais vraie ! 
Du coup avant même le premier appel, on quitte la boucle. Il n'y a donc aucun appel à drawImage... Il te faudrait plutôt une condition du genre i<Xmax. 
Et pour en venir au setX(posX+50) tu peux directement faire posX+=50 
En effet tes fonctions setX et getX ne servent pas à grand chose d'autant qu'elles ne se correspondent pas (à cause de ton *50) ! 

Bref, voici ce que je te propose (en espérant avoir compris ce que tu voulais) : 

import java.awt.Graphics; 
import java.awt.Image; 
import java.io.File; 
import java.io.IOException; 
import javax.imageio.ImageIO; 
import javax.swing.JPanel; 

public class Terrain extends JPanel 
{ 
    private static final long serialVersionUID = 1L; 
     
    private int Xmax, Ymax; 
    private Image img; 
     
    public Terrain() throws IOException 
    { 
        img = ImageIO.read(new File("D:\rien.png")); 
        Xdim=img.getWidth(null); 
        Ydim=img.getHeight(null); 
    } 
     
    @Override 
    public void paintComponent(Graphics g) 
    { 
        int Xmax=getWidth(); 
        int Ymax=getHeight(); 
         
        for (int x=0; x<Xmax; x+=Xdim) 
            for (int y=0; y<Ymax; y+=Ydim) 
                g.drawImage(img,x,y,this); 
    } 
} 

import javax.swing.JFrame; 

public class Sokoban extends JFrame 
{ 
    private static final long serialVersionUID = 1L; 
     
    public static final int Xmax=500, Ymax=400; 
     
    public Sokoban() throws IOException 
    { 
        setTitle("Sokoban"); 
        setSize(Xmax,Ymax); 
        setLocationByPlatform(true); 
        setDefaultCloseOperation(EXIT_ON_CLOSE); 
        //setResizable(false); 
        setContentPane(new Terrain()); 
        setVisible(true); 
    } 
     
    public static void main(String...args) throws IOException 
    { 
        new Sokoban(); 
    } 
}
La confiance n'exclut pas le contrôle

Sokoban java

1 réponse

Discussions similaires