ロガーはプライベートスタティックかどうか

ロガーを静的に宣言する必要がありますか？通常、ロガーの宣言には2つのタイプがあります。

    保護されたログログ=新しいLog4JLogger（aClass.class）;

または

    プライベート静的ログログ=新しいLog4JLogger（aClass.class）;

どちらを使用する必要がありますか？両方の長所と短所は何ですか？

非静的形式の利点は、正しいクラス名が使用されることを心配せずに、次のように（抽象的な）基本クラスで宣言できることです。

protected Log log = new Log4JLogger(getClass());

ただし、その欠点は、クラスのインスタンスごとにまったく新しいロガーインスタンスが作成されることです。これ自体は高価ではありませんが、かなりのオーバーヘッドが追加されます。これを避けたい場合は、static代わりにフォームを使用します。しかしその欠点はgetClass()、静的コンテキストでは使用できないため、ロガーの構築中に正しいクラス名が使用されるように、個々のクラスごとに宣言し、すべてのクラスで注意する必要があることです。ただし、平均的なIDEでは、このためのオートコンプリートテンプレートを作成できます。たとえばlogger+ ctrl+space。

一方、すでにインスタンス化されたロガーをキャッシュするファクトリーによってロガーを取得する場合、非静的フォームを使用してもオーバーヘッドはそれほど増えません。たとえば、Log4jにLogManagerはこの目的のためのがあります。

protected Log log = LogManager.getLogger(getClass());

以前は、すべてのロガーは静的である必要があると思っていました。ただし、wiki.apache.orgのこの記事では、クラスローダーのリークに関して、いくつかの重要なメモリの問題を取り上げています。ロガーを静的として宣言すると、宣言したクラス（および関連するクラスローダー）が、共有クラスローダーを使用するJ2EEコンテナーでガベージコレクションされなくなります。アプリケーションを何度も再デプロイすると、PermGenエラーが発生します。

ロガーを非静的として宣言する以外に、このクラスローダーのリークの問題を回避する方法はありません。

最も重要な違いは、それがログファイルにどのように影響するかです。ログはどのカテゴリに移動しますか？

• 最初の選択では、サブクラスのログはスーパークラスのカテゴリになります。それは私には直感に反しているようです。

• 最初のケースのバリアントがあります：

  保護されたログログ=新しいLog4JLogger（getClass（））;

  その場合、ログカテゴリは、ログに記録されたコードがどのオブジェクトで動作していたかを示します。

• 2番目の選択肢（プライベートスタティック）では、ログカテゴリはログコードを含むクラスです。したがって、通常は、ログに記録されていることを実行しているクラスです。

私はその最後のオプションを強くお勧めします。他のソリューションと比較して、次の利点があります。

• ログとコードの間には直接的な関係があります。ログメッセージの送信元を簡単に見つけることができます。
• ロギングレベルを調整する必要がある場合（これはカテゴリごとに行われます）、それは通常、特定のクラスによって書かれた特定のメッセージに関心がある（または関心がない）ためです。カテゴリがメッセージを書き込んでいるクラスではない場合、レベルを調整するのが難しくなります。
• 静的メソッドでログインできます
• ロガーはクラスごとに1回だけ初期化（またはルックアップ）する必要があるため、作成されたインスタンスごとではなく、起動時に初期化する必要があります。

また、欠点もあります。

• メッセージを記録するすべてのクラスで宣言する必要があります（スーパークラスロガーを再利用しないでください）。
• ロガーを初期化するときは、正しいクラス名を付けるように注意する必要があります。（しかし、良いIDEはあなたのためにそれを処理します）。

制御の反転を使用して、ロガーをコンストラクターに渡します。クラス内にロガーを作成すると、単体テストで時間の悪魔が生まれます。あなたはユニットテストを書いていますね？