如何在抽象类/继承中正确初始化 slf4j（使用 log4j）记录器

Question

在我的项目中，我有一个顶级抽象类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();
    }
}

还有两个继承类：CameraFrameGrabber和GrayscaleFilter。然而，当从 CameraFrameGrabber 或 GrayscaleFilter 调用方法 fireEvent() 时，消息将记录在 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

Answer 1

感谢斯卡夫曼指出了寻找可能更正的正确方向。我最终将我的 log4j.properties 更改为（作为 PatternLayout 文档 说）：

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

...一切都被正确记录。

Answer 2

某些人认为记录抽象类是一种反模式，因为它引入了潜在的意外依赖。但是，如果您确定您正在处理的所有内容都是您自己的，那么我想那没问题。

如果您想让日志消息显示具体实现而不是抽象类名称（这是有道理的），那么将抽象类中的日志初始化语句更改为：

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

而不是类似：

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

即

• 没有静态，因为它可以转换为其他的东西。
• getClass() 而不是具体的类