Javaの重力バグ

Question

私の重力クラスに問題があります。ユーザがジャンプするたびにベースに達するまでy座標を変更する必要がありますが、ジャンプボタン（W）を繰り返し押すとバグが発生し、キューブはベース。

メインクラス：

import javax.swing.*; 
import java.awt.*; 
import java.awt.event.*; 

@SuppressWarnings("serial") 
public class Frame extends JFrame implements Runnable{ //main class 

    static Panel panel; 
    static Frame frame; 

    //CONSTRUCTOR 
    Frame(){ //constructor 

     super("Game Frame"); 
     setLayout(new GridLayout(1,1)); 

     panel = new Panel(); 

     add(panel); 

    } //constructor 

    public static void main (String args[]){ //main method 

     frame = new Frame(); 
     frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
     frame.setVisible(true); 
     frame.setResizable(true); 
     frame.setSize(500,300); 
     frame.setLocationRelativeTo(null); 
     Thread T1 = new Thread(frame); 
     T1.start(); 

    } //main method 


    public void run(){ //run method 

     frame.addKeyListener(new KeyListener(){ //key listener method 

      @Override 
      public void keyPressed(KeyEvent arg0) { //key pressed 

       switch(arg0.getKeyChar()){ 

       //A (LEFT) 
       case 'a': 
        panel.x -= 10; 
        panel.repaint();break; 

       //D (RIGHT) 
       case 'd': 
        panel.x += 10; 
        panel.repaint();break; 

       //W (UP) 
       case 'w': 
        panel.y -= 40; 
        panel.repaint(); 
        Thread T1 = new Thread(new Gravity()); 
        T1.start(); 

       } //switch 

      } //key pressed method 

      @Override 
      public void keyReleased(KeyEvent arg0) {} 

      @Override 
      public void keyTyped(KeyEvent arg0) {} 

     }); //key listener method 
    } //run method 
} //main class 

Panelクラス：

import javax.swing.*; 
import java.awt.*; 
import java.awt.image.BufferedImage; 
import java.io.IOException; 

import javax.imageio.ImageIO; 

@SuppressWarnings("serial") 
public class Panel extends JPanel{ //class 

    static BufferedImage bg; 
    static int x = 20; 
    static int y = 195; 


    //PAINT METHOD 
    public void paint(Graphics g){ //method 

     super.paint(g); 
     g.setColor(Color.ORANGE); 
     this.setBackground(Color.BLACK); 

     //IMPORTING IMAGES 
     try { 
      bg = ImageIO.read(getClass().getResourceAsStream("Background.png")); 
     } catch (IOException e) { 
      // TODO Auto-generated catch block 
      e.printStackTrace(); 
     } 

     //DRAWING GAME 
     g.drawImage(bg, 0, 0,500,275,null); 
     g.fillRect(x, y, 50, 50); 

    } //method 




} //class 

重力クラス：

public class Gravity implements Runnable{ //class 

    @Override 
    public void run() { //run method 

      while(Frame.panel.y != 195){ //while loop 

       try { 
        Thread.sleep(100); 
       } catch (InterruptedException e) { //catch exception 
        // TODO Auto-generated catch block 
        e.printStackTrace(); 
       } //catch exception 

       Frame.panel.y += 5; 
       Frame.panel.repaint(); 

      } //while loop 

    } //run method 

} //class 

Answer 1

私はあなたが1を再利用ではなく、複数のスレッドを作成しないでください示唆しています。 2つのtheadsがFrame.panel.yを変更することがあります。最初は195に移動し、2つ目は寝ており、2つ目が200に移動し、！= 195の条件が満たされます。クラッシュは後でフレームの外側にペイントすることによって発生します。簡単な回避策は、 "while（Frame.panel.y！= 195）"を "while（Frame.panel.y < 195）"に置き換えることです。