如何避免对对象进行繁琐的空检查

Question

如果我有一个类想要接受可选记录器来记录调试信息：

public class A {

private Logger logger ; 

public A( Logger logger ) { 
    this.logger = logger ; 
}

public A() { 
    this( null ) ;
}

public void hello() {
    logger.debug( "hello" ) ;
}

public void goodbye() {
    logger.warn( "goodbye" ) ; 
}

}

有什么方法可以避免需要不断的空检查？

public void hello() {

    if( logger != null ) { 
        logger.debug( "hello" ) ;
    }

}

public void goodbye() {

    if( logger != null ) {
        logger.warn( "goodbye" ) ; 
    }

}

我想也许如果我创建一个记录器包装类，它可以为我进行检查，然后什么都不做，或者记录消息。但随后我需要包装我想要使用的每个方法。

Answer 1

我将创建 Logger 作为接口并实现真正的记录器和不记录任何内容的虚拟类。这样，您始终可以访问记录器而无需任何空检查。

Answer 2

所有答案基本上都建议使用空对象模式，请参阅：

• http://en。 wikipedia.org/wiki/Null_Object_pattern
• 空对象模式以避免空检查？

Answer 3

两件事：

1. 始终初始化记录器
2. （和/或）使用根本不做任何事情的哨兵记录器

Answer 4

可能是这样的：

private void logDebug(final String str, final Object ... parameters ) {
    if (this.logger != null && this.logger.isDebugEnabled()) {
        this.logger.debug(
           parameters.length > 0
              ? MessageFormat.format(str, parameters)
              : str
        );
    }
}

private void logInfo(final String str,final Object ... parameters) {
    if (this.logger != null && this.logger.isInfoEnabled()) {
        this.logger.info(
           parameters.length > 0
              ? MessageFormat.format(str, parameters)
              : str
        );
    }
}

现在您只需调用 logDebug(str) 或 logInfo(str)，您还可以使用 MessageFormat.format()：

logDebug("Called {0} method with parameters {1} and {2}","doStuff",foo,bar);

Answer 5

抱歉，如果这听起来很愚蠢，但是使方法 Warn 和 Debug static 不会解决这个问题吗？
如果您需要一个实例，只需在静态方法中创建它，或者如果它是单例，则在静态方法中检查是否为 null。

Answer 6

您可以在 1 参数构造函数中将布尔值“enableLogging”设置为 true，在空构造函数中将布尔值“enableLogging”设置为 false。因此，您可以为您的支票赋予更多意义 - 而且速度更快。

Answer 7

主要问题是您到处都使用 null，这意味着您自然需要到处测试 null。创建一个 Logger，其中所有方法都不执行任何操作，并在适当的情况下使用该实例。