Bagaimana cara kerja metode statis tersinkronisasi di Java dan dapatkah saya menggunakannya untuk memuat entitas Hibernate?

Jika saya memiliki kelas util dengan metode statis yang akan memanggil fungsi Hibernate untuk mencapai akses data dasar. Saya bertanya-tanya apakah membuat metode synchronizedini merupakan pendekatan yang tepat untuk memastikan keamanan benang.

Saya ingin ini mencegah akses info ke instance DB yang sama. Namun, saya sekarang yakin jika kode berikut mencegah getObjectByIddipanggil untuk semua Kelas ketika dipanggil oleh kelas tertentu.

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
}

Dengan menggunakan sinkronisasi pada metode kunci statis Anda akan menyinkronkan metode dan atribut kelas (sebagai lawan dari metode dan atribut instan)

Jadi anggapan Anda benar.

Saya bertanya-tanya apakah membuat metode ini disinkronkan adalah pendekatan yang tepat untuk memastikan keamanan thread.

Tidak juga. Anda seharusnya membiarkan RDBMS Anda melakukan itu sebagai gantinya. Mereka bagus dalam hal-hal semacam ini.

Satu-satunya hal yang akan Anda dapatkan dengan menyinkronkan akses ke database adalah membuat aplikasi Anda sangat lambat. Lebih jauh lagi, dalam kode yang Anda posting Anda sedang membangun Pabrik Sesi setiap kali, dengan cara itu, aplikasi Anda akan menghabiskan lebih banyak waktu mengakses DB daripada melakukan pekerjaan yang sebenarnya.

Bayangkan skenario berikut:

Klien A dan B berupaya memasukkan informasi yang berbeda ke dalam catatan X dari tabel T.

Dengan pendekatan Anda, satu-satunya hal yang Anda dapatkan adalah memastikan satu dipanggil setelah yang lain, ketika hal ini akan tetap terjadi dalam DB, karena RDBMS akan mencegah mereka memasukkan setengah informasi dari A dan setengah dari B pada saat yang sama. . Hasilnya akan sama tetapi hanya 5 kali (atau lebih) lebih lambat.

Mungkin lebih baik untuk melihat bab "Transaksi dan Konkurensi" dalam dokumentasi Hibernate. Sebagian besar masalah yang Anda coba selesaikan, sudah dipecahkan dan cara yang jauh lebih baik.

Untuk menjawab pertanyaan secara lebih umum ...

Perlu diingat bahwa menggunakan metode yang disinkronkan benar-benar hanya singkatan (anggap kelas adalah SomeClass):

synchronized static void foo() {
    ...
}

sama dengan

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

dan

synchronized void foo() {
    ...
}

sama dengan

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

Anda dapat menggunakan objek apa pun sebagai kunci. Jika Anda ingin mengunci himpunan bagian dari metode statis, Anda bisa

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) {...}
    }
}

(untuk metode non-statis, Anda ingin menjadikan kunci sebagai bidang non-statis)

Metode statis menggunakan kelas sebagai objek untuk mengunci, yaitu Utils.class untuk contoh Anda. Jadi ya, tidak apa-apa.

static synchronizedberarti memegang kunci pada objek kelas di Classmana sebagai synchronizedberarti memegang kunci pada objek kelas itu sendiri. Itu berarti, jika Anda mengakses metode tersinkronisasi non-statis di utas (eksekusi) Anda masih dapat mengakses metode tersinkronisasi statis menggunakan utas lain.

Jadi, mengakses dua jenis metode yang sama (baik dua metode statis atau dua metode non-statis) pada suatu titik waktu oleh lebih dari utas tidak mungkin.

Mengapa Anda ingin menegakkan bahwa hanya satu utas yang dapat mengakses DB pada suatu waktu?

Merupakan tugas driver database untuk mengimplementasikan penguncian yang diperlukan, dengan asumsi a Connectionhanya digunakan oleh satu utas pada satu waktu!

Kemungkinan besar, basis data Anda mampu menangani beberapa akses paralel

Jika ada hubungannya dengan data dalam database Anda, mengapa tidak menggunakan penguncian isolasi database untuk mencapai?

Untuk menjawab pertanyaan Anda, ya itu: synchronizedmetode Anda tidak dapat dijalankan oleh lebih dari satu utas sekaligus.

Bagaimana synchronized kerja kata kunci Java

Ketika Anda menambahkan synchronized kata kunci ke metode statis, metode ini hanya dapat dipanggil dengan utas tunggal setiap kali.

Dalam kasus Anda, setiap pemanggilan metode akan:

• buat yang baru SessionFactory
• buat yang baru Session
• ambil entitasnya
• mengembalikan entitas kembali ke pemanggil

Namun, ini adalah persyaratan Anda:

• Saya ingin ini mencegah akses ke info ke instance DB yang sama.
• mencegah getObjectByIddipanggil untuk semua kelas ketika dipanggil oleh kelas tertentu

Jadi, bahkan jika getObjectByIdmetodenya aman, implementasinya salah.

SessionFactory praktik terbaik

Ini SessionFactoryadalah thread-safe, dan ini adalah objek yang sangat mahal untuk dibuat karena perlu mengurai kelas entitas dan membangun representasi metamodel entitas internal.

Jadi, Anda tidak harus membuat SessionFactorypada setiap getObjectByIdpemanggilan metode.

Sebagai gantinya, Anda harus membuat contoh tunggal untuk itu.

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

The Sessionharus selalu tertutup

Anda tidak menutup Sessiondalam finallyblok, dan ini bisa membocorkan sumber daya basis data jika pengecualian dilemparkan saat memuat entitas.

Menurut Session.loadmetode JavaDoc mungkin membuang HibernateExceptionjika entitas tidak dapat ditemukan dalam database.

Anda sebaiknya tidak menggunakan metode ini untuk menentukan apakah ada instance (gunakan get()saja). Gunakan ini hanya untuk mengambil contoh yang Anda anggap ada, di mana ketidakberadaan akan menjadi kesalahan aktual.

Itu sebabnya Anda perlu menggunakan finallyblok untuk menutup Session, seperti ini:

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(); 
         }
     }
 }

Mencegah akses multi-utas

Dalam kasus Anda, Anda ingin memastikan hanya satu utas yang mendapatkan akses ke entitas tertentu.

Tetapi synchronizedkata kunci hanya mencegah dua utas memanggil getObjectByIdsecara bersamaan. Jika kedua utas memanggil metode ini satu demi satu, Anda masih akan memiliki dua utas menggunakan entitas ini.

Jadi, jika Anda ingin mengunci objek database tertentu sehingga tidak ada utas lain yang dapat memodifikasinya, maka Anda perlu menggunakan kunci basis data.

Kata synchronizedkunci hanya berfungsi dalam JVM tunggal. Jika Anda memiliki beberapa web node, ini tidak akan mencegah akses multi-thread di beberapa JVM.

Yang perlu Anda lakukan adalah menggunakan LockModeType.PESSIMISTIC_READatauLockModeType.PESSIMISTIC_WRITE saat menerapkan perubahan pada DB, seperti ini:

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(); 
    }
}

Jadi, inilah yang saya lakukan:

• Saya membuat yang baru EntityTransactiondan memulai transaksi basis data baru
• Saya memuat Postentitas sambil memegang kunci pada catatan database terkait
• Saya mengubah Postentitas dan melakukan transaksi
• Dalam kasus Exceptionpelemparan, saya memutar kembali transaksi

Untuk detail lebih lanjut tentang ACID dan transaksi basis data, lihat artikel ini juga.