synchronization
Java ディレクティブを使用して、クラスにきめ細かい同期を実装しようとしています。つまり、同期できるコードの量を最小限に抑えようとしています。コードをインラインでコメントして、何をするかを説明し、コードの後にコメントします。コードを改善する方法を尋ねます。
public class MyClass {
private static volatile MyClass singletonInstance = null;
private HashMap<String, Integer> mHashMap = null;
private String mStringA = null;
private String mStringB = null;
// Use double check technique to use synchronization only
// at the first getInstance() invocation
public static MyClass getInstance() {
if (singletonInstance == null) {
synchronized (MyClass.class) {
if (singletonInstance == null)
singletonInstance = new MyClass();
// Initialize class member variables
singletonInstance.mHashMap = new HashMap<String,Integer>();
singletonInstance.mStringA = new String();
singletonInstance.mStringB = new String();
}
}
return singletonInstance;
}
// The following two methods manipulate the HashMap mHashMap
// in a secure way since they lock the mHashMap instance which
// is always the same and is unique
public Integer getIntegerFromHashmap(String key) {
synchronized (mHashMap) {
return mHashMap.get(key);
}
}
public void setIntegerIntoHashmap(String key, Integer value) {
synchronized (mHashMap) {
mHashMap.put(key, value);
}
}
// With the two String members mStringA and mStringB the problem is
// that the instance of String pointed by the member is varied by the
// setter methods, so we can not lock in a fine grained way and we
// must lock on the singletonInstance.
public String getStringA() {
synchronized (singletonInstance) {
return mStringA;
}
}
public String getStringB() {
synchronized (singletonInstance) {
return mStringB;
}
}
public void setStringA(String newString) {
synchronized (singletonInstance) {
mStringA = newString;
}
}
public void setStringB(String newString) {
synchronized (singletonInstance) {
mStringB = newString;
}
}
}
String
2 つのメンバー変数の getter メソッドと setter メソッドについて私が気に入らない点は、ロックをオンにsingletonInstance
すると、アクセスしようとしているスレッドが、操作中のスレッドがそのロックを解放するmStringB
まで待機する可能性があることです。mStringA
この場合、あなたはどうしますか?private final Integer mStringALock = new Integer(0)
とprivate final Integer mStringBLock = new Integer(0)
inのような 2 つのメンバー変数を作成し、それぞれとMyClass
のゲッター メソッドとセッター メソッドの同期ブロックで使用しますか?mStringA
mStringB
String
上記のコードとメンバー変数のきめの細かい同期のために提案されたバリエーションを改善する方法についていくつかのアイデアがある場合は、大歓迎です:)