Java 1.4の同期:メソッドの1つのインスタンスのみの実行を許可します(ブロッキングなし)?

StackOverflow https://stackoverflow.com/questions/152138

質問

翻訳ユーティリティを提案するクラスがあります。翻訳自体は30分ごとにリロードする必要があります。そのためにSpring Timerサポートを使用します。基本的に、私のクラスは次のようになります:

public interface Translator {
    public void loadTranslations();
    public String getTranslation(String key);
}

loadTranslations()は実行にかなり時間がかかる可能性があるため、実行中は古い翻訳を引き続き使用できます。これは、翻訳をローカルマップにロードし、すべての翻訳がロードされたときに参照を変更するだけで行われます。

私の問題は、スレッドがすでに翻訳をロードしているときに、2番目のスレッドも実行しようとしていることを確認する方法です。

同期メソッドはロードのみをキューに入れます...私はまだJava 1.4を使用しているため、java.util.concurrentはありません。

ご協力ありがとうございます!

役に立ちましたか?

解決

何らかの形式のロックメカニズムを使用して、タスクがまだ進行中でない場合にのみ実行します。ロックトークンの取得は、1ステップのプロセスである必要があります。参照:

/**
 * @author McDowell
 */
public abstract class NonconcurrentTask implements Runnable {

    private boolean token = true;

    private synchronized boolean acquire() {
        boolean ret = token;
        token = false;
        return ret;
    }

    private synchronized void release() {
        token = true;
    }

    public final void run() {
        if (acquire()) {
            try {
                doTask();
            } finally {
                release();
            }
        }
    }

    protected abstract void doTask();

}

タスクが同時に実行された場合に例外をスローするテストコード:

public class Test {

    public static void main(String[] args) {
        final NonconcurrentTask shared = new NonconcurrentTask() {
            private boolean working = false;

            protected void doTask() {
                System.out.println("Working: "
                        + Thread.currentThread().getName());
                if (working) {
                    throw new IllegalStateException();
                }
                working = true;
                try {
                    Thread.sleep(1000);
                } catch (InterruptedException e) {
                    throw new RuntimeException(e);
                }
                if (!working) {
                    throw new IllegalStateException();
                }
                working = false;
            }
        };

        Runnable taskWrapper = new Runnable() {
            public void run() {
                while (true) {
                    try {
                        Thread.sleep(100);
                    } catch (InterruptedException e) {
                        throw new RuntimeException(e);
                    }
                    shared.run();
                }
            }
        };
        for (int i = 0; i < 100; i++) {
            new Thread(taskWrapper).start();
        }
    }

}

他のヒント

私は.netのバックグラウンドから来ました(Javaの経験はまったくありません)が、メソッドの最初に実行されているかどうかをチェックする単純な静的フラグを試すことができます。次に、そのフラグの読み取り/書き込みが同期されていることを確認するだけです。そのため、開始時にフラグを確認し、設定されていない場合は設定し、設定されている場合は戻ります。設定されていない場合は、メソッドの残りを実行し、完了後に設定を解除します。 try / finallyにコードを入れ、finallyにiunsettingフラグを設定して、エラーが発生した場合に常に設定が解除されるようにしてください。非常に単純化されていますが、必要なものはこれだけです。

編集:これは、実際にはメソッドを同期するよりもおそらくうまく機能します。終了する前に新しい翻訳をすぐに本当に必要とするからです。そして、しばらく待たなければならない場合、スレッドを長時間ロックアップしたくないかもしれません。

ロードスレッドのハンドルを保持して、実行中かどうかを確認しますか

または、同期フラグを使用してロードが進行中かどうかを示すことはできませんか?

これは、実際にシングルトン(gasp!)の構築を管理するのに必要な、古典的な方法で行われるコードと同一です:

if (instance == null) {
  synchronized {
    if (instance == null) {
       instance = new SomeClass();
    }
  }
}

内部テストは外部テストと同一です。外側のテストは、定期的に同期ブロックに入らないようにするためです。内側のテストは、最後にテストを行ってから状況が変わっていないことを確認するためです(同期に入る前にスレッドを横取りできた可能性があります)。

あなたの場合:

if (translationsNeedLoading()) {
  synchronized {
    if (translationsNeedLoading()) {
       loadTranslations();
    }
  }
}

UPDATE:シングルトンを構築するこの方法は、JDK1.4では確実に機能しません。説明については、こちらをご覧ください。ただし、このシナリオでは大丈夫だと思います。

ライセンス: CC-BY-SA帰属
所属していません StackOverflow
scroll top