扩展JPanel以绘制图像。但在添加到JFrame的同一面板上

Question

更新：正如Dmich所指出的，因为我在面板类之外绘制，这导致了对我的面板和动画类初始化的递归调用。所以我怎么还能在动画课上完成在我的面板上的绘图，但没有这个问题。

我的代码有一个非常具体的问题，我不知道如何描述正在发生的事情，但我会尽力。我尝试搜索stackOverFLow，但问题是我甚至不知道搜索什么。

我来了：

为了尽可能好地组织这篇文章，我将首先写下我正在处理的所有课程。

public class MyPanel extends JPanel implements Runnable{
private Animation anim = new Animation();
}
public class Animation extends ??? implements KeyListener{}

所以我有一个名为MyPanel的类，它扩展了JPanel。我已经使用add（new MyPanel（））将这个类添加到了我的JFrame中。我在MyPanel中使用了一个线程，该线程调用@Overrid public void paintComponent（Graphics g）paintComponent调用动画中的一个方法，在该方法中我将图像绘制到屏幕g2d。drawImage（image，int，int，ImageObserver）。问题是使用这种方法我需要一个ImageObserver，如果动画扩展了JPanel，我就可以得到它。但是如果我扩展了JPanel，就不会在JFrame上绘制任何内容，因为这是一个新的JPanel，没有添加到JFrame中。

但是如果我扩展我的面板（它被添加到我的JFrame），我会得到一大堆错误。

Eclipse错误：

Exception in thread "main" java.lang.StackOverflowError
at java.awt.Component.setBackground(Unknown Source)
at javax.swing.JComponent.setBackground(Unknown Source)
at javax.swing.LookAndFeel.installColors(Unknown Source)
at javax.swing.LookAndFeel.installColorsAndFont(Unknown Source)
at javax.swing.plaf.basic.BasicPanelUI.installDefaults(Unknown Source)
at javax.swing.plaf.basic.BasicPanelUI.installUI(Unknown Source)
at javax.swing.JComponent.setUI(Unknown Source)
at javax.swing.JPanel.setUI(Unknown Source)
at javax.swing.JPanel.updateUI(Unknown Source)
at javax.swing.JPanel.<init>(Unknown Source)
at javax.swing.JPanel.<init>(Unknown Source)
at javax.swing.JPanel.<init>(Unknown Source) 

在这个错误消息下，这两行将继续运行，直到eclipse终止。

at MyPanel.<init>(MyPanel.java:9)
at Animation.<init>(Animation.java:9)

任何帮助都将不胜感激。

更新2根据建议添加代码示例：

import javax.swing.JFrame;

public class Simulation extends JFrame {
private MyPanel panel = new MyPanel();
public Simulation() {
    initUI();
}

private void initUI() {

    add(panel);
    setResizable(false);
    pack();

    setTitle("Simulation");
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setLocationRelativeTo(null); // centers
}
public static void main(String[] args) {

    Simulation ex = new Simulation();
    ex.setVisible(true);
}
}


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

public class MyPanel extends JPanel implements Runnable { 
private Animation anim;
public MyPanel() {
    anim = new Animation();
    initPanel();
}

private void initPanel() {
    //other customizables
}

@Override
public void run() {
//Thread that calls repaint();
}

@Override
public void paintComponent(Graphics g) {
    super.paintComponent(g);
    Graphics2D g2D = (Graphics2D) g;
    anim.step(g2D, this);
}
}


import java.awt.Graphics2D;

public class Animation {
private Sprite player;
private KeyBinder kB;

public Animation() {
    player = new Sprite();
  kB = new KeyBinder(player);
}

public void step(Graphics2D g2d, JPanel p) {
    player.move();
    drawSprite(g2d, p);
}

private void drawSprite(Graphics2D g2d, JPanel p) {
    g2d.drawImage(player.getImage(), player.getX(), player.getY(), p);
}
}

import java.awt.Image;
import java.awt.event.KeyEvent;
import javax.swing.ImageIcon;

