Discussion :

[ia.jenny]

Bonjour,
je ne comprend pas pourquoi, il n'y a pas de réaction sur "dx" quand j’appuie sur la touche gauche
j'ai longtemps chercher sur internet et pas de solution.
je ne veux pas de timer qui réinitialise tout les ms.
dois je faire une autre class. comment faire pour que cela fonctionne.
merci encore.

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
70
71
72
73
74
package imageTest;
 
import javax.swing.JFrame;
import javax.swing.JPanel;
import java.io.File;
import java.io.IOException;
import javax.imageio.ImageIO;
import java.awt.Graphics;
import java.awt.image.BufferedImage;
 
import java.awt.event.KeyListener; // <------------------------
import java.awt.event.KeyEvent; // <------------------------
import java.awt.event.*; // <------------------------
 
 
	public class Main extends JPanel
	{
		 private int dx; // <------------------------
 
	    BufferedImage[] bufferedImage  = new BufferedImage[100];
	    public static void main(String[] args)
	    {  
	        JFrame frame=new JFrame();
	        frame.setTitle("Ma premier fenetre");
	        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	        frame.setSize(800,600);
 
	        frame.getContentPane().add(new Main());
	        frame.setVisible(true);
	     } 
 
 
// <------------------------ 
	    public void keyReleased(KeyEvent e) 
	    {
	    	int key = e.getKeyCode();
 
                   if (key == KeyEvent.VK_LEFT)
                  {  
            	      dx--;
            	       repaint();
                   }  
	    }
// <------------------------
 
 
		 public void paintComponent(Graphics g)
		 { 
			 super.paintComponent(g); // <------------------------
			 for(int i=1; i<2; i++) // 1 image pour le moment
			 {
				try
				{
					this.bufferedImage[i] =ImageIO.read(new File(i + ".jpg"));
				}
				catch(IOException e)
				{
					e.printStackTrace();
				}
			 }
 
 
			 	for(int i=0; i<6; i++)
			 	{
				 	for(int j=0; j<4; j++)
				 	{
				 		g.drawImage( this.bufferedImage[1] ,i*100+dx*100,j*100,100,100,this);
				 	}
			 	} 
 
		 }  
 
 
	}

[bouye]

Tu as vérifié que ça fonctionnait avec les autres touches ? Ton composant est focusable et a le focus ?

[ia.jenny]

Bonjour,
électivement, c'est un problème de focus. peut être un problème avec "public void keyReleased(KeyEvent e)"
la touche est peut être reconnu sauf que l'image n'est pas actualiser

j'ai modifier le programme, mais ça ne fonctionne toujours pas.

Code :

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

1
2
3
4
5
6
7
8
	        Main ep = new Main();
	        JFrame frame=new JFrame();
	        frame.setTitle("Ma premier fenetre");
	        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	        frame.setContentPane(ep);
	        frame.setVisible(true);
	        frame.setSize(800,600);
	        ep.requestFocus(true);

[joel.drigo]

Salut,

La méthode keyReleased ne sera pas appelée si tu n'enregistres pas d'écouteur sur le composant.

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
public class Main extends JPanel {
 
   private int dx; // <------------------------
 
   BufferedImage[] bufferedImage  = new BufferedImage[100];
 
 
    public Main() {
          addKeyListener(new KeyAdapter() { // enregistrement de l'écouteur
                  public void keyReleased(KeyEvent e) {
	    	      int key = e.getKeyCode();
 
                   if (key == KeyEvent.VK_LEFT) {  
            	      dx--;
            	       repaint();
                   }  
 
	        }
 
          });
    }
 
    public void paintComponent(Graphics g)
		 { 
			 super.paintComponent(g); // <------------------------
			 for(int i=1; i<2; i++) // 1 image pour le moment
			 {
				try
				{
					this.bufferedImage[i] =ImageIO.read(new File(i + ".jpg"));
				}
				catch(IOException e)
				{
					e.printStackTrace();
				}
			 }
 
 
			 	for(int i=0; i<6; i++)
			 	{
				 	for(int j=0; j<4; j++)
				 	{
				 		g.drawImage( this.bufferedImage[1] ,i*100+dx*100,j*100,100,100,this);
				 	}
			 	} 
 
		 }  
 
 
 
}

