単純なデリゲートであれば、ゲッターの戻り値をキャッシュする必要がありますか？

Question

例：

私はCSVファイルを作成していますが、私はCsvOptionsオブジェクトを持っていて、CSVファイルの作成にいくつかのパラメータを保持しています。

if (cells.hasNext()) { 
    output.write(csvOptions.getDelimiter()); 
} 

csvOptionsは単なるコンテナオブジェクト、あなたがまったくゲッターを使用しないように誘惑されているようなものです。 getterはprivateフィールドから値を返します。

それは

• ...実行速度の面で、私の例ではgetDelimiter()のように、些細なゲッターの戻り値をキャッシュする価値がありますか？
• ...コーディングスタイル？

Answer 1

私はMartin Fowler氏のリファクタリングが明示的に代わり、彼が直接

csvOptions.getDelimiter() 

を使用して有利に働くこの

char delimiter = csvOptions.getDelimiter(); 

// code here to use delimiter 

をやってない表彰することを信じています。引数は2倍です。最初にパフォーマンスのオーバーヘッドが最小限に抑えられ、コンパイラとJITersは関数呼び出しを最適化できます。次に、このメソッドを使用することで、実際にコードを将来リファクタリングするのが容易になります。

Fowler's book referenced here

Answer 2

キャッシングはオプションであってはなりません。 コンテナの場合は、不変にします。つまり、すべてのフィールドを最終的にパブリックにします。

Answer 3

JITは、実行時にすべての簡単なゲッターをインライン化できる必要があります。だから、これは本当に心配ではありません。

コーディングスタイルに関しては、値が1回だけ必要なときに、長すぎる場所に行を作成しない限り、ダイレクトコールを使用することをお勧めします。