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

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

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

在我编写代码的时候,我认为这是最好的选择,因为我删除了对类名的引用(可能会被重构).但现在我不再那么肯定了……

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

另一方面,保持对类名的引用,但它删除一个方法调用.对于一个班级来说,这可能不是一个很大的改进,但是当你有很多班级时,这可能就是一个问题.

所以我的问题是：

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

请用正反面来激励你的答案.谢谢.

我会在这里分享我的意见.我想说的是,你不应该从性能的角度来打扰你.可能在代码中有部分可以比这个东西更优化:)

现在,关于你的问题.看看LoggerFactory’s 代码

请注意,getLogger(Class<？> name)只调用重载方法：

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

并进行一些额外的计算.所以使用String的方法显然要快一些.

通常,模式是将Logger引用保持为类中的静态字段,如下所示：

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

在这种情况下,您无法真正使用this.getClass(),因为它实际上并不存在(您在静态上下文中运行).

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

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

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

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

希望这可以帮助

相关文章