同期された静的メソッドはJavaでどのように機能し、それをHibernateエンティティのロードに使用できますか？

基本的なデータアクセスを実行するためにHibernate関数を呼び出す静的メソッドを持つutilクラスがある場合。メソッドの作成がsynchronizedスレッドの安全性を確保するための適切なアプローチであるかどうか疑問に思っています。

これにより、同じDBインスタンスへの情報へのアクセスを防ぐことができます。ただし、次のコードがgetObjectById特定のクラスから呼び出されたときに、すべてのクラスに対して呼び出されるのを妨げているかどうかは確信しています。

public class Utils {
     public static synchronized Object getObjectById (Class objclass, Long id) {
           // call hibernate class
         Session session = new Configuration().configure().buildSessionFactory().openSession();
         Object obj = session.load(objclass, id);
         session.close();
         return obj;
     }

     // other static methods
}

静的メソッドロックで同期を使用することで、クラスのメソッドと属性を同期します（インスタンスのメソッドと属性ではなく）

だからあなたの仮定は正しいです。

メソッドを同期させることがスレッドセーフを確保するための正しいアプローチかどうか疑問に思っています。

あんまり。代わりに、RDBMSにその機能を実行させる必要があります。彼らはこの種のものを得意としています。

データベースへのアクセスを同期することによって得られる唯一のことは、アプリケーションをひどく遅くすることです。さらに、投稿したコードで毎回セッションファクトリを構築しているため、アプリケーションは実際のジョブを実行するよりもDBへのアクセスに多くの時間を費やします。

次のシナリオを想像してみてください。

クライアントAとBは、テーブルTのレコードXに異なる情報を挿入しようとします。

あなたのアプローチでは、RDBMSがAからの半分の情報とBからの半分の情報を同時に挿入するのを防ぐので、これがDBでとにかく発生するとき、次々に呼び出されることを確認することだけが得られます。 。結果は同じですが、遅くなるのは5倍（またはそれ以上）だけです。

おそらく、Hibernateドキュメントの「Transactions and Concurrency」の章を参照することをお勧めします。ほとんどの場合、解決しようとしている問題はすでに解決されており、はるかに優れた方法です。

より一般的な質問に対処するには...

同期されたメソッドを使用することは、実際には省略形であることに注意してください（クラスがSomeClassであると想定）

synchronized static void foo() {
    ...
}

と同じです

static void foo() {
    synchronized(SomeClass.class) {
        ...
    }
}

そして

synchronized void foo() {
    ...
}

と同じです

void foo() {
    synchronized(this) {
        ...
    }
}

ロックとして任意のオブジェクトを使用できます。静的メソッドのサブセットをロックしたい場合は、

class SomeClass {
    private static final Object LOCK_1 = new Object() {};
    private static final Object LOCK_2 = new Object() {};
    static void foo() {
        synchronized(LOCK_1) {...}
    }
    static void fee() {
        synchronized(LOCK_1) {...}
    }
    static void fie() {
        synchronized(LOCK_2) {...}
    }
    static void fo() {
        synchronized(LOCK_2) {...}
    }
}

（非静的メソッドの場合、ロックを非静的フィールドにする必要があります）

静的メソッドは、クラスをロックのオブジェクトとして使用します。これは、例ではUtils.classです。はい、大丈夫です。

static synchronized クラスのロックを保持することを意味します Classオブジェクトにsynchronizedロックを保持することを意味し、as はクラスのオブジェクト自体にロックを保持することを意味します。つまり、（実行の）スレッドで非静的同期メソッドにアクセスしている場合でも、別のスレッドを使用して静的同期メソッドにアクセスできます。

したがって、任意の時点で、2つ以上の同じ種類のメソッド（2つの静的メソッドまたは2つの非静的メソッド）にスレッド以外の方法でアクセスすることはできません。

一度に1つのスレッドのみがDBにアクセスできるように強制するのはなぜですか？

必要なロックを実装するのは、データベースドライバーの仕事です。Connectionが一度に1つのスレッドでのみ使用されるとです。

ほとんどの場合、データベースは複数の並列アクセスを完全に処理できます