Par ailleurs, peut-être que ce serait plus simple de gérer ça avec un KeyboardFocusManager, ce qui te permettra de gérer plus globalement les problématiques de touches et de focus.

Par ailleurs, évite de charger des fichiers en mémoire dans l'Event Dispatch Thread (donc dans les méthodes de dessin de Swing). Il faut charger au minimum ce thread si on veut une animation fluide, et les I/O même sur du SSD et même s'il y a du cache, tout ça, c'est lent relativement. Et bonjour l'impact sur le GC. Surtout quand tu vas en charger 100 à chaque repaint ! Et ça ne sert à rien de charger sans arrêt les mêmes fichiers.

[ia.jenny]

merci pour ton aide

Par ailleurs, évite de charger des fichiers en mémoire dans l'Event Dispatch Thread (donc dans les méthodes de dessin de Swing). Il faut charger au minimum ce thread si on veut une animation fluide...

Je sais tout ça, je ne sais pas le faire pour le moment.
il y a beaucoup de chose à revoir dans mon code.

au sujet des touches, dx incrémente bien merci, mais l'image devrais bouger. Comment Faire et sans timer ?
(plus tard, ça sera seulement les textures bufferedImage[1] qui changeront et non tout les images i*100+dx*100,j*100)

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
package imageTest;
 
import javax.swing.JFrame;
import javax.swing.JPanel;
import java.io.File;
import java.io.IOException;
import javax.imageio.ImageIO;
import java.awt.Graphics;
import java.awt.image.BufferedImage;
import java.awt.event.*; 
 
	public class Main extends JPanel
	{
	    static int dx;
	    BufferedImage[] bufferedImage  = new BufferedImage[100];
 
	     public static void main(String[] args)
	    {  
 
	    	 JFrame frame=new JFrame();
	        frame.setTitle("Ma premier fenetre");
	        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	        frame.setSize(800,600);
	        frame.getContentPane().add(new Main());
	        frame.setVisible(true);
	        frame.setFocusable( true );
 
	        frame.addKeyListener(new KeyAdapter() {
	            public void keyPressed(KeyEvent e) {
	               if (e.getKeyCode() == KeyEvent.VK_UP) {
	            	   dx--;
	            	   System.out.println(dx);
	               }
	            }
	         });   
 
	     } 
 
 
 
 
		 public void paintComponent(Graphics g)
		 { 
			 super.paintComponent(g);
			 for(int i=1; i<2; i++) // 1 image pour le moment
			 {
				try
				{
					this.bufferedImage[i] =ImageIO.read(new File(i + ".jpg"));
				}
				catch(IOException e)
				{
					e.printStackTrace();
				}
			 }
 
 
			 	for(int i=0; i<6; i++)
			 	{
				 	for(int j=0; j<4; j++)
				 	{
				 		g.drawImage( this.bufferedImage[1] ,i*100+dx*100,j*100,100,100,this);
				 	}
			 	} 
 
		 }  
 
 
	}

[joel.drigo]

au sujet des touches, dx incrémente bien merci, mais l'image devrais bouger. Comment Faire et sans timer ?
(plus tard, ça sera seulement les textures bufferedImage[1] qui changeront et non tout [B]les images i

l'image ne bouge juste parce qu'il n'y a pas de repaint() après le dx--;. De manière général, toute modification d'état servant dans l'affichage peut soit nécessiter un repaint, soit nécessiter un relayout et un repaint. Swing gère tout seul tout ce qui est modification dans ses composants, ou presque (disons qu'il y'en a certaines où pour éviter de faire trop souvent des repaints ou des relayouts, il ne le fait pas tout seul et il faut le faire en appelant les méthode repaint(), et/ou revalidate().
Et c'est encore plus vrai lorsque ce sont juste des variables à toi que tu modifies et que Swing ne peut pas connaître. Il faut donc appeler explicitement repaint pour forcer Swing à remettre à jour l'affichage en fonction des nouvelles valeurs de tes variables, si tu ne veux pas évidemment attendre une action automatique de Swing qui le ferait (par exemple celle qui pourrait se faire en changeant la taille de la fenêtre à la souris).

[ia.jenny]

beaucoup, ça fonctionne

+1