1

这里我有一个简单的类，它扩展了 ACM 图形库中的一个名为 GRect 的类。Grect 是一个可以绘制到 GCanvas（也是 ACM 的一部分）的基本矩形。我想要实现的是一个新对象，它是一个矩形，但有一个随矩形移动的附加标签。

在我的代码中，我创建了一个名为labeledRect 的类，它扩展了GRect，并包含一个GLabel 类型的实例变量“rectLabel”，它在labeledRects 构造函数中初始化。我想覆盖 GRect 的一些方法，以便在移动 labeldRect 时，rectLabel 随之移动。

我的问题是，尽管“rectLabel”被声明为实例变量，并在构造函数中进行了初始化，但它在被覆盖的方法“setLocation”中变为 NULL。我也尝试在声明时初始化“rectLabel”，但出现了同样的问题。

import acm.graphics.*;

public class labeledRect extends GRect {

    //GLabel declared as an instance variable
    public GLabel rectLabel;

    public labeledRect(double x, double y, double width, double height, String theLabel) {
        //Call GRect constructor
        super(x, y, width, height);

        //Label initialized. 
        //Location setting and adding to the canvas works fine here. 
        rectLabel = new GLabel(theLabel);
        rectLabel.setLocation(
                    x + (width / 2) - (rectLabel.getWidth() / 2),
                    y + (height / 2) + (rectLabel.getAscent() / 2.2));
    }

    public void setLocation(double x, double y)
    {
        //Setting GRect's location. Works correctly./
        super.setLocation(x, y);

        //Attempt to set the label's location 
        //and get a NullPointer exception for rectLabel
        rectLabel.setLocation(
                super.getX() - (rectLabel.getWidth() / 2),
                super.getY() - (rectLabel.getHeight() / 2));
    }
}
4

1 回答 1

2

鉴于它是一个公共字段，很难准确地推断出可能将变量设置为 null 的原因。这是你应该改变的第一件事。字段应该几乎总是私有的。

接下来，你说setLocation的是一个被覆盖的方法——它是否在GRect构造函数中被调用？如果是这样，将在构造函数中设置值之前labeledRect调用它，这很可能是您的问题的原因。

在 Java 中，超类构造函数在子类中的任何代码之前执行 - 在实例变量初始化程序和构造函数主体之前。所有变量都有其默认值。这就是为什么从构造函数调用虚方法是个坏主意。

（顺便说一句，该名称labeledRect不遵循 Java 命名约定。）

于 2012-08-19T20:50:05.833 回答