LoggerFactory.getLogger(ClassName.class) vs LoggerFactory.getLogger(this.getClass().getName())

Question

我正在努力提高我的 Java 优化技能。 为了实现这一目标，我制作了一个旧程序，并且正在尽最大努力让它变得更好。 在这个程序中，我使用 SL4J 进行日志记录。 为了得到记录器，我做了：

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

在我写代码的时候，我认为这是最好的选择，因为我删除了对类名的引用（可能会被重构）。 但现在我不太确定了......

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

另一方面，保留对类名的引用，但它删除了一个方法调用。 对于一个班级来说，这可能不是一个很大的性能改进，但是当你有很多班级时，这可能是一些事情。

所以我的问题是：

哪种方法更好？ 使用类名还是通过反射获取？

请用利弊来激励你的答案。 谢谢你。

Answer 1

我会在这里分享我的看法。 我想说的是，从性能的角度来看，您不应该受到打扰。 可能在代码中有比这件事可以优化的部分:)

现在，关于你的问题。 看一下LoggerFactory 的代码

注意getLogger(Class<?> name)只是调用了重载的方法：

Logger logger = getLogger(clazz.getName());

并进行一些额外的计算。 所以带String的方法显然要快一些。

一般来说，模式是将 Logger 引用作为类中的静态字段来维护，如下所示：

public class SomeClass {
   private static final Logger LOG =   LoggerFactory.getLogger(SomeClass.class);
}

在这种情况下，你不能真正使用this.getClass()因为this实际上并不存在（你在一个静态的环境中运行）。

根据我的经验，最好使用ClassName.getClass()作为参数，除非您真的想使用来自不同类的相同记录器。 在这种情况下，您最好使用一些表示记录器的逻辑常量。

例如，假设您尝试使用 3 个不同的类来访问数据库。 因此，您创建记录器“DB”，分配一个将写入 database.log 的文件附加程序，并且您希望在这 3 个不同的类中重用相同的记录器。

所以你应该使用以下代码：

public class SomeClass {
   private static final Logger LOG =   LoggerFactory.getLogger("DB");
}

希望这可以帮助

Answer 2

迟到！

因为我将来可能会寻找这个。

有一种方法可以通过使用 Java 7 的 MethodHandles 类来创建对复制/粘贴友好的 Logger 实例（当然这几乎从来都不是做某事的好理由！）。

private static final Logger LOGGER = LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());

Answer 3

我通常做的是

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

然而，成语

protected final Logger log = LoggerFactory.getLogger(getClass());

同样常见。 在这个问题中，您可以找到有关这些约定的更多信息。

Answer 4

我更喜欢

Logger logger = LoggerFactory.getLogger(ClassName.class);

因为

this.getClass() 

可以由班级孩子之一覆盖，您将在日志中看到子班级名称。 有时可能会令人困惑，因为日志实际上是在父类中执行的

Answer 5

在静态上下文中，您不能使用LoggerFactory.getLogger(getClass())或LoggerFactory.getLogger(this.getClass().getName())

然后你必须使用LoggerFactory.getLogger(ClassName.class)

无论如何，我更喜欢 Lombok 的@Log4j2 ，更少的代码并完成工作： https : @Log4j2

Answer 6

如果不想每次声明记录器都写类名，可以使用下面的实用方法：

    public static org.slf4j.Logger getLogger() {
        final Throwable t = new Throwable();
        t.fillInStackTrace();
        return LoggerFactory.getLogger(t.getStackTrace()[1].getClassName());
    }

该方法可以采用以下方式：

private static final Logger LOG = TheClassContainingTheMethod.getLogger();

使用这种方法，所有类的记录器声明总是相同的。

Answer 7

我用这个

private static final Logger log = LoggerFactory.getLogger(ClassName.class);

Answer 8

用这个

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

}
// logic
try
{
// todo
}

catch (NullPointerException e) {
        logger.error("Error:-" + e.getMessage());
          return ResponseUtil.errorResponse(e.getMessage());
        }

        catch (Exception e) {
            logger.error("Error:-" + e.getMessage());
            return ResponseUtil.errorResponse(e.getMessage());
        }