从父类内部调用父方法

Question

的摘录

package dictionary;

import java.util.Iterator;
import java.util.LinkedList;
import java.util.regex.*;

public class IntelliCwDB extends CwDB {

    public IntelliCwDB(String filename) {
        super(filename);
    }

    @Override
    public void add(String word, String clue) {
        System.out.println("inelli");
    }
}

这是我的代码和 CwDB

package dictionary;

import java.util.LinkedList;
import java.io.*;
import java.util.Scanner;

public class CwDB {
    protected LinkedList<Entry> dict;

    public CwDB(String filename) {
        dict = new LinkedList<Entry>();
        createDB(filename);
    }

    public void add(String word, String clue) {
        System.out.println("cwdb");
        dict.add(new Entry(word, clue));
    }

    protected void createDB(String filename) {
        try {
            BufferedReader f = new BufferedReader(new FileReader(filename));
            while (f.ready()) {
                this.add(f.readLine(), f.readLine());
            }
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
}

...在 main() 部分中，我创建了一个新的 IntelliCwDB 对象，该对象触发 createDB()< /代码>。

问题是我希望 CwDB.createDB() 使用它自己的 CwDB.add() 方法，而不是来自 IntelliCwDB 的方法。除了单独创建 CwDB，然后将其传递到 IntelliCwDB 的构造函数中以重写 LinkedList字典数据库之外，还有其他巧妙的解决方案吗？ ？

Answer 1

您经历了不应从构造函数调用虚拟方法的原因之一。有关此内容的更多详细信息，请参阅 Effective Java 第 2 版，第 17 项：设计和记录继承，否则禁止继承。。

解决您的问题的最简单的解决方案是将基类方法拆分为一个非虚拟（final 和/或 private）方法，以及另一个虚拟方法，该方法调用前者在基类实现中。

@aioobe 更快地提供了一个例子:-)

Answer 2

您可以这样解决它：

• 创建 CwDB.add 的私有（或最终）版本，我们称之为 privateAdd。

• 让 CwDB 中旧的 add 方法改为调用此方法。

• 每当您想确保使用 add 的 CwDB 版本时，只需调用 privateAdd 即可。

示例代码

public class CwDB {

    // ...

    public void add(String word, String clue) {
        privateAdd(word, clue);
    }

    private void privateAdd(String word, String clue) {
        System.out.println("cwdb");
        dict.add(new Entry(word, clue));
    }

    protected void createDB(String filename) {
        // ...
            // "Calling parent method from within the parent class"  :-)
            this.privateAdd(f.readLine(), f.readLine());
        // ...
    }

    // ...
}

---

正如 @Péter Török 正确指出的那样：您永远不应该从构造函数中（直接或间接）调用虚拟方法。原因很简单：子类将在其超类（及其自身）正确初始化之前运行代码。 （无论它是否适用于这个特定的例子，都是有道理的。）

Answer 3

我会将 add 方法移至 CwDB 中的 addInternal，并创建一个新的 add 来调用 addInternal。然后在createDB方法中，调用addInternal来获取正确的方法。
例如。

class CwDB {
   ..
   private void addInternal(String word, String clue) {
     ..
   }

   public void add(String word, String clue) {
     addInternal(word, clue);
   }

   public void createDB(String filename) {
      ..
      addInternal(w, c);
      ..
   }
}