スレッドを使用したJavaの最適化

Question

私はHashMapのキータイプとしてJavaでカスタムクラスFooを使用しています。 Fooインスタンスのすべてのフィールドは不変です（これらはfinalとprivateで宣言され、コンストラクタでのみ値が割り当てられます）。したがって、与えられたFooオブジェクトのhashCode()も修正されています。最適化のために、コンストラクタで計算して、hashCode()メソッドでその値を返すだけです。

Fooのインスタンスもvalue()のメソッドを持ち、オブジェクトがインスタンス化されると同様の固定値を返します。現在、コンストラクタで計算して返していますが、hashCode()とvalue()の間に違いがあります。hashCode()は、オブジェクトの作成後ほぼ即時に呼び出されますが、後でvalue()が呼び出されます。私は、ハッシュコードを計算するために別のスレッドを有するという理由だけで、同期の問題の実行時間を増加させることを理解し、しかし：

• これはvalue()を計算するための良い方法ですか？それは実行時間をまったく改善するでしょうか？
• は簡単ですThreads、またはプールなどを使用する必要がありますか？

注：私は私のプログラムの間違った部分を最適化してるようこれは見えるかもしれませんが、私はすでに「正しい」部分に働いて、へ〜17秒から下の平均実行時間を持ってきました〜2秒。 編集：これは5000個以上のオブジェクトがありますが、これは控えめな見積もりです。

Answer 1

遅延計算はここでの良いアプローチですが、これらのオブジェクトをたくさん作成すると、スレッドプールが使用できます。

value()の準備が完了するまでの戻り値は、無効な値を返すのを避け、代わりにブロックします（isValueReady()ヘルパーを追加する）か、即座に "未来"を返すようにします同じものを提供しますisReadyとブロック方法get。

また、「ずっと後で」に頼ってはいけません。使用する前に必ず値を確認してください。

Answer 2

私はvalueためFutureを作成することはお勧め - 静的fixedTheadPoolとsubmitその上value計算を作成。 valueは、それが利用可能になる前にアクセスされるというリスクはありません。この方法で - 最悪の場合には、valueにアクセスされているもの（例えば、デッドロックが懸念される場合やuse the version with a timeout）ができる

Future.getためにチェックスロー例外をFuture.getコールでブロックされるということです迷惑も、あなたのクラスのgetterメソッドでget呼び出しをラップし、概念とアーキテクチャの面でRuntimeException

class MyClass { 
    private static final ExecutorService executor = Executors.newFixedThreadPool(/* some value that makes sense */); 

    private final Future<Value> future; 

    public MyClass() { 
     future = executor.submit(/* Callable */); 
    } 

    public boolean isValueDone() { 
     return future.isDone(); 
    } 

    public Value value() { 
     try { 
      return future.get(); 
     } catch(InterruptedException|ExecutionException e) { 
      throw new RuntimeException(e); 
     } 
    } 
}