java - 开闭和接口(interface)隔离

Question

我正在编写一个代码，它基本上读取一个文本文件(表格格式)并检查该文件是否包含预期的数据类型。为此我写了下面的课。

示例文件应该是这样的。

name    age
abc     20
xyz     vf
aaa     22

我有 JSON 文件，其中显示哪个列应该包含什么？

{
    filename:"test.txt",
    cols:{
            name:string,
            age: int
    }

}

JSON 文件包含每行的数据类型，因此我知道会发生什么？

以下代码运行没有任何问题。然而，这段代码似乎违反了开放封闭和接口(interface)隔离原则。

public class DataValidation {

    public boolean isInt(String value) {
        try {
            Integer.parseInt(value);
            return true;
        } catch (NumberFormatException ne) {
            return false;
        }
    }


    public boolean isFloat(String value) {
        try {
            Float.parseFloat(value);
            return true;
        } catch (NumberFormatException ne) {
            return false;
        }
    }

}

所以我想按如下所述对代码进行Refector。但是，想知道我会得到什么好处以及有什么更好的方法吗？

public interface DataValidation {

    boolean validate(String value);
}

public class IntValidator implements DataValidation {
    public boolean validate(String value) {
        try{
            Integer.parseInt(value);
            return true;
        }catch (NumberFormatException ne){
            return false;
        }
    }
}

Answer 1

开闭原理 (OCP) 的基本定义:(Meyer 1988)
开放/封闭原则规定“软件实体(类、模块、函数等)应该对扩展开放，但对修改关闭”；也就是说，这样的实体可以允许在不修改其源代码的情况下扩展其行为。 see Reference

但是:另一方面，this reference 中的鲍勃叔叔对 OCP 的含义进行一些澄清。 (我在下面使用了它们)

首先:在我看来，你的类(class)( DataValidation )没有冲突 开闭原则。

您的类只需检查原始数据类型。 (正如您在评论中回答我的问题一样)。 Java 中只有 8 种基本数据类型。 这个数字将来不会改变。因此，如果将所有 8 个方法放在一个类中，将来就不需要对数据类型进行任何扩展或修改。
另一方面，OCP 是在不对旧代码进行任何更改的情况下添加新的源代码。因此，如果Java添加新的数据类型，您可以轻松添加方法，而无需修改代码的其他部分。

因此，我认为你的类还不够大，不足以违反开闭原则。

第二:使用接口(interface)隔离原则(ISP)
要使用 ISP，我们需要一些先决条件。我们的系统(或类)的一部分之间应该有一些依赖关系。我们应该需要依赖管理来管理系统的某些部分，并且我们有意识地决定系统的每个部分可以依赖什么。 please read this reference in-depth

我认为您的类只是一个 Checker 类并且没有任何状态(属性或字段)。所以没有任何理由使用ISP。

总结:使用所有面向对象的原则和启发式(如 SOLID)，应该有助于我们降低复杂性。在你的项目中，没有必要使用它们。

为您的问题提供解决方案
您可以使用enum DataTypes {boolean, char, _etc_}对于原始数据类型，仅使用一种方法，如 DataType getDataType(String S)获取给定 String 的类型如enum 。 但是你的方法( DataValidation class )也足够好了。