通过使用产量或改变算法来优化代码

Question

下面的代码可以工作，但我想使用产量或通过更改算法来优化代码。

public IEnumerable<Book> GetAuthorWithBookName(string keyword)
{
    var bookAndAuthorList = new List<Book>();
    List<Author> AuthorNameList = getAuthorName(keyword);
    foreach (var author in AuthorNameList)
    {
        XmlNode booksNames = getBook(author);
        XDocument XDOCbooksNames = XDocument.Parse(booksNames.OuterXml);

        var bookNameList = (
            from x1 in XDOCbooksNames.Descendants("Books")
            select x1.Elements("book").Select(g => g.Attribute("name").Value))
            .ToList();
        foreach (var bookName in bookNameList)
        {
            bookAndAuthorList.Add(new Book()
            {
                authorName = author.authorName,
                bookName = bookName
            });
        }
    }
    return bookAndAuthorList;
}

public class Book
{
    public string authorName { get; set; }
    public string bookName { get; set; }
}

Answer 1

鲁本斯和卢克的回应正确解释了产量的使用。

然而，这对我来说看起来很可疑。

XmlNode booksNames = getBook(author);
XDocument XDOCbooksNames = XDocument.Parse(booksNames.OuterXml);

您将 XML 转换为字符串，然后再次解析它，只是因为您想将其从 DOM 节点转换为 Xml.Linq 节点。如果你谈论的是优化，那么这比创建额外的列表效率低得多。

Answer 2

为了快速获胜，您可以删除 .ToList() 调用。您所做的只是枚举项目，因此无需这样做。同样，无需创建 bookAndAutherList。

最终我认为你可以将其简化为：

public IEnumerable<Book> GetAuthorWithBookName(string keyword)
{
    return from author in getAuthorName(keyword)
           let book = getBook(author)
           from xmlBook in XDocument.Parse(book.OuterXml).Descendants("Books")
           select new Book
           {
               authorName = author.AuthorName,
               bookName = xmlBook.Attribute("name").Value
           };
}

Answer 3

不确定你会得到很多“优化”。当您经常在使用正在生成的枚举时中断部分时，最好使用yield，这样您就不会进行不必要的计算。但这里有一句话：

这是未经测试的。

public IEnumerable<Book> GetAuthorWithBookName(string keyword)
{
    foreach (var author in getAuthorName(keyword))
    {
        XDocument XDOCbooksNames = XDocument.Parse(getBook(author).OuterXml);

        var bookNameList = from x1 in XDOCbooksNames.Descendants("Books")
                            select x1.Elements("book").Select(g => g.Attribute("name").Value);

        foreach (var bookName in bookNameList)
        {
            yield return new Book()
                {
                    authorName = author.authorName,
                    bookName = bookName
                };
        }
    }
}

我做了什么：

1. 删除 ToList()
   表达式，它已经返回一个
   可数。删除了一些代码
   将 getAuthorName 合并到
   foreach（注意 - 如果可以的话，请确保该函数也能产生并且可枚举）
2. 产生返回而不是添加到列表中

Answer 4

试试这个：

foreach (var bookName in bookNameList)
{
    yield return new Book()
    {
        authorName = author.authorName,
        bookName = bookName
    };
}

并删除其他 return 语句。