我可以在Java中调用构造函数中的方法吗？

Question

我有一种情况，当类被实例化时，我只想读取一次配置文件。

假设我有一个名为 readConfig() 的方法，它读取配置并将其放入 Map 对象中。当程序需要使用配置值时，它会使用其定义键读取对象。我想知道构造函数仅在其生命周期中调用一次。我可以将我的方法 readConfig() 放入构造函数中，这将使我受益于一次调用，还是有另一种机制可以做到这一点？

Answer 1

你可以：这就是构造函数的用途。此外，您还要明确该对象永远不会在未知状态下构造（未加载配置）。

你不应该：在构造函数中调用实例方法是危险的，因为对象尚未完全初始化（这主要适用于可以重写的方法）。此外，构造函数中的复杂处理也会对可测试性产生负面影响。

Answer 2

更好的设计是

public static YourObject getMyObject(File configFile){
    //process and create an object configure it and return it
}

• 工厂设计模式

Answer 3

我可以将我的方法 readConfig() 放入构造函数中吗？

在构造函数中调用不可重写的方法是一种可接受的方法。
如果该方法仅由构造函数使用，您可能想知道是否确实需要将其提取到方法（甚至 private）中。

如果您选择将构造函数完成的某些逻辑提取到方法中，则对于任何方法，您都必须选择适合方法要求的访问修饰符，但在这种特定情况下，它更重要的是保护方法免受重写该方法的执行必须冒着使超类构造函数不一致的风险。

因此，如果它仅由类的构造函数（和实例方法）使用，则它应该是private。
否则，如果该方法在包内或子类中重用，则它应该同时是package-private和final。

这会给我一次调用带来好处，或者是否有另一种机制可以做到这一点？

使用这种方式没有任何好处或缺点。
我不鼓励在构造函数中执行太多逻辑，但在某些情况下，在构造函数中初始化多个事物可能是有意义的。
例如，复制构造函数可以执行很多事情。
多个 JDK 类说明了这一点。
以 HashMap 复制构造函数为例，它构造一个新的 HashMap，其映射与指定的 Map 参数相同：

public HashMap(Map<? extends K, ? extends V> m) {
    this.loadFactor = DEFAULT_LOAD_FACTOR;
    putMapEntries(m, false);
}

final void putMapEntries(Map<? extends K, ? extends V> m, boolean evict) {
    int s = m.size();
    if (s > 0) {
        if (table == null) { // pre-size
            float ft = ((float)s / loadFactor) + 1.0F;
            int t = ((ft < (float)MAXIMUM_CAPACITY) ?
                     (int)ft : MAXIMUM_CAPACITY);
            if (t > threshold)
                threshold = tableSizeFor(t);
        }
        else if (s > threshold)
            resize();
        for (Map.Entry<? extends K, ? extends V> e : m.entrySet()) {
            K key = e.getKey();
            V value = e.getValue();
            putVal(hash(key), key, value, false, evict);
        }
    }
}

提取映射填充的逻辑putMapEntries() 中的方法是一件好事，因为它允许：

• 在其他上下文中重用该方法。例如，clone() 和 putAll() 也使用它
• （次要但有趣），给出一个有意义的名称来传达执行的逻辑

Answer 4

构造函数仅被调用一次，因此您可以安全地执行您想要的操作，但是从构造函数内部而不是直接调用方法的缺点是，如果方法失败，您将无法获得直接反馈。调用的方法越多，这就会变得越困难。

一种解决方案是提供方法，您可以在构造对象后调用这些方法来查询对象的“运行状况”。例如，方法 isConfigOK() 可用于查看配置读取操作是否正常。

另一种解决方案是在失败时在构造函数中抛出异常，但这实际上取决于这些失败的“致命性”程度。

class A
{
    Map <String,String> config = null;
    public A()
    {
        readConfig();
    }

    protected boolean readConfig()
    {
        ...
    }

    public boolean isConfigOK()
    {
        // Check config here
        return true;
    }
};

Answer 5

你可以。但是通过将其放在构造函数中，您会使对象难以测试。

相反，您应该：

• 提供带有 setter 的配置
• 有一个单独的 init() 方法

依赖注入框架为您提供这些选项。

public class ConfigurableObject {
   private Map<String, String> configuration;
   public ConfigurableObject() {

   }

   public void setConfiguration(..) {
       //...simply set the configuration
   }
}

第二个选项的示例（当对象由容器管理时最好使用）：

public class ConfigurableObject {
   private File configFile;
   private Map<String, String> configuration;
   public ConfigurableObject(File configFile) {
       this.configFile = configFile;
   }

   public void init() {
       this.configuration = parseConfig(); // implement
   }
}

当然，可以通过仅使用构造函数来编写，

public ConfigurableObject(File configfile) {
    this.configuration = parseConfig(configFile);
}

但是这样您将无法提供模拟配置。

我知道第二个选项听起来更冗长并且容易出错（如果您忘记初始化）。如果您在构造函数中执行此操作，那么不会对您造成太大伤害。但是，使代码更加面向依赖注入通常是一个很好的做法。

第一个选项是最好的 - 它可以与 DI 框架和手动 DI 一起使用。

Answer 6

单例模式

public class MyClass() {

    private static MyClass instance = null;
    /**
    * Get instance of my class, Singleton
    **/
    public static MyClass getInstance() {
        if(instance == null) {
            instance = new MyClass();
        }
        return instance;
    }
    /**
    * Private constructor
    */
    private MyClass() {
        //This will only be called once, by calling getInstanse() method. 
    }
}

Answer 7

为什么不使用静态初始化块？其他详细信息请参见此处：
静态初始化块