public class Sprite {
private final String PLAYER_IMAGE_FILE = "Image location";
private Image image;
private int width, height;
private int x, y, dx, dy;

public Sprite() {
    x = 0;
    y = 0;
    loadImage();
}

private void loadImage() {
    ImageIcon ii = new ImageIcon(PLAYER_IMAGE_FILE);
    image = ii.getImage();

    width = image.getWidth(null);
    height = image.getHeight(null);
}

public void move() {
    x += dx;
    y += dy;
}

public void keyPressed(KeyEvent e) {

    //assings meaning to keypressed
}

public void keyReleased(KeyEvent e) {

    //assings meaning to keyReleased
}

public int getX() {
    return x;
}

public int getY() {
    return y;
}

public int getW() {
    return width;
}

public int getH() {
    return height;
}

public Image getImage() {
    return image;
}
}

import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

public class KeyBinder extends MyPanel implements KeyListener {
Sprite p;
public KeyBinder(Sprite player) {
    p = player;
    addKeyListener(this);
    setFocusable(true);
    setFocusTraversalKeysEnabled(false);
}

@Override
public void keyPressed(KeyEvent e) {
    p.keyPressed(e);
}
@Override
public void keyReleased(KeyEvent e) {
    p.keyReleased(e);
}
@Override
public void keyTyped(KeyEvent e) {

}
}

通过尝试解决代码问题，我对代码进行了修改，主要是通过尝试将JPanel传递给nimation.step方法，并为密钥监听创建一个单独的类。

新的错误列表基本相同：

Exception in thread "main" java.lang.StackOverflowError
at java.awt.Component.setBackground(Unknown Source)
at javax.swing.JComponent.setBackground(Unknown Source)
at javax.swing.LookAndFeel.installColors(Unknown Source)
at javax.swing.LookAndFeel.installColorsAndFont(Unknown Source)
at javax.swing.plaf.basic.BasicPanelUI.installDefaults(Unknown Source)
at javax.swing.plaf.basic.BasicPanelUI.installUI(Unknown Source)
at javax.swing.JComponent.setUI(Unknown Source)
at javax.swing.JPanel.html" target="_blank">setUI(Unknown Source)
at javax.swing.JPanel.updateUI(Unknown Source)
at javax.swing.JPanel.<init>(Unknown Source)
at javax.swing.JPanel.<init>(Unknown Source)
at javax.swing.JPanel.<init>(Unknown Source)

然后无限循环，直到以下三行终止：

at MyPanel.<init>(MyPanel.java:10)
at KeyBinder.<init>(KeyBinder.java:6)
at Animation.<init>(Animation.java:11)

Answer 1

这是另一个继承不良的例子。我修复了它通过实现KeyListener在MYPanel类，并传递作为参数的动画的我的面板。对我来说，尝试从不同的类实现一个关键监听器到一个JPanel是不好的做法。

Answer 2

我不建议让类扩展JPanel，而是在实例化时引入JPanel参数：

public class Panel implements Runnable{
    private JPanel panel;

    public Panel(JPanel panel){
        this.panel = panel;
    }
}

此外，我认为明智的做法是将您拥有的Panel类扩展为组件，以便在将其添加到标题中提到的现有面板时实现无缝集成：

public class Panel extends Component implements Runnable{
    private JPanel panel;

    public Panel(JPanel panel){
        this.panel = panel;
    }
}

请让我知道，如果这不起作用，或者你有其他问题！

Answer 3

所以我有一个名为Panel的类，它扩展了JPanel。我已经使用add（new Panel（））将这个类添加到我的JFrame中。我正在使用面板中的一个线程，该线程调用@Overrid public void paintComponent（Graphics g）

首先，Swing不是线程安全的，您不应该从事件调度线程的上下文之外更新UI。

其次，在任何情况下，都不应该直接调用paintComponent，也不需要将其设置为public，因为不应该调用它。

有关更多详细信息，请参阅Swing中的并发

paintComponent调用动画中的一个方法，在该方法中，我将图像绘制到屏幕g2d。drawImage（image，int，int，ImageObserver）。问题是使用这种方法我需要一个ImageObserver，如果动画扩展了JPanel，我就可以得到它。

为什么不让动画“可绘制”，提供一种方法，要求调用者传递完成工作所需的信息？

public class Animation {
    public void paint(Graphics2D g2d, ImageObserver observer) {
        //...
    }
}

您还可以将可能需要的其他信息传递给paint方法，例如建模当前状态的模型。

这基本上是一个委托模型——您将特定任务的责任“委托”给另一个类

那么KeyListener呢？

我听到你问。

嗯，首先，处理它可能不是动画的责任。相反，控制器应该修改状态，以响应事件，最终由渲染器绘制

另外，您应该改用密钥绑定API，它将解决与KeyListener