使用 super() 构造函数时未初始化空白最终字段

Question

诚然，我对面向对象编程有点陌生，因此非常感谢您的帮助！ 我有一个类“Gerade”（德语为直线）和一个类“Strahl”（德语为射线）。我想在子类“Strahl”中调用超类“Gerade”的构造函数。但现在我收到错误消息：'空白的最终字段 p1 [和 p2] 可能尚未初始化'。这是什么原因？我非常感谢任何答案。提前谢谢！

public class Gerade {
    
private final Punkt p1;
private final Punkt p2;

public Gerade(Punkt p1, Punkt p2)
{
    if(p1.equals(p2)==true)
    {
    System.out.println("Die Eingabe ist ungueltig. Beide Punkte haben die selben Koordinaten.");
    p1 = null;
    p2 = null;
    }
    

    if(p1.getXkoord().equals(p2.getXkoord())==true)
    {
        if((p1.getYkoord().compareTo(p2.getYkoord()) < 0))
        {
            this.p1 = p1;
            this.p2 = p2;
        }
        else
        {
            this.p1 = p2;
            this.p2 = p1;
        }
    }    
    
    else if((p1.getXkoord().compareTo(p2.getXkoord()) > 0))
    {
        this.p2 = p1;
        this.p1 = p2;
    }
    
    else 
    {
        this.p1 = p1;
        this.p2 = p2;
    }   
}

“Gerade”类使用“Punkt”类的某些属性和方法。坐标是 BigDecimals。以免您感到困惑。

public class Strahl extends Gerade 
{
    

    private final Punkt p1;
    private final Punkt p2;
    private final Punkt start;


public Strahl(Punkt a, Punkt b)
    {
        super(a, b);
        this.start = a;

    }

Answer 1

在执行 Gerade 的构造函数后，p1 和 p2 都应该被初始化。但由于控制流程复杂，编译器无法保证会发生这种情况。仔细查看您的条件 - 如果条件都不满足，您必须提供回退

Answer 2

您的类Gerade 中已经有两个Punkt 实例变量。所以你不需要在课堂上重复它们Strahl。

其实这是问题之一：Strahl中的final变量p1和p2没有在任何地方初始化！

所以，只需删除它们：

public class Strahl extends Gerade {
    private final Punkt start;
    public Strahl(Punkt a, Punkt b) {
        super(a, b);
        this.start = a;
    }
}

另一个问题是@KonstantinPribluda 在他的回答中提到的。

Answer 3

既然您调用的是super(a, b)，那么您没有理由在Strahl 中重新声明它们。不用写p1.equals(p2) == true，你可以直接写p1.equals(p2)。

可以是这样的

class Gerade {

    private final Punkt p1 = null;
    private final Punkt p2 = null;

    public Gerade(Punkt p1, Punkt p2) {
        if (p1.equals(p2)) {
            System.out.println("Die Eingabe ist ungueltig. Beide Punkte haben die selben Koordinaten.");
            p1 = null;
            p2 = null;
        }

        // ...
    }
}

class Strahl extends Gerade {

    private final Punkt start;

    public Strahl(Punkt a, Punkt b) {
        super(a, b);
        this.start = a;
    }
}

我还建议你只使用构造函数进行初始化，任何业务逻辑都应该重构为一些方法。