それがデータベースのデータに関係している場合は、データベース分離ロックを利用して実現してみませんか？

あなたの質問に答えるために、はいそれはします：あなたのsynchronizedメソッドは一度に複数のスレッドによって実行されることはできません。

どのようにsynchronizedJavaのキーワード作品

synchronized静的メソッドにキーワードを追加すると、そのメソッドは一度に1つのスレッドからのみ呼び出すことができます。

あなたの場合、すべてのメソッド呼び出しは次のようになります：

• 新しいを作成します SessionFactory
• 新しいを作成します Session
• エンティティをフェッチする
• エンティティを呼び出し元に返す

ただし、これらはあなたの要件でした：

• これにより、同じDBインスタンスへの情報へのアクセスを防ぐことができます。
• 防止は、getObjectByIdそれが特定のクラスによって呼び出されたときに、すべてのクラスのために呼び出されています

したがって、getObjectByIdメソッドがスレッドセーフであっても、実装は間違っています。

SessionFactory ベストプラクティス

の SessionFactoryスレッドセーフであり、それはそれはエンティティクラスを解析し、内部実体のメタモデル表現を構築する必要があるとして作成するには、非常に高価な物です。

したがって、SessionFactoryすべてを作成する必要はありませんgetObjectByIdメソッド呼び出しでを。

代わりに、そのためのシングルトンインスタンスを作成する必要があります。

private static final SessionFactory sessionFactory = new Configuration()
    .configure()
    .buildSessionFactory();

の Session必ず閉じる必要があります

あなたはで閉じませんでしSessionたfinallyブロックたため、エンティティの読み込み中に例外がスローされた場合、データベースリソースがリークする可能性があります。

Session.loadメソッドによると、HibernateExceptionエンティティがデータベースで見つからない場合、JavaDocはをスローする可能性があります。

このメソッドを使用してインスタンスが存在するかどうかを判断しないでください（get()代わりに使用してください）。これは、存在しないことが実際のエラーになる、存在すると想定するインスタンスを取得する場合にのみ使用してください。

そのfinallyためSession、次のようにブロックを使用してを閉じる必要があります。

public static synchronized Object getObjectById (Class objclass, Long id) {    
     Session session = null;
     try {
         session = sessionFactory.openSession();
         return session.load(objclass, id);
     } finally {
         if(session != null) {
             session.close(); 
         }
     }
 }

マルチスレッドアクセスの防止

あなたの場合、1つのスレッドだけがその特定のエンティティにアクセスできるようにする必要がありました。

ただし、このsynchronizedキーワードは、2つのスレッドがgetObjectById同時にです。2つのスレッドがこのメソッドを順番に呼び出す場合でも、このエンティティを使用する2つのスレッドがあります。

したがって、他のスレッドがオブジェクトを変更できないように特定のデータベースオブジェクトをロックする場合は、データベースロックを使用する必要があります。

synchronizedキーワードは、単一のJVMで動作します。複数のWebノードがある場合、これは複数のJVMにわたるマルチスレッドアクセスを妨げません。

あなたがする必要があるのは、次のように、DBに変更を適用するLockModeType.PESSIMISTIC_READかLockModeType.PESSIMISTIC_WRITE使用することです：

Session session = null;
EntityTransaction tx = null;

try {
    session = sessionFactory.openSession();

    tx = session.getTransaction();
    tx.begin();

    Post post = session.find(
        Post.class, 
        id, 
        LockModeType.LockModeType.PESSIMISTIC_READ
    );

    post.setTitle("High-Performance Java Perisstence");

    tx.commit();
} catch(Exception e) {
    LOGGER.error("Post entity could not be changed", e);
    if(tx != null) {
        tx.rollback(); 
    }
} finally {
    if(session != null) {
        session.close(); 
    }
}

だから、これは私がやったことです：

• EntityTransaction新しいデータベーストランザクションを作成して開始しました
• Post関連するデータベースレコードをロックしている間にエンティティをロードしました
• Postエンティティを変更してトランザクションをコミットしました
• Exceptionスローされた場合、トランザクションをロールバックしました

ACIDとデータベーストランザクションの詳細については、こちらの記事もご覧ください。