Silinecek öğe belirtilmediğinde hizmet istisna veya geri dönmelidir

Ben temsil edilebilir kod parçası var:

public class ItemService {

    public void DeleteItems(IEnumerable<Item> items)
    {
        // Save us from possible NullReferenceException below.
        if(items == null)
            return;

        foreach(var item in items)
        {
            // For the purpose of this example, lets say I have to iterate over them.
            // Go to database and delete them.
        }
    }
}

Şimdi bunun doğru bir yaklaşım mı yoksa istisna mı atmam gerektiğini merak ediyorum. Ben istisna önleyebilirsiniz, çünkü dönen boş bir koleksiyon üzerinde yineleme ile aynı olacaktır, yani, önemli bir kod zaten yürütülmez, ancak diğer yandan muhtemelen kodun bir yerinde sorunları saklıyorum, çünkü neden kimse aramak istiyorsun DeleteItemsile nullparametre? Bu, kodda başka bir yerde bir sorun olduğunu gösterebilir.

Bu çoğu zaman bir şey yapmak ve bir sonuç döndürmez, çünkü genellikle hizmet yöntemlerinde bir sorun var, bu yüzden birisi geçersiz bilgi geçerse hizmetin yapacak bir şey yok, bu yüzden döner.

Bunlar iki farklı soru.

Kabul etmeli misin null? Bu null, kod tabanındaki genel politikanıza bağlıdır . Kanımca, nullaçıkça belgelenen yerler dışında her yeri yasaklamak çok iyi bir uygulamadır, ancak kod tabanınızın zaten sahip olduğu sözleşmeye bağlı kalmak daha iyi bir uygulamadır.

Boş koleksiyonu kabul etmeli misiniz? Bence: EVET, kesinlikle. Tüm arayanları boş olmayan koleksiyonlarla sınırlamak, matematiksel olarak doğru olanı yapmaktan çok daha fazla çaba sarf ediyor - sıfır kavramına meraklı bazı geliştiricileri şaşırtsa bile.

Boş değer

@KilianFoth'un söylediği gibi, genel politikanıza sadık kalın. Eğer nullboş bir liste için "steno" olarak değerlendirmek gerekiyorsa, bu şekilde yapın.

nullDeğerler konusunda tutarlı bir politikanız yoksa aşağıdakileri öneriyorum:

null"normal" tipiyle ifade edilemeyen durumları temsil etmek için ayrılmalıdır, örneğin null"Bilmiyorum" ifadesini kullanmak için. Ve bu iyi bir seçim, çünkü bu değeri dikkatsizce kullanmaya çalışan herkes bir istisna alacak, bu doğru olanı.

nullBoş bir liste için kısayol olarak kullanmak , mükemmel bir temsil olduğu için sıfır öğeli bir liste olduğu için bu şekilde nitelendirilmez. Ve teknik olarak kötü bir seçimdir, çünkü kodunuzun listelerle ilgili her bölümünü geçerli stenoyu kontrol etmeye zorlar null.

Boş liste

Bir DeleteItems()yöntem için, boş bir listeyi etkin bir şekilde geçirmek hiçbir şey yapmamak anlamına gelir. Buna bir argüman olarak izin verdim, bir istisna atmak değil, sadece çabucak geri dönüyorum.

Tabii ki, arayan önce sıfır öğeleri kontrol DeleteItems()edebilir ve bu durumda aramayı atlayabilir . Bir web API'sinden bahsediyorsak, verimlilik nedenleriyle arayan , gereksiz trafik ve gidiş-dönüş gecikmelerinden kaçınmak için bunu yapmalıdır . Ancak API'nızın bunu zorlaması gerektiğini düşünmüyorum.

Bir istisna atın ve arama kodunda null değerlerini kullanın.

Bir tasarım kuralı olarak parametre değerleri olarak null değerinden kaçınmaya çalışın. NullPointerExceptions'ı genel olarak azaltacaktır, çünkü null'ler gerçekten bir istisna olacaktır.

Bunun yanı sıra, kodunuzun geri kalanına bakın. Bu, projenizde ortak bir kalıpsa tutarlı olun.

Genel olarak konuşursak, istisnalar atma istisnai durumlar için ayrılmalıdır, yani kodun mevcut bağlamda gerçekleştirilecek makul bir eylem tarzı yoksa.

Bu düşünce sürecini bu duruma uygulayabilirsiniz. Bunun herkese açık bir yöntemi olan genel bir sınıf olduğu göz önüne alındığında, genel bir API'niz var ve teoride neyin geçtiği üzerinde hiçbir kontrol yok.

Kodunuzda neyi çağırdığı hakkında bir bağlam yoktur (olması gerektiği gibi) ve null değeriyle makul bir şekilde yapabileceğiniz hiçbir şey yoktur. Bu kesinlikle bir ArgumentNullException.

Bu soru istisnalar değil, nullgeçerli bir argüman olmakla ilgili .

Yani, her şeyden önce null, bu yöntemin argümanı için izin verilen bir değer olup olmadığına karar vermelisiniz . Öyleyse, bir istisnaya ihtiyacınız yoktur. Değilse, bir istisnaya ihtiyacınız vardır.

İzin vermek isteyip istemediğiniz null , bu konuda çok sayıda Google isabeti ile gösterildiği gibi tartışmalıdır. Yani, net bir cevap alamayacaksınız ve çalıştığınız yerde biraz fikir ve geleneklere kalmış.

Başka bir çekişme alanı, bir kütüphane işlevinin hatalı parametreleri "düzeltmeye" çalışmadığı sürece, tür şeyler hakkında gerçekten katı mümkün olduğunca yumuşak . Bunu ağ protokolleri, posta transferi vb. İle karşılaştırın (bunlar programlama yöntemlerinde olduğu gibi arayüz sözleşmeleri). Genellikle, politika, gönderenin protokole olabildiğince sıkı bir şekilde uyması gerektiğidir, ancak alıcı, gelen her şeyle çalışabilmek için kendi yolundan gitmelidir.

Yani karar vermelisiniz: nullElleçleme gibi oldukça büyük bir politika uygulamak gerçekten bir kütüphane yönteminin işi midir? Özellikle kütüphaneniz başka kişiler tarafından da kullanılıyorsa (farklı politikaları olabilir).

Muhtemelen nulldeğerlere izin verme , semantiklerini tanımlama ve belgeleme tarafında hata yapardım (yani, null = empty arraybu durumda) ve diğer benzer kodunuzun ("kütüphane" stil kodu)% 99'u bunu yapmazsa , bir istisna atmazdım 'yuvarlak.