スレッドセーフと静的クラスのジェネリック

Question

私はStats計算に使用するUtilクラスを持っています。指数移動平均は複数のスレッド間で計算されます。また、このスレッドはdifferents値を倍にし、もう一方をlongに渡します。私はジェネリック医薬品を作りたいと思っており、をに同期させてメソッドのシグネチャに適用していることを確認してください。

public class StatUtils { 

    public static class WMA { 
     // MMA 
    } 
    public static class EMA { 

     /** The alpha. */ 
     private static double staticAlpha = 0.9; 

     /** The old value. */ 
     private static double staticOldValue = 1.0; 

     /** 
     * Compute. 
     * 
     * @param pValue the value 
     * @return the double 
     */ 
     public static synchronized double compute(double pValue) { 
      if (staticOldValue == 0.0) { 
       staticOldValue = pValue; 
       return pValue; 
      } 
      double lValue = staticOldValue + staticAlpha * (pValue - staticOldValue); 
      staticOldValue = lValue; 
      return lValue; 
     } 
    } 
} 

computeメソッドのスレッドは安全ですか？この静的なクラスを一般的にすることが可能な場合は？

Answer 1

あなたのメソッドはスレッドセーフですが、ジェネリックを静的クラスにすることはできません。ジェネリック型は、クラスがインスタンス化されるときに解決されますが、これは静的クラスでは発生しません。

次のような一般的なメソッドを定義することができます。

public static synchronized <T extends Number> compute(T pValue) 

しかし、長い値は問題なく倍増するキャストすることができますので、あなたのケースでは、単に二重を使用することができます。

Answer 2

はい、現在のアプローチはスレッドセーフです。しかし、全体としてcomputeメソッドが1つのスレッドでブロックされるほど効率的ではないかもしれません。

私はEMAアルゴリズムについてはよく分からないですが、あなたは大丈夫です場合のスレッドがstaticOldValueのほぼ最新の値を使用することを、あなたが好きそれを改善することができます

public static class EMA { 

    /** The alpha. */ 
    private static double staticAlpha = 0.9; 

    /** The old value. */ 
    private static double staticOldValue = 1.0; 

    private static Object monitor = new Object(); 

    /** 
    * Compute. 
    * 
    * @param pValue the value 
    * @return the double 
    */ 
    public static double compute(double pValue) { 
     synchronized (monitor) { 
     if (staticOldValue == 0.0) { 
      staticOldValue = pValue; 
      return pValue; 
     } 
     } 

     double lValue = staticOldValue + staticAlpha * (pValue - staticOldValue); 
     synchronized (monitor) { 
     staticOldValue = lValue; 
     } 

     return lValue; 
    } 
    }