Ne zaman ve neden boşluk kullanmalısınız (örneğin, bool / int yerine)

Bazen bir geliştiricinin işlev için kritik olmayan bir şey döndürmeyi seçtiği yöntemler ile karşılaşıyorum. Demek istediğim, koda bakarken, göründüğü kadar güzel bir şekilde çalışıyor voidve bir an düşündüğümden sonra, "Neden?" Bu size tanıdık geliyor mu?

Bazen bunun bir gibi bir şey dönmek daha iyidir çoğunlukla kabul ediyorum boolya intziyade sadece yapın void. Yine de, büyük resimde, artılar ve eksiler hakkında emin değilim.

Duruma göre, geri dönerek intarayanı yöntemden etkilenen satır veya nesne miktarından haberdar edebilirsiniz (örneğin, MSSQL'e kaydedilen 5 kayıt). "InsertSomething" gibi bir yöntem bir boolean döndürürse, truebaşarılı olursa geri dönmek için tasarlanmış yöntemi alabilirim false. Arayan kişi bu bilgi üzerinde hareket etmeyi seçmemeyi seçebilir.

Diğer yandan,

• Bir yöntem çağrısının daha az net bir amacına yol açabilir mi? Kötü kodlama çoğu zaman beni yöntem içeriğini iki kez kontrol etmeye zorlar. Bir şeyler dönerse, o yöntem bir tür olduğunu söyler sahip dönen sonuç bir şeyler yapmak.
• Başka bir sorun, eğer yöntem uygulaması sizin için bilinmiyorsa, geliştirici işlevsellik açısından kritik olmayan bir şeyi geri getirmeye karar vermiştir? Tabii ki yorum yapabilirsiniz.
• İşlem, yöntemin kapanış parantezinde sonlandırılabildiğinde, dönüş değerinin işlenmesi gerekir.
• Kaputun altında ne olur? Çağrılan yöntem false, atılmış bir hata nedeniyle mi oldu? Yoksa değerlendirilen sonuçtan dolayı yanlış mı döndü?

Bununla deneyimleriniz neler? Bu konuda nasıl hareket edersiniz?

Bir yöntemin başarısını veya başarısızlığını gösteren bir bool dönüş değeri olması durumunda, Tryçeşitli .NET yöntemlerinde kullanılan -prefix paradigmasını tercih ederim .

Örneğin void InsertRow(), aynı anahtarla bir satır zaten varsa, bir yöntem bir istisna atabilir. Sorun şu ki, arayanın InsertRow'u çağırmadan önce sırasının benzersiz olmasını sağladığını varsaymak mantıklı mı? Cevap hayır ise, o zaman bool TryInsertRow()satır zaten varsa yanlış olanı da veririm . Db bağlantı hataları gibi diğer durumlarda, TryInsertRow, db bağlantısını korumanın arayanın sorumluluğu olduğunu farz ederek hala bir istisna atabilir.

Bir "durum kodu" döndüren IMHO, C # gibi orta - üst düzey dillerde istisnalar yaygın hale gelmeden önceki tarihlerden kaynaklanır. Bugünlerde, beklenmeyen bir hata, yönteminizin başarılı olmasını engelliyorsa, bir istisna atmak daha iyidir. Bu şekilde hatanın farkedilmediğinden emindir ve arayan kişi bununla başa çıkabilir. Dolayısıyla, bu durumlarda, bir yöntemin voidbir boolean durum bayrağı veya bir tamsayı durum kodu yerine hiçbir şey (yani ) döndürmemesi gayet iyi . (Tabii ki, yöntemin hangi istisnaları ne zaman ve ne zaman verebileceğini belirtmek gerekir.)

Öte yandan, eğer katı anlamda bir işlevse, yani girdi parametrelerine dayanarak bazı hesaplamalar yaparsa ve bu hesaplamanın sonucunu döndürmesi bekleniyorsa, geri dönüş tipi açıktır.

Bir ikisi arasında bir gri alan var olabilir onun arayanlar için faydalı görüldüğü takdirde, yönteme bir miktar "ekstra" bilgisini dönmeye karar verirler. Örnekteki gibi, etkilenen satır sayısındaki örneğiniz gibi. Veya bir öğeyi ilişkisel bir koleksiyona koyma yönteminde, eğer varsa, bu anahtarla ilişkili önceki değeri döndürmek faydalı olabilir. Bu kullanımlar, bir API'nin (bilinen ve öngörülen) kullanım senaryolarını dikkatlice analiz ederek belirlenebilir.

Peter'ın cevabı istisnaları iyi kapsar. Buradaki diğer hususlar, Komut Sorgu Ayrımı ile ilgilidir.

CQS prensibi, bir yöntemin bir komut veya bir sorgu olması gerektiğini ve her ikisinin de olmaması gerektiğini söyler. Komutlar hiçbir zaman bir değer döndürmemeli, yalnızca durumu değiştirmeli ve sorgular yalnızca durumu değiştirmeli ve değiştirmemelidir. Bu, anlambilimi çok net tutar ve kodun daha okunaklı ve bakımsız olmasına yardımcı olur.

CQS ilkesini ihlal eden birkaç vaka var iyi bir fikir. Bunlar genellikle performans veya iplik güvenliği ile ilgilidir.

Yol ne olursa olsun, onunla yürüyeceksin. Gelecekte kullanmak için geri dönüş değerleri yoktur (örneğin, işlevler her zaman doğrudur) bu çok büyük bir çaba kaybıdır ve kodu okumayı daha açık hale getirmez. Bir dönüş değerine sahipseniz, her zaman kullanmalı ve kullanabilmeniz için en az 2 olası dönüş değerine sahip olmalısınız.

Eskiden pek çok geçersiz fonksiyon yazıyordum. Ama tüm yöntemi çatlak zincirleme yöntemine bindiğimden beri, bunu geçersiz kılmak yerine geri göndermeye başladım - birilerinin geri dönüşün avantajlarından faydalanmasına izin verebilirim çünkü boşluğa dayanamazsınız. Ve onunla hiçbir şey yapmak istemiyorlarsa, bunu görmezden gelebilirler.