Veritabanı bağlantısı - parametre olarak iletilmeli mi?

Ortak bir yöntem kullanarak veritabanı bağlantısının bir kez alındığı ve kullanılacak ilgili sınıf boyunca geçirildiği bir sistemimiz var. Veritabanı sınıfını farklı sınıflara bir parametre olarak geçirirken sorun yaratabileceğinden şüphe var, bu yüzden bu gerçekten uygun olup olmadığını görmek için kontrol ediyorum ve bunu yapmak için daha iyi kalıplar var mı?

Kalıcılığı sağlayacak bazı ORM araçları olduğunu biliyorum ama henüz buna giremeyiz.

Herhangi bir geri bildirim bekliyoruz, teşekkürler.

Evet, bir bağlantının etrafından geçmek güvenlidir. Bağlantıyı bir dış kontrol bloğunda idare edersiniz. Bu konuda güvensiz bir şey yok.

Güvensiz olan, bağlantının zamanında uygun şekilde atandığını garanti etmeyen kod yazmaktır. Bir kaynağı temizlemeyi unutmak, onu geçirmeyle ilgisi yoktur. Asılı bir bağlantıyı herhangi bir yere iletmeden bırakan kodu kolayca yazabilirsiniz.

C ++ 'da, yığına ayırırsanız veya akıllı işaretçiler kullanırsanız RAII tarafından korunursunuz. C # 'da tüm tek kullanımlık nesnelerin (bağlantılar gibi) bir "using" bloğunda bildirilmesi zor bir kural oluşturur. Java'da try-finally mantığı ile temizleyin. Bunu sağlamak için tüm veri katmanı kodunda kod incelemeleri yapın.

En yaygın kullanım durumu, birçok permütasyonda birleştirilebilecek birkaç işleminiz olduğu zamandır. Ve bu permütasyonların her birinin bir atomik işlem olması gerekir (hepsi başarılı veya geri dönüş). işlemi (ve dolayısıyla ilgili bağlantıyı) tüm yöntemlere aktarmanız gerekir.

Atomik işlemler olarak çeşitli şekillerde birleştirilebilecek birçok foobar () eylemimiz olduğunu varsayalım.

//example in C#
//outer controlling block handles clean up via scoping with "using" blocks.
using (IDbConnection conn = getConn())
{
    conn.Open();
    using (IDbTransaction tran = conn.BeginTransaction())
    {
        try
        {//inner foobar actions just do their thing. They don't need to clean up.
            foobar1(tran);
            foobar2(tran);
            foobar3(tran);
            tran.Commit();
        }
        catch (Exception ex)
        { tran.Rollback(); }
    }
}//connection is returned to the pool automatically

BTW bağlantıları olabildiğince geç açmak isteyeceksiniz, mümkün olan en kısa sürede atın. Bağlantıları nesne üyeleri olarak ele alıyorsanız, gereksiz durum olarak tanıtırsanız ve bağlantıları gereğinden fazla açık bırakırsanız, takım arkadaşlarınız doğru olabilir. Ancak, bir bağlantıyı veya işlemi parametre olarak iletme eylemi doğası gereği yanlış değildir.

BTW. Dilinizin birinci sınıf işlevleri desteklemesine bağlı olarak foobar () eylemleri listesinde yer alabilirsiniz. Böylece bir işlev, eylemlerin tüm permütasyonlarını kaldırabilir. Her permütasyon için dış kontrol bloğunun çoğaltılmasının ortadan kaldırılması.

Bağımlılık Enjeksiyonundan sonra gibisin . Yani, havuzlanmış bağlantı bir kez oluşturulur ve gerektiğinde enjekte edilir. Kesinlikle bir yöntem parametresi aracılığıyla bağlantıya geçmek bağımlılık enjekte etmenin bir yoludur, ancak Guice, PicoContainer veya Spring gibi bir IoC kapsayıcısı bunu yapabileceğiniz başka (daha güvenli) bir yoldur.

DI'yi kullanmak, mantığı, temel iş mantığınızdan uzakta, bağlantının oluşturulması, açılması, kullanılması ve kapatılması etrafında düzgün bir şekilde sarabileceğiniz anlamına gelir.

Spring JDBC ve arkadaşları sizin için bu tür davranışları gerçekleştirmenin diğer örnekleridir

Veri şeyleri yerine veritabanı şeylerinin etrafından dolaşmak sorunlara yol açabilir. Bu kapsamda, ne zaman pratik olursa olsun, uygun veritabanı hijyenini garanti edemezse bir veritabanı şeyini geçmeyin.

Veritabanı şeyler etrafında geçen sorun özensiz olabilir olmasıdır. Birisi bir veritabanı bağlantısı etrafında geçen biri ile kod birden fazla hata gördüm, biri daha sonra bir sonuç kümesi kapmak ve yerel bir nesnede (sonuç kümesi, hala veritabanına bağlı) saklar ve sonra bir imleç bağlar önemli bir süre için veritabanı. Başka bir örnek, bir başkasına bir sonuç kümesi geçirdi (daha sonra saklandı) ve sonra sonuç kümesini geçen yöntem onu ​​(ve ifadeyi) kapattı ve diğer yöntemler artık olmayan sonuç kümesiyle çalışmaya çalıştığında hatalara yol açtı.

Bütün bunlar veritabanına, bağlantıya, ifadeye, sonuç kümesine ve yaşam döngülerine saygı duymamaktan kaynaklanıyor.

Bundan kaçınmak için, veritabanları ile daha güzel oynayan ve kapalı oldukları sınıflardan çıkmak için veritabanı şeylerine sahip olmayan mevcut desenler ve yapılar vardır.

Geçme Connectionetrafında örneklerini en durumda tek DAO uygulamaları onlarla ilgisi olmalı rağmen genellikle bir sorun değildir. Şimdi, probleminiz kullandıktan sonra bağlantıların kapatılmamasıyla ilgili, aslında düzeltilmesi kolaydır: Connectionnesnenin açıldığı aynı seviyede, yani aynı yöntemde kapatılması gerekir. Şahsen aşağıdaki kod desenini kullanıyorum:

final Connection cnx = dataSource.getConnection();
try {
    // Operations using the instance
} finally {
    cnx.close();
}

Bu şekilde, blok içinde bir istisna atılsa bile, tüm bağlantıların her zaman kapalı olduğundan emin olurum. Aslında, aynı örnek Statementve ResultSetörnekler için kullandığım sürece devam ediyorum ve her şey şimdiye kadar pürüzsüz yelken oldu.

Edit 2018-03-29: Kullanıcı1156544 tarafından aşağıdaki yorumlarda belirtildiği gibi, Java 7 ile başlayarak, kaynaklarla deneme yapısının kullanılması tercih edilmelidir. Bunu kullanarak, ilk cevabımda verdiğim kod deseni şöyle basitleştirilebilir:

try (final Connection cnx = dataSource.getConnection()) {
    // Operations using the instance
}

gerektiğinde alabileceğiniz bir singleton kullanmak yerine işleri bu şekilde yapma dengesi vardır. Geçmişte her iki şekilde de işler yaptım.

Genel olarak, veritabanı bağlantı yönetiminin sonuçlarını düşünmeniz gerekir ve bu, veritabanı sorgusu kullanımına dik olabilir veya olmayabilir. Örneğin, belirli bir uygulama örneği için bir db bağlantınız varsa ve kullanılmadığında kapatılırsa, bu dikeydir. Yönetimi tek bir sınıfa koyun ve onu aktarmayın. Bu, ihtiyacınız olan db bağlantısını yönetmenizi sağlar. Örneğin, her bağlantıda bir bağlantıyı kapatmak (ve bir sonraki çağrıda yeniden açmak) istiyorsanız, bunun tek birtonda yapılması daha kolaydır, çünkü bunun için API merkezi olabilir.

Öte yandan, belirli bir aramanın rasgele bir bağlantı kullanması gerekebileceği bir bağlantı havuzunu yönetmeniz gerektiğini varsayalım. Bu, örneğin birden çok sunucuda dağıtılmış işlemler yaparken olabilir. Bu durumda, db bağlantı nesnesini geçerek tekil nesnelerle çalışmanız genellikle çok daha iyidir. Bence bu genellikle daha nadir bir durumdur, ancak gerektiğinde bunu yapmakta yanlış bir şey yoktur.