无法在JPanel上绘制矩形

Question

我目前正在尝试学习有关挥杆绘画的更多信息，并且正在创建一个小游戏，但似乎被卡住了。 我试图将几个矩形（敌人）添加到ArrayList中，然后在paintComponent()方法中绘制它们，但这似乎不起作用。

这是我的敌人班

import utils.RandomUtils;

import java.awt.*;
import java.awt.geom.Rectangle2D;

public class Enemy
{
    double x, y, w, h;
    Shape enemy;

    Toolkit tk = Toolkit.getDefaultToolkit();
    double width = tk.getScreenSize().width;
    double height = tk.getScreenSize().height;

    RandomUtils utils = new RandomUtils();

    public Enemy(int w, int h)
    {
        this.w = w;
        this.h = h;

        enemy = new Rectangle2D.Double(setX(), setY(), w, h);
    }

    private double setX()
    {
        return x = utils.randInt((int) w, (int) width);
    }

    private double setY()
    {
        return y = utils.randInt((int) h, (int) height);
    }

    public Shape getEnemy()
    {
        return enemy;
    }
}

这是我的董事会班

import players.Enemy;

import javax.swing.*;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.util.ArrayList;

public class Board extends JPanel
{
    int x, y, speed = 1, height = 25, width = 25;

    public Board()
    {
        addControls();
    }

    public void addControls()
    {
        Action upAction = new AbstractAction()
        {
            @Override
            public void actionPerformed(ActionEvent e)
            {
                y = y - speed;
                repaint();
            }
        };

        getInputMap(WHEN_FOCUSED).put(KeyStroke.getKeyStroke("W"), "Up Action");
        getActionMap().put("Up Action", upAction);

        Action downAction = new AbstractAction()
        {
            @Override
            public void actionPerformed(ActionEvent e)
            {
                y = y + speed;
                repaint();
            }
        };

        getInputMap(WHEN_FOCUSED).put(KeyStroke.getKeyStroke("S"), "Down Action");
        getActionMap().put("Down Action", downAction);

        Action leftAction = new AbstractAction()
        {
            @Override
            public void actionPerformed(ActionEvent e)
            {
                x = x - speed;
                repaint();
            }
        };

        getInputMap(WHEN_FOCUSED).put(KeyStroke.getKeyStroke("A"), "Left Action");
        getActionMap().put("Left Action", leftAction);

        Action rightAction = new AbstractAction()
        {
            @Override
            public void actionPerformed(ActionEvent e)
            {
                x = x + speed;
                repaint();
            }
        };

        getInputMap(WHEN_FOCUSED).put(KeyStroke.getKeyStroke("D"), "Right Action");
        getActionMap().put("Right Action", rightAction);
    }

    boolean enemiesDrawn = false;
    java.util.List<Enemy> enemies = new ArrayList<>();
    java.util.List<Shape> enemyShapes = new ArrayList<>();

    public void setEnemies()
    {
        for(int enemyCount = 0; enemyCount < 15; enemyCount++)
        {
            System.out.println(enemyCount);
            enemies.add(new Enemy(15, 15));
            enemyShapes.add(enemies.get(enemyCount).getEnemy());
        }
    }

    @Override
    public void paintComponent(Graphics g)
    {
        super.paintComponent(g);

        Graphics2D g2 = (Graphics2D) g;

        g.setColor(Color.RED);
        g.fillOval(x, y, width, height);

        if(!enemiesDrawn)
        {
            setEnemies();

            g2.setColor(Color.BLUE);
            g2.setBackground(Color.BLUE);

            enemyShapes.forEach(g2::draw);

            enemiesDrawn = true;
        }
    }
}

实用类

import java.util.Random;

public class RandomUtils
{
    public RandomUtils()
    {
    }

    public int randInt(int min, int max)
    {
        Random rand = new Random();
        return rand.nextInt((max - min) + 1) + min;
    }
}

和主班

import javax.swing.*;

public class MainFrame
{
    JFrame mainFrame;

    public MainFrame()
    {
        mainFrame = new JFrame("Game");

        addComponents();

        mainFrame.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
        mainFrame.setExtendedState(JFrame.MAXIMIZED_BOTH);
        mainFrame.setLocationRelativeTo(null);
        mainFrame.pack();
        mainFrame.setVisible(true);
    }

    public void addComponents()
    {
        mainFrame.add(new Board());
    }

    public static void main(String[] args)
    {
        SwingUtilities.invokeLater(MainFrame::new);
    }
}

绘制椭圆形（播放器）效果很好，我似乎无法在列表中绘制敌人。 我应该/可以做些什么来解决这个问题？

Answer 1

所以你的主要问题在这里...

if (!enemiesDrawn) {
    setEnemies();

    g2.setColor(Color.BLUE);
    g2.setBackground(Color.BLUE);

    System.out.println("...");
    enemyShapes.forEach(g2::draw);

    enemiesDrawn = true;
}

一旦paintComponent被调用一次，敌人将永远不会被再次绘制。 paintComponent可能被调用很多次（通常是快速连续调用），每次调用它时，都希望从头开始重新绘制组件的整个状态。

查看AWT和Swing中的 绘画以及执行自定义绘画 ，以了解有关在Swing中绘画如何工作的更多详细信息

观察 ...

这是没有意义的...

mainFrame.setExtendedState(JFrame.MAXIMIZED_BOTH);
mainFrame.setLocationRelativeTo(null);
mainFrame.pack();

您正在将框架设置为最大化，但随后尝试打包呢？ 这可能会在某些系统上引起问题，请提防。

RandomUtils应该创建一个Random实例，因此冒着使Random以相同顺序返回相同值的风险。 您可以单身而行

public enum RandomUtils {

    INSTANCE;

    private Random rand = new Random();

    public int randInt(int min, int max) {
        return rand.nextInt((max - min) + 1) + min;
    }
}

在Enemy使用Toolkit并不是一个好主意，除了敌人应该对外界几乎一无所知的事实之外，考虑到您的窗口带有装饰物， Toolkit并不是确定实际可见屏幕尺寸的最佳选择并且桌面上有诸如任务栏之类的东西，这冒着一些敌人会出现在屏幕上的风险

由于程序的工作方式，很难建议一些可以复制输出的内容。 但基本上，您应该将所需的位置/大小传递给Enemy类的构造函数。

您可以使用ComponentListener并监视componentResized方法，该方法首先使其稳定下来，然后可以根据组件的当前大小创建敌人。