Sadece bilmek istiyorum, returnbir usingbloğun içinden arama güvenli / iyi bir yaklaşım mı
Örn.
using(var scope = new TransactionScope())
{
// my core logic
return true; // if condition met else
return false;
scope.Complete();
}
En sondaki küme ayracının dispose()iptal edileceğini biliyoruz . Fakat yukarıdaki durumda ne olacak, çünkü returnkontrolü verilen kapsamın (AFAIK) dışına atıyor ...
scope.Complete()?dispose()yöntemi için.scope.Complete()Yani bunu siz işlem olacak hep geri alma, çağrı sağladığınız örnek ile vurmak asla.
using'ler dispose()döndüğünüzde denir, bu içeren fonksiyon usingbloğu döndü ve kendisine ait her şeyi yetim edilecektir. Dolayısıyla, scope"tarafından using" elden çıkarılmamış olsa bile (diğerlerinin açıkladığı gibi olacaktır), işlev sona erdiği için yine de elden çıkarılacaktır. C # gotoifadesi olsaydı - gülmeyi bitirdin mi? iyi- o zaman dönüş yerine gotokapanış ayracından sonra geri dönmeden yapabilirsiniz. Mantıksal olarak, scopeyine de atılır, ancak gotoC # koydunuz , bu aşamada mantığı kimin umursadığı.
Yanıtlar:
Kullanım bloğu sadece bir blok olduğu returniçin usingbloğunuzun içinde aramak tamamen güvenlidir try/finally.
Yukarıdaki örneğinizde iade sonrası true, kapsam elden çıkarılacak ve değer iade edilecektir. return falseVe scope.Complete()olacak değil denilen olsun. DisposeAncak, nihayet bloğunun içinde yer aldığından bağımsız olarak çağrılacaktır.
Kodunuz esasen bununla aynıdır (eğer bu anlaşılmasını kolaylaştırırsa):
var scope = new TransactionScope())
try
{
// my core logic
return true; // if condition met else
return false;
scope.Complete();
}
finally
{
if( scope != null)
((IDisposable)scope).Dispose();
}
İşlemi gerçekleştirmenin bir yolu olmadığından işleminizin asla taahhüt edilmeyeceğini lütfen unutmayın scope.Complete().
Dispose olacak denilen olsun. OP ne olduğunu bilmiyorsa using, büyük ihtimalle ne olacağını bilmiyordur finally.
using, örneğin using (var callersVar = MyFunc(..)) .., yerine akışı verildi ve yoluyla kapatılması için sorumlu olan bir arayan ortalama - kullanarak iç "MyFunc" sahip usingya da açık bir şekilde, ya da (b) MyFunc'ın , güvenli bir şekilde geri aktarılabilecek diğer nesnelere ihtiyaç duyulan her türlü bilgiyi çıkarmasını sağlayın - daha sonra temeldeki veri nesneleri veya akış sizin tarafınızdan atılabilir using. Sızdıran kod yazmak zorunda kalmamalısınız.
Sorun değil - finallycümlecikler ( usingcümlenin kapanış küme parantezinin başlık altında yaptığı şeydir ), kapsam bırakıldığında, nasıl olursa olsun her zaman yürütülür.
Bununla birlikte, bu yalnızca nihayet bloğunda (kullanılırken açıkça ayarlanamaz using) ifadeler için geçerlidir . Bu nedenle, örneğinizde scope.Complete()asla çağrılmaz (derleyicinin sizi erişilemez kod hakkında uyarmasını bekliyorum).
Genel olarak iyi bir yaklaşımdır. Ancak sizin durumunuzda, aramadan önce dönerseniz scope.Complete(), TransactionScope'u çöpe atar. Tasarımınıza bağlıdır.
Dolayısıyla, bu örnekte Complete () çağrılmaz ve kapsam, IDisposable arabirimi miras aldığını varsayarak elden çıkarılır.
kapsam.Complet kesinlikle önceden çağrılmalıdır return. Derleyici bir uyarı gösterecek ve bu kod asla çağrılmayacaktır.
returnKendisine gelince - evet, onu usingifadenin içinde aramak güvenlidir . Kullanmak, sahnenin arkasındaki blok denemeye çevrilir ve son olarak blok kesinlikle yürütülür.
Sağladığınız örnekte bir sorun var; scope.Complete()asla aranmaz. İkinci olarak, returnifadelerin içinde usingifadelerin kullanılması iyi bir uygulama değildir . Aşağıdakilere bakın:
using(var scope = new TransactionScope())
{
//have some logic here
return scope;
}
Bu basit örnekte nokta şudur; scopeifade kullanımı bittiğinde değeri boş olacaktır.
Bu nedenle, ifadeleri kullanarak içeri dönmemek daha iyidir.
scopeboş olmayacak - yani olmuş tek şey Dispose()o örneğinde çağrılan olacaktır ve bu nedenle örnek olmalıdır artık kullanılamaz (ama boş değil ve denemek ve kullanmak engelleyen bir şey yok atılan nesne, bu gerçekten tek kullanımlık bir nesnenin uygunsuz bir kullanımı olmasına rağmen).
return scopebir başvuru döndürürsünüz . Bu şekilde, iade sırasında bu referansı atarsanız, GC'nin atılan nesneyi temizlemesini engellersiniz.
Çağrılacağından emin olmak için scope.Complete()sarın try/finally. disposeEğer ile sarın çünkü denir usingbir alternatif olduğunu try/finallybloğu.
using(var scope = new TransactionScope())
{
try
{
// my core logic
return true; // if condition met else
return false;
}
finally
{
scope.Complete();
}
}
using{}kapsamı bitti, ilgili nesneler, bertaraf olsunreturnwill "mola" kapsam - nesneleri beklendiği gibi tanzim alacak böylece