搜索集合[关闭]

Question

1. 下面的代码 sn-p 应该通过文档标题通过通用排序例程进行搜索，我不确定它是否这样做。有什么帮助吗？

public Document searchByTitle (String aTitle)
{
  foreach(Document doc in this)
  {
    if (doc.Title == aTitle)
    {
      return doc;
    }
    else
    {
      return null;
    }
  }
}

2.这段代码 sn-p 应该显示一个方法返回一个集合中存储的书籍数量。这有意义吗？

public static int NoofBooks()
{
  int count = 0;
  foreach(Document doc in this)
  {
   if (doc.Type == "Book")
   {
     count++;
   }
   return count;    
  }
}

Answer 1

不，您的代码不正确，应该是：

public Document searchByTitle (String aTitle)
{
  foreach(Document doc in this)
  {
    if (doc.Title == aTitle)
    {
      return doc; // if we have found the required doc we return it
    }
  }

  // we've scanned the entire collection and haven't find anything
  // we return null in such a case
  return null;
}

请注意，只有在检查了整个集合之后，您才应该return null;。

我们经常使用Linq 来查询集合，例如

using System.Linq;

...

public Document searchByTitle (String aTitle) 
{
  return this.FirstOrDefault(doc => doc.Title == aTitle);
}

编辑：与第二个 sn-p 完全相同的问题（过早 return）：

 public static int NoofBooks()
 {
   int count = 0;

   // scan the entire collection...
   foreach(Document doc in this)
   {
     if (doc.Type == "Book")
     {
       count++;
     }
   } 

   // ..and only then return the answer
   return count; // <- return should be here   
 }

或者再一次，把它写成 Linq

 public static int NoofBooks()
 {
   return this.Count(doc => doc.Type == "Book");
 }