Bir deneme yakalama bloğu içinden dönmek kötü bir uygulama mı?

Bu yüzden bu sabah şuna benzeyen bir kodla karşılaştım:

try
{
    x = SomeThingDangerous();
    return x;
}
catch (Exception ex)
{
    throw new DangerousException(ex);
}
finally
{
    CleanUpDangerousStuff();
}

Şimdi bu kod iyi bir şekilde derleniyor ve olması gerektiği gibi çalışıyor, ancak bir try bloğunun içinden dönmek doğru gelmiyor, özellikle de bir sonuç varsa.

Benim asıl sorunum, sonunda kendi başına bir istisna atarsa ​​ne olacağı? Döndürülen bir değişkeniniz var ama aynı zamanda başa çıkmanız gereken bir istisna var ... Bu yüzden başkalarının bir try bloğundan dönme hakkında ne düşündüğünü bilmek isterim?

Hayır, kötü bir uygulama değil. returnMantıklı olduğu yere koymak , okunabilirliği ve sürdürülebilirliği artırır ve kodunuzun anlaşılmasını kolaylaştırır. finallyBir returnifade ile karşılaşılırsa bloğun çalıştırılacağına dikkat etmemelisiniz .

Nihayet ne olursa olsun idam edilecek, bu yüzden önemli değil.

Şahsen, bu tür bir kodlamadan kaçınırım çünkü son ifadelerden önce dönüş ifadeleri görmek istemiyorum.

Zihnim basit ve işleri oldukça doğrusal olarak işliyor. Bu nedenle, kuru çalıştırma kodunu gözden geçirdiğimde, return ifadesine ulaşabildiğimde, her şeyin takip edileceğini düşünme eğiliminde olacağım, bu durumda bu açıkça oldukça yanlıştır (bu, return ifadesini etkileyeceğinden değil ama yan etkiler neler olabilir).

Böylece, dönüş ifadesinin her zaman final ifadelerinden sonra görünmesi için kodu düzenlerdim.

Bu sorunuzu cevaplayabilir

Bir denemede gerçekten ne olur {return x; } sonunda {x = null; } Beyan?

Bu soruyu okuduktan sonra, bir istisna oluşturabileceğini düşünüyorsanız, nihayet ifadesinde başka bir try catch yapısına sahip olabilirsiniz. Derleyici değeri ne zaman döndüreceğini bulacaktır.

Bununla birlikte, kodunuzu yine de yeniden yapılandırmanız daha iyi olabilir, böylece daha sonra sizi veya bundan haberi olmayan başka birini karıştırmasın.

İşlevsel olarak hiçbir fark yoktur.

Ancak bunu yapmamak için bir neden var. Birkaç çıkış noktası olan daha uzun yöntemleri okumak ve analiz etmek genellikle daha zordur. Ancak bu itirazın, catch ve nihayet bloklardan çok return ifadeleriyle ilgisi vardır.

Örneğinizde her iki şekilde de eşdeğerdir, derleyici aynı kodu oluştursa bile şaşırmam. Nihayet bloğunda bir istisna olursa, return ifadesini bloğa veya dışına koysanız da aynı sorunları yaşarsınız.

Asıl soru, biçimsel olarak hangisinin en iyisi olduğudur. Yöntemlerimi tek bir return ifadesi olacak şekilde yazmayı seviyorum, bu şekilde yöntemden çıkan akışı görmek daha kolay oluyor, bunun sonucu olarak da return ifadesini en sona koymaktan hoşlanıyorum, bu yüzden olduğunu görmek kolay yöntemin sonu ve geri döndüğü şey bu.

Bence son ifade olarak çok düzgün bir şekilde yerleştirilmiş dönüş ifadesi ile, diğerlerinin gelip birden çok iade ifadesini yöntemin diğer bölümlerine serpiştirme olasılığı daha düşüktür.