在父构造函数中初始化子之前的Java抽象父调用子方法

Question

我有以下课程：

abstract class Parent
{
    private ClassDependentOnSize classDependentOnSize;

    public Parent ()
    {
        this.classDependentOnSize = new ClassDependentOnSize(size());
    }

    public abstract int size ();
}

class Child extends Parent
{
    private String DBSelection;
    private String[] DBSelectionArgs;

    public Child (String selection, String... selectionArgs)
    {
        super();
        this.DBSelection = selection;
        this.DBSelectionArgs = selectionArgs;
    }

    @Override
    public int size()
    {
        //FAILS because Parent calls size before DBSelectionArgs is initialized
        String temp = "";
        for(String str : DBSelectionArgs)
           temp += str;
        return temp.length;

        //This function basically does a calculation that should not be
        //done before hand. I have posted the exact method below.
    }
}

class ClassDependentOnSize 
{
    public ClassDependentOnSize(int size)
    {

    }
}

虽然这不是我的 exact 类，但这是同样的问题。我写了这个来删除所有不必要的代码。

如你所见，超类试图在子类完成初始化之前调用size()来构造依赖于size的类。我很好奇过去有人如何解决这个问题。我相信每个人都知道， super() 必须是子构造函数的第一行。正如我已经列出了这些课程，这是一个糟糕的设计吗？感觉这个问题应该是以前发生过的，但是找不到解决办法。

编辑： 这是确切的方法。它用于 Android 应用程序。

protected Cursor getCursor()
{
    if (songCursor == null)
    {
        songCursor = GET_SONGS_CURSOR(getContext(), MediaStore.Audio.Media.EXTERNAL_CONTENT_URI.toString(), selection, selectionArgs, selection);

        if (!songCursor.moveToFirst())
            Standard.Loge("|NewPlaylist.getCursor| Could not move to first.");
    }

    int count = songCursor.getCount();
    songCursor.close();
    return count;
}

Answer 1

我以前肯定遇到过这个问题。经验教训是，您永远不应该在 Parent 构造函数中调用非私有方法，这样就不会发生这种情况

Answer 2

针对您的特定情况的简单解决方案是将大小作为参数传递给父构造函数。

这当然假设你真正的问题具有相同的特征，即基类构造函数所需的所有信息在调用构造函数时都可用。

Answer 3

更改父级以将大小作为参数是一种选择：

public Parent (int size)
{
    this.classDependentOnSize = new ClassDependentOnSize(size);
}

...

public Child (String selection, String... selectionArgs)
{
    super(selectionArgs.length);
    this.DBSelection = selection;
    this.DBSelectionArgs = selectionArgs;
}

另一种选择，有点像黑客，是有一个 init 方法，必须在构造后调用：

abstract class Parent
{
    private ClassDependentOnSize classDependentOnSize;

    public final void init()
    {
        this.classDependentOnSize = new ClassDependentOnSize(size());
    }

    public abstract int size ();
}

...

public Child (String selection, String... selectionArgs)
{
    this.DBSelection = selection;
    this.DBSelectionArgs = selectionArgs;
    init();
}

这里的正确答案可能会改变你的代码结构，但是，如果没有更具体的细节，我无法就你想如何去做这件事提供有意义的反馈。