Bir koleksiyon isteği sıfır öğeye sahip olduğunda beklenen davranış

Diyelim ki size aşağıdakiler veriliyor ...

List<Thing> theThings = fubar.Things.All();

Döndürülecek bir şey olmasaydı, fubar.Things.All () öğesinin geri dönmesini beklersiniz?

Düzenleme: Görüşleriniz için teşekkürler. Biraz bekleyeceğim ve girişleri en fazla yukarı ile kabul edeceğim.

Şimdiye kadar, özellikle boş bir koleksiyon öneren cevaplara katılıyorum. Bir satıcı, yukarıdaki örneğe benzer birkaç çağrı içeren bir API sağladı. Geçen yıl API'ları aracılığıyla 4.6 milyon dolar gelir elde eden bir satıcı BTW. Temelde aynı fikirde olmadığım bir şey yapıyorlar - bir istisna atıyorlar.

İki olasılıktan (yani bir geri dönen nullveya boş bir koleksiyon dönen) boş bir koleksiyon dönen seçer, çünkü arayanın döndürülen değerin kontrolünü atlamasını sağlar. Bunu yazmak yerine

List<Thing> theThings = fubar.Things.All();
if (theThings != null) {
    for (Thing t : theThings) {
        t.doSomething();
    }
}

bunu yazabileceklerdi:

List<Thing> theThings = fubar.Things.All();
for (Thing t : theThings) {
    t.doSomething();
}

Bu ikinci kod parçası daha kısadır ve okunması daha kolaydır, çünkü iç içe geçme seviyesi bir azalır.

Boş bir liste beklerdim. theThingsHala bir nesne olurdu, ama theThings.Countya theThings.size()dönecekti 0.

Bunun gibi tasarım sorunları Boş Nesne kalıbıyla giderilir

... Bir nesnenin (örneğin, var olmayan bir müşteri) yokluğunu iletmek için null referans kullanmak yerine, beklenen arabirimi uygulayan, ancak yöntem gövdesi boş olan bir nesne kullanılır. Varsayılan bir uygulama üzerinde bu yaklaşımın avantajı, bir Boş Nesnenin çok öngörülebilir olması ve hiçbir yan etkisi olmamasıdır: hiçbir şey yapmaz.

Örneğin, bir işlev bir dizindeki dosyaların listesini alabilir ve her biri üzerinde bazı eylemler gerçekleştirebilir. Boş bir dizin olması durumunda, yanıtlardan biri liste yerine istisna atmak veya boş bir başvuru döndürmek olabilir. Bu nedenle, bir listeyi bekleyen kod, devam etmeden önce bir listeye sahip olduğunu doğrulamalıdır, bu da tasarımı karmaşıklaştırabilir ...

Özellikle davanızda geçerli olan öneri ( Listhiç olmadığında geri dönüş Thing):

... Bunun yerine null bir nesne (yani boş bir liste ) döndürerek , dönüş değerinin aslında bir liste olduğunu doğrulamaya gerek yoktur. Çağıran işlev, listeyi normal olarak yineleyerek etkin bir şekilde hiçbir şey yapmaz. Bununla birlikte, dönüş değerinin boş bir nesne (örneğin boş bir liste) olup olmadığını kontrol etmek ve istenirse farklı tepki vermek hala mümkündür.

IMHO, bir EMPTY değeri döndürmelisiniz. C # hakkında bilmiyorum, ama Java'da bu var:

  List list = Collections.EMPTY_LIST;
  Set set = Collections.EMPTY_SET;
  Map map = Collections.EMPTY_MAP;

  // For the type-safe 
  List<String> s = Collections.emptyList();
  Set<Long> l = Collections.emptySet();
  Map<Date> d = Collections.emptyMap();

http://docs.oracle.com/javase/1.4.2/docs/api/java/util/Collections.html

Bu şekilde arama kodunda boş bir doğrulama yazmaktan kaçınabileceğiniz için boş bir değer döndürerek boş bir değer döndürürdüm.

İki çözüm farklı şeyler ifade ediyor.

Eğer iade ettiğiniz şeyin sadece sıfırı varsa, DAİMA boş bir koleksiyon iade edersiniz! Bir dizin listesinin durumunu ele alın. Dizinde dosya yoksa, boş bir dosya koleksiyonu döndürürsünüz.

Öte yandan, dizin yoksa, bu gerçekten uygun değildir. "Hiçbir şey geri dönemem" tamamen farklı bir şey demektir. Bu durumda, duruma bağlı olarak null döndürmeli veya istisna atamalısınız, hiçbir şey yanlış değil gibi boş bir koleksiyon döndürmeyin.