10

在我的项目中,我有一个顶级抽象类FrameProducer。我在这个级别添加了一个 slf4j 记录器,以便每个继承类都已经拥有它。这里的代码:

public abstract class FrameProducer extends Observable {

    protected final Logger logger = LoggerFactory.getLogger(this.getClass());

    protected BufferedImage frame;

    public BufferedImage getFrame() {
        return frame;
    }

    public void fireEvent() {
        logger.debug("Firing event. Implementing class: {}", this.getClass());
        setChanged();
        notifyObservers();
    }
}

还有两个继承类:CameraFrameGrabberGrayscaleFilter. 然而,当方法fireEvent()被调用CameraFrameGrabberGrayscaleFilter消息被记录在FrameProducer级别时。为了清楚起见,这里是日志:

FrameProducer.fireEvent - Firing event. Implementing class: class com.ofj.frameaccess.CameraFrameGrabber
FrameProducer.fireEvent - Firing event. Implementing class: class com.ofj.frameaccess.GrayscaleFilter

是否可以初始化记录器以FrameProducer使所有内容都记录在我的类层次结构中最专业的级别?

谢谢你的帮助。


编辑:我的 log4j.properties 看起来像这样:

log4j.rootCategory=TRACE, CONSOLE
log4j.appender.CONSOLE=org.apache.log4j.ConsoleAppender
log4j.appender.CONSOLE.Threshold=DEBUG
log4j.appender.CONSOLE.layout=org.apache.log4j.PatternLayout
log4j.appender.CONSOLE.layout.ConversionPattern=%C{1}.%M - %m%n
4

2 回答 2

10

登录抽象类被一些人认为是一种反模式,因为它引入了潜在的意外依赖。但是,如果您确定您正在处理的所有东西都是您自己的,那我想这没问题。

如果您想获取日志消息以显示具体实现而不是抽象类名称(这很有意义),那么将抽象类中的日志记录初始化语句更改为:

private final Logger logger = LoggerFactory.getLogger(getClass());

而不是类似的东西:

private static final Logger logger = LoggerFactory.getLogger(MyAbstractClass.class);

IE

  • 没有静态,因为它可以转换为其他东西。
  • getClass() 而不是特定的类
于 2015-04-09T11:30:48.487 回答
1

感谢您指出正确的方向以寻找可能的更正 skaffman。我最终将我的更改为(如PatternLayout 文档log4j.properties中的示例之一所述):

log4j.rootCategory=TRACE, CONSOLE
log4j.appender.CONSOLE=org.apache.log4j.ConsoleAppender
log4j.appender.CONSOLE.Threshold=DEBUG
log4j.appender.CONSOLE.layout=org.apache.log4j.PatternLayout
log4j.appender.CONSOLE.layout.ConversionPattern=%-6r [%15.15t] %-5p %30.30c %x - %m%n

...并且一切都会正确记录。

于 2010-10-12T12:24:02.300 回答