JFrame getHeight()和getWidth()返回0

时间:2014-02-05 20:42:14

标签: java swing jpanel paint thread-sleep

我正在做一个简单的乒乓球比赛;并且部分碰撞力学需要获得画布的宽度和高度以重定向球。但是,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。 DERP。简单的解决方法是将它们更改为game.getWidth()和game.getHeight()。谢谢你的帮助!您的所有输入也有助于其他领域。 :)

2 个答案:

答案 0 :(得分:7)

    默认情况下,
  1. Graphics / Java2D永远不会重复Dimension,结果为零Dimension,您必须覆盖getPreferredSize JPanel然后getWidth/Height将返回JPanels大小的正确坐标。

  2. 然后使用JFrame.pack()而不是任何尺寸。

  3. paintComponent 1st内覆盖Swing JComponents而不是paint()的{​​{1}}。代码行应为paintComponent,否则绘制累积。

  4. 永远不要在super.paintComponent中使用Thread.sleep(int),也不要在Java7和Swing中使用自定义绘画或动画,而是使用Swing而不是Swing Timer停止的无限循环。< / p>

答案 1 :(得分:3)

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

通常,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);
        }
    }
}