Оптимизация кода с помощью алгоритма доходности или изменения

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

Как быстро победить, может удалить вызов .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 2

Попробуйте это:

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

и удалить другие return заявление.

Answer 3

Не уверен, что вы получите большую «оптимизацию». Доходность лучше использовать, когда вы часто ломаете часть пути, используя нумерацию, которую вы генерируете, чтобы вы не делали ненужных вычислений. Но здесь идет:

Это непроверено.

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 в Еогеасп (примечание - убедитесь, что функция дает и IEnumerable тоже, если вы можете)
2. Выход возврата вместо добавления в список

Answer 4

Ответы Рубенса и Луки правильно объяснить использование урожая.

Однако это выглядит подозрительно для меня.

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

Вы преобразовать XML в строку, а затем разобрать его снова, только потому, что вы хотите, чтобы преобразовать его из DOM узла к узлу Xml.Linq. Если вы говорите об оптимизации, то это гораздо более неэффективно, чем создание дополнительного списка.