开闭与接口隔离

Question

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

样本文件应该是这样的。

name    age
abc     20
xyz     vf
aaa     22

我有 JSON 文件说，哪个列应该包含什么？

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

}

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

以下代码工作正常。但是，这段代码似乎违反了开放封闭和接口隔离的原则。

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;
        }
    }

}

所以我正在考虑按照下面提到的方式重构代码。但是，想知道我会得到什么优势，有没有更好的方法？

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）
open/closed 原则指出 "software entities (classes, modules, functions, etc.) should be open for extension, but closed for modification"；也就是说，这样的实体可以允许在不修改其源代码的情况下扩展其行为。 see Reference

BUT：另一方面，Uncle Bob在this reference中提供了一些关于OCP含义的说明。（我在下面使用了它们）

首先：在我看来，你的class(DataValidation)并不冲突 开闭原则。

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

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

其次：使用接口隔离原则（ISP）
要使用 ISP，我们需要一些先决条件。我们的部分系统（或 class）之间应该有一些依赖关系。我们应该需要依赖管理来管理系统的某些部分，我们有意识地决定系统的每个部分可以依赖什么。 please read this reference in-depth

我认为您的 class 只是一个检查器 class，没有任何状态（属性或字段）。所以没有任何理由使用 ISP。

总结：使用所有面向对象的原则和启发式方法（如 SOLID），应该可以帮助我们降低复杂性。在你的项目中，没有必要使用它们。

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

Answer 2

Gholamali-Irani 提供了很好的答案，但我想添加一些我自己对你的主题的看法：

首先，几乎所有的最佳实践、范例等都试图提高可维护性、可测试性和可扩展性的程度。你真的需要它们吗？添加一些自定义和复杂类型的概率有多大？如果它非常低，那么您的第一个变体可能足以完成您的任务（不适用于一般的验证任务，只适合您的任务）。

其次，很大程度上取决于您如何使用它。你没有展示你是如何使用所有这些的 methods/classes/interfaces。 "Servant" 代码可能非常好，它可以是世界上最干净的代码，但是谁在乎它是否使用不正确或很难使用？

开闭与接口隔离

Open Closed and Interface Segregation

java

oop

solid-principles