JFrame getHeight() 和 getWidth() 返回 0

Question

我正在制作一个简单的乒乓球游戏；部分碰撞机制需要获取画布的宽度和高度来重定向球。但是，getWidth() 和 getHeight() 出于某种原因返回 0。这是主要的代码块。

package pong;

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

public class Main extends JPanel {

    static int gameSpeed = 10;
    Ball ball = new Ball(this);

    private void move() {
        ball.move();
    }

    public void paint(Graphics g) {
        super.paint(g);
        Graphics2D g2d = (Graphics2D) g;
        ball.paint(g2d);
    }

    public static void main(String args[]) throws InterruptedException {
        JFrame frame = new JFrame("Pong");
        Main game = new Main();
        frame.add(game);
        frame.setSize(400, 400);
        frame.setVisible(true);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        while (true) {
            game.move();
            game.repaint();
            Thread.sleep(gameSpeed);
        }
    }
}

这是处理移动条件的实际 Ball 类。

package pong;

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

public class Ball extends JPanel {

    int x = 1;
    int y = 1;
    int dx = 1;
    int dy = 1;
    private Main game;

    public Ball(Main game) {
        this.game = game;
    }

    void move() {
        System.out.println(getWidth() + getHeight());

        if (x + dx < 0) {
            dx = 1;
        }
        if (y + dy < 0) {
            dy = 1;
        }
        if (x + dx > (getWidth() - 30)) {
            dx = -1;
        }
        if (y + dy > (getHeight() - 30)) {
            dy = -1;
        }
        x = x + dx;
        y = y + dy;
    }

    public void paint(Graphics2D g) {
        g.fillOval(x, y, 30, 30);
    }
} 

编辑：问题解决了，我只是没有告诉 getWidth() 和 getHeight() 指的是什么。显然，如果我不告诉他们要获得什​​么，他们将返回 null。德普。简单的解决方法是将它们更改为 game.getWidth() 和 game.getHeight()。不过感谢您的帮助！您的所有输入也有助于其他领域。 :)

Answer 1

1. Graphics/Java2D 默认从不返回合理的Dimension，结果为零Dimension，您必须为JPanel 覆盖getPreferredSize，然后getWidth/Height 将返回正确的坐标JPanel 的大小。

2. 然后使用JFrame.pack() 代替任何大小。

3. 覆盖paintComponent 为Swing JComponents，而不是paint()，在paintComponent 1st 中。代码行应为super.paintComponent，否则绘制累积。

4. 永远不要在Swing 中使用Thread.sleep(int)，也不要在Java7 和Swing 中使用Swing Timer，而是使用Thread.sleep(int) 停止的无限循环。

Answer 2

这是一个复杂的问题，由于某些设计决策而变得更糟。

通常，Swing 希望组件在布局管理器的约束中使用。布局管理器需要一定数量的信息才能决定如何最好地布局这些组件。这包括getPreferred/Minimum/Maximum 大小。

您遇到的问题是您实际上不想使用布局管理器，因为您想手动定位Ball。当您抛弃布局管理器时，您将负责很多工作，确保组件在父容器中的大小和位置正确。

更简单的解决方案是生成一个“游戏”表面，您可以在其上绘制游戏元素或实体。

问题是，您似乎不知道要使用哪种方法。您的 Ball 扩展自 JPanel，但您的 Main 面板正在绘制自身...这不是组件应该呈现的方式。

您可以简单地将Main 用作主要游戏界面并将所有实体直接呈现给它，而不是使用组件作为游戏实体的基础，这会带来许多问题和管理开销。这使您可以更好地控制并简化流程 - 恕我直言

例如...

import java.awt.Container;
import java.awt.Dimension;
import java.awt.EventQueue;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.Rectangle;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.Timer;
import javax.swing.UIManager;
import javax.swing.UnsupportedLookAndFeelException;

public class Main extends JPanel {

    public static final int GAME_SPEED = 40;
    Ball ball = new Ball();

    protected void move() {
        ball.move(this);
    }

    public Main() {
        setLayout(null);
        Timer timer = new Timer(GAME_SPEED, new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                move();
                repaint();
            }
        });
        timer.start();
    }

    @Override
    protected void paintComponent(Graphics g) {
        super.paintComponent(g); 
        ball.paint((Graphics2D)g);
    }

    @Override
    public Dimension getPreferredSize() {
        return new Dimension(400, 400);
    }

    public static void main(String args[]) throws InterruptedException {
        EventQueue.invokeLater(new Runnable() {
            @Override
            public void run() {
                try {
                    UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
                } catch (ClassNotFoundException | InstantiationException | IllegalAccessException | UnsupportedLookAndFeelException ex) {
                }

                JFrame frame = new JFrame("Pong");
                frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                Main game = new Main();
                frame.add(game);
                frame.pack();
                frame.setLocationRelativeTo(null);
                frame.setVisible(true);
                frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
            }
        });
    }

    public interface Entity {

        public void paint(Graphics2D g2d);
        public Rectangle getBounds();

    }

    public interface MovableEntity extends Entity {

        public void move(Container parent);

    }

    public class Ball implements MovableEntity {

        private int dx = 1;
        private int dy = 1;

        private int x;
        private int y;

        @Override
        public void move(Container parent) {

            int width = parent.getWidth();
            int height = parent.getHeight();

            int x = getX();
            int y = getY();

            x += dx;
            y += dy;

            if (x + dx < 0) {
                dx = 1;
            }
            if (y + dy < 0) {
                dy = 1;
            }
            if (x + dx > (width - getBounds().width)) {
                dx = -1;
            }
            if (y + dy > (height - getBounds().height)) {
                dy = -1;
            }

            setLocation(x, y);

        }

        @Override
        public void paint(Graphics2D g2d) {
            int width = getBounds().width;
            int height = getBounds().height;
            int dim = Math.min(width, height);

            int xPos = x + ((width - dim) / 2);
            int yPos = y + ((height - dim) / 2);

            g2d.fillOval(xPos, yPos, dim, dim);
        }

        @Override
        public Rectangle getBounds() {
            return new Rectangle(x, y, 30, 30);
        }
    }
}