java - 在设计类似记录器的类时处理条件语句？

Question

在设计像 log4j 这样的类时，如何处理条件语句？

以 log4j 为例，一个类可以有不同的级别(信息、警告、错误、其他...)。现在要打印不同级别的内容，每个级别都需要不同的方法。所以可以有如下的基本设计:

public class CustomLogger {

    enum Level{
        INFO, WARNING, ERROR;
    }

    private Level level;

    public CustomLogger(Level level) {
        this.level = level;
    }

    public void info(String s){
        if(level == Level.INFO || level == Level.WARNING || level == Level.ERROR){
            System.out.println(s);
        }
    }

    public void warning(String s){
        if(level == Level.WARNING || level == Level.ERROR){
            System.out.println(s);
        }
    }

    public void error(String s){
        if(level == Level.ERROR){
            System.out.println(s);
        }
    }
}

我猜这个设计的问题是它有太多的 if 条件，如果我需要添加一个新的状态，我就必须修改所有的 if 情况。所以我认为更好的设计如下:

有一个 State 父类(super class)，它由类 Info、类 Warning 和类 Error 扩展。 State 类具有方法 printInfo()、printWarning() 和 printError()。所有子类都实现这些方法，并在需要时留空。作为示例，Warning 类如下所示:

class Warning extends State{

    public void printInfo(String s){
        //don't print
    }

    public void printWarning(String s){
        System.out.println(s);
    }

    public void printError(String s){
        System.out.println(s);
    }
}

现在，CustomLogger 将与 State 关联，该关联将在某个时刻注入(inject)其中，并且 CustomLogger 中的所有方法都将调用State 类实现的方法。像这样的东西:

public class CustomLogger {

    private State state;

    public CustomLogger(State state) {
        this.state = state;
    }

    public void info(String s){
        state.printInfo(s);
    }

    public void warning(String s){
        state.printWarning(s);
    }

    public void error(String s){
        state.printError(s);
    }

}

尽管这消除了条件情况，但它引起了方法调用的依赖性。如果我添加一个新的 State ，那么该状态 printState() 方法就必须添加到每个子类中，我认为这不是一个好的做法。遇到这种情况还能做什么？

Answer 1

在这种情况下，最佳答案取决于您的系统要求将来如何变化。即使您遵循Open/close principle您无法阻止违反此原则的所有可能情况，因此您尝试在设计中涵盖您认为更有可能出现的情况。

无论如何，对于您的案例研究来说，一个好的解决方案将类似于 Java Logger 实现的解决方案。 。其中数值与 Logger 级别关联(每个可能的 Level 都有其值)，允许您检查当前的 Logger 是否应写入使用单个 if 将任何内容记录到日志中。此外，无需修改当前代码即可轻松扩展其行为。

例如，当您调用Logger#info(String)时:

public void info(String msg) {
    log(Level.INFO, msg);
}

方法log如下:

public void log(Level level, String msg) {
    if (!isLoggable(level)) {
        return;
    }
    // do any logging you want here
}

最后是关键isLoggable:

public boolean isLoggable(Level level) {
    if (level.intValue() < levelValue || levelValue == offValue) {
        return false;
    }
    return true;
}

其中levelValue是Logger的“当前有效级别值”。