如何防止在我的类的实例上调用静态方法？ [复制]

Question

我有以下类，用于在我的 Android 应用程序的各个位置控制一些调试和 beta 测试选项。它只包含一些标志和一些逻辑来（反）序列化它到/从 JSON。

public class DevConfiguration {
    public boolean dontSendSMS;


    public static String toJsonString(DevConfiguration devConfiguration) {
        JSONObject json = new JSONObject();

        if( devConfiguration != null ) {
            try {
                json.put("dontSendSMS", devConfiguration.dontSendSMS);
            } catch (JSONException e) {
                e.printStackTrace();
            }
        }

        return json.toString();
    }


    public static DevConfiguration fromJsonString(String jsonString) {
        if( jsonString.isEmpty() )
            return null;

        DevConfiguration result = new DevConfiguration();

        try {
            JSONObject jsonObj = new JSONObject(jsonString);
            result.dontSendSMS = jsonObj.optBoolean("dontSendSMS", false);
        } catch (JSONException e) {
            e.printStackTrace();
        }

        return result;
    }

}

现在，在我的一个服务中，我在Intent 中接收到一个序列化的DevConfiguration 对象，然后可能会将其传递给另一个服务：

serviceIntent.putExtra("dev_conf", DevConfiguration.toJsonString(mDevConfiguration));

我选择将toJsonString() 方法设为静态，这样我就不会冒险在null 实例上调用它。但是，仍然有可能在某处出错并在实例上调用静态方法 - 可能是 null 实例！

mDevConfiguration.toJsonString(mDevConfiguration);

Android Studio 中有一个 Lint 警告，但它仍然是一个潜在的 NullPointerException 错误等待发生。我认为可以通过定义类似的 private 方法来隐藏它，但签名不同

/** Hide instance implementation **/
private String toJsonString(Object o){ return ""; }

当然，使用 DevConfiguration 参数调用它无论如何都会调用静态方法，并且 IDE 也不会给出比以前更多的警告。

有没有办法从实例变量中“隐藏”静态方法？

编辑

评论清楚地表明，在null 实例上调用静态方法是完全合法的。然而，问题不在于“如何在空实例上调用静态方法时防止 NullPointerException？”，而是更一般的 “如何防止在空实例上调用静态方法？我的班级的实例？”。

换句话说 - 如果一个人不小心尝试在实例上调用静态方法，有什么方法可以防止编译器编译？

Answer 1

对具有空值的变量调用静态方法不会引发 NullPointerException。以下代码将打印42，即使变量i 为空。

public class Test {
    public static void main(String... args) {
        Integer i = null;
        System.out.println(i.parseInt("42"));
    }
}

当通过变量调用静态方法时，真正重要的是变量的声明类型，而不是其值的引用类型。这与java中的静态方法不是多态的有关。

---

“如何防止在我的类的实例上调用静态方法？”

通过变量调用静态方法只是 Java 规范中定义的常规语言功能。如果有任何方法可以抑制它，我会感到惊讶。

如果我必须为选定的类执行此操作，我可能会将静态方法迁移到单独的“companion”实用程序（如另一个答案中所述）。

但是在你的类中拥有这样的静态（工厂）方法是一个非常好的习惯用法（例如，参见：Joshua Bloch，“Effective Java”，第 1 项：考虑静态工厂方法而不是构造函数）。我不会轻易放弃。

Answer 2

我看到了一些你可以做到这一点的方法：

1. 使用 Utils 类：

   public class Utils {
       public static String toJsonString(DevConfiguration devConfiguration) {
           JSONObject json = new JSONObject();
   
           if( devConfiguration != null ) {
               try {
                   json.put("dontSendSMS", devConfiguration.dontSendSMS);
               } catch (JSONException e) {
                   e.printStackTrace();
               }
           }
   
           return json.toString();
       }
   
   
       public static DevConfiguration fromJsonString(String jsonString) {
           if( jsonString.isEmpty() )
               return null;
   
           DevConfiguration result = new DevConfiguration();
   
           try {
               JSONObject jsonObj = new JSONObject(jsonString);
               result.dontSendSMS = jsonObj.optBoolean("dontSendSMS", false);
           } catch (JSONException e) {
               e.printStackTrace();
           }
   
           return result;
       }
   }
   

   现在您只需拨打Utils.method() 即可避免混淆。

2. 使用 Kotlin

   Kotlin 实际上让在动态接收器上调用静态方法变得非常困难（如果不是不可能的话）。它不会显示在方法建议中，如果您手动输入，它将带有红色下划线。它甚至可能无法编译，尽管我还没有走到那一步。

   Kotlin 还具有内置的 null 保护：instance?.method()。 ? 表示如果 instance 为空，method() 将不会执行。

3. 只是不要在动态接收器上调用静态方法。如果您不小心这样做，请返回并修复它。您不应该依赖 Java 为您解决语法错误。

最后，为什么还要这样做？我非常怀疑mDevConfiguration 是否永远为空，除非你在一个非常奇怪的地方初始化它。如果是这样，您可能希望重新组织您的代码。因为，同样，您不应该依赖 Java 来为您解决语法错误。此外，如果它为 null，它不会抛出 NPE，至少在 Java 中是这样，因为它不需要动态接收器来运行（这在 Kotlin 中可能不同）。

您可以自行决定编写应有的代码，并实施适当的空值检查、错误处理等。如果您遗漏了什么，也没什么大不了的；这就是为什么您在发布代码之前测试您的代码并修复您捕获的崩溃和错误的原因。 Google Play 控制台（如果您在此处发布）或 Firebase（如果您实施）或您的用户将报告您未捕获的任何内容。

很抱歉，如果上述内容听起来很苛刻，但我真的很难理解您为什么要这样做而不是仅仅检查您的代码。

如果你真的想保留这个结构，至少让DevConfiguration的构造函数私有：

public class DevConfiguration {
    //...

    private DevConfiguration() {}

    //...
}

这样，只有里面的静态方法才能创建实例。