JFrame移动矩形为什么我的矩形没有移动并清除路径

Question

我对java中的JFrames和图形非常陌生。我的长期目标是创建一个RayCast游戏世界。下面的代码是我第一次在jframe中以其坐标移动矩形。当用户按下箭头键时，坐标会发生变化。然而，似乎有些问题，因为当我使用该程序时，矩形只会被绘制并创建一条路径。我想在坐标的位置画一个矩形。

import javax.swing.JFrame;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.KeyListener;
import java.awt.event.KeyEvent;
import java.awt.event.ActionListener;
import javax.swing.Timer;
import java.awt.event.ActionEvent;
import javax.swing.JComponent;
import javax.swing.JPanel;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JMenu;



public class Raycast_World_2 extends JFrame implements ActionListener, KeyListener, Runnable
{
    public int px,py;
    int velx = 0, vely = 0;
    Graphics f;
   
    public Raycast_World_2()
    {
        // Instanzvariable initialisieren
        px = 100;
        py = 100;
        setSize(1280,960);
        setVisible(true);
        setDefaultCloseOperation(3);
        addKeyListener(this);
         setFocusable(true);
        setFocusTraversalKeysEnabled(false);
        setTitle("Raycast_World-Try4_Version 3.0");
        JMenuBar menuBar = new JMenuBar();
JMenu menuFile = new JMenu("File");
JMenuItem menuItemExit = new JMenuItem("Exit");
menuFile.add(menuItemExit);
 
menuBar.add(menuFile);
 
// adds menu bar to the frame
setJMenuBar(menuBar);
    }
    public void actionPerformed(ActionEvent e){
        update(getGraphics());
        repaint();
    
    
}
    public void paint(java.awt.Graphics g) {
    
  g.setColor(Color.red);
  
  g.fillRect(px,py,20,20);
  g.dispose();  
  repaint();
}
public void run(){
   
    
}

    public void keyPressed(KeyEvent e){
    
    int c = e.getKeyCode();
    if(c == KeyEvent.VK_LEFT){
        
        px = px-10;
    }
    if(c == KeyEvent.VK_UP){
        
        py = py -10;
    }
    if(c == KeyEvent.VK_RIGHT){
        
        px = px +10;
    }
    if(c == KeyEvent.VK_DOWN){
        
        py = py +10;
    }
}
public void keyTyped(KeyEvent e){
}
public void keyReleased(KeyEvent e){}
}

请解释我做错了什么。提前谢谢你帮助我！

Answer 1

不幸的是，你画得不对。

• 不要扩展JFrame。顶级类很少是子类。您所需要的只是一个JFrame的实例，您将在其中添加一个用于绘制的JPanel。框架用于固定组件

下面是执行上述操作的代码的修改版本。

import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

import javax.swing.JFrame;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JPanel;
import javax.swing.SwingUtilities;

public class Raycast_World_2 extends JPanel implements KeyListener{
    public int px, py;
    int velx = 0, vely = 0;
    JFrame frame = new JFrame();
    public Raycast_World_2() {
        // Instanzvariable initialisieren
        px = 100;
        py = 100;
        // okay here but in some situations getPreferredSize() should be
        // overridden.
        setPreferredSize(new Dimension(600,500));
        setVisible(true);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        addKeyListener(this);
        setFocusable(true);
        setFocusTraversalKeysEnabled(false);
        frame.setTitle("Raycast_World-Try4_Version 3.0");
        frame.add(this);
        JMenuBar menuBar = new JMenuBar();
        JMenu menuFile = new JMenu("File");
        JMenuItem menuItemExit = new JMenuItem("Exit");
        menuFile.add(menuItemExit);
        menuBar.add(menuFile);
        frame.setJMenuBar(menuBar);
        frame.pack(); // size and layout components.
        frame.setLocationRelativeTo(null); // center on screen
        frame.setVisible(true);
    }
    
    public static void main(String[] args) {
        SwingUtilities.invokeLater(()->new Raycast_World_2());
    }

    public void paintComponent(Graphics g) {
        super.paintComponent(g);
        Graphics gg = g.create();
        gg.setColor(Color.red);
        
        gg.fillRect(px, py, 20, 20);
        gg.dispose();
    }
    }
    
    
    
    public void keyPressed(KeyEvent e) {
        
        int c = e.getKeyCode();
        if (c == KeyEvent.VK_LEFT) {
            
            px = px - 10;
        }
        if (c == KeyEvent.VK_UP) {
            
            py = py - 10;
        }
        if (c == KeyEvent.VK_RIGHT) {
            
            px = px + 10;
        }
        if (c == KeyEvent.VK_DOWN) {
            
            py = py + 10;
        }
        repaint();
    }
    
    public void keyTyped(KeyEvent e) {
    }
    
    public void keyReleased(KeyEvent e) {
    }
}

查看Java教程中的绘画方法。

还有许多在StackOverflow上正确绘制的优秀示例。

这是另一个建议，但不是必需的。使用内部类来创建您的keyListener。通过子类化KeyAdapter，您不必自己添加未实现的方法。KeyAdapter只是一个具有空方法的类。MouseAdapter是另一个。请注意，内部类可以访问包含类的字段。因此您可以执行以下操作：

public class MyKeyListener extends KeyAdapter {
     public void keyPressed(KeyEvent e) {
    
    int c = e.getKeyCode();
    if (c == KeyEvent.VK_LEFT) {    
        px = px - 10;
    } else
    if (c == KeyEvent.VK_UP) {
        py = py - 10;
    } else
    if (c == KeyEvent.VK_RIGHT) {
        px = px + 10;
    } else
    if (c == KeyEvent.VK_DOWN) {
        py = py + 10;
    }
    repaint();
}

然后您可以执行以下操作。

addKeyListener(new MyKeyListener());

Answer 2

您的问题似乎是每次调用“油漆”方法时都没有重置屏幕。所以发生的情况是，每次您运行此方法时，它都会采用屏幕的先前状态并在该现有状态上绘制您想要的任何内容。更清楚地说，它只是使屏幕上的油漆过载。为了修复它，您所要做的就是在油漆方法的开头添加以下代码行：

super.paint(g);

这个从超类JFrame调用的方法将确保之前绘制到屏幕上的所有内容都将被擦除。因此，这一行代码之后的屏幕将是空的，并且现在将只使用这一行代码之后在屏幕上绘制的内容进行绘制。