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());
        }