验证器类，它通过几个简单的标准进行验证--这是GOD-Object吗？

Question

public class Validator {
    Collection<ValidationMessage> validationMessages;

    class ValidationMessage {
    }

    enum MessageType {
    }

    public validate(lines) throws ValidationException {
        for (line : lines) {
            validateCriteriaA(line)
            validateCriteriaB(line)
            ... 
            validateCriteriaF(line)
        }
        if (...)
            throw new ValidationException();
    }

    private validateCriteriaA(Line line) {
        if (...) {
            validationMessages.add("criteria failed")
        }
    }
    private validateCriteriaB(Line line)
    private validateCriteriaC(Line line)
    private validateCriteriaD(Line line)
    private validateCriteriaE(Line line)
    private validateCriteriaF(Line line)
    private helperMethodX
    private helperMethodY
    private helperMethodZ
}

Answer 1

我不会称它为上帝类，但我会称它违反了开闭原则。

我看到的一个问题是这个验证器不能伸缩。只有半打的标准，它的大小是合理的；但要在以后添加条件,您必须每次增加该类。此外，该验证器不能在运行时更改其逻辑，例如跳过或重新排序其条件。

这些标准可以被分离成单独的策略类来解决所有这些问题。