我正在努力提高我的
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"); }
希望这可以帮助