Question

使用文章列表，当显示一篇文章时，我还会显示下一篇和上一篇文章，我使用下面的代码。我正在寻找一种方法来使Linq的代码更精简？

var article = allArticles.Where(x => x.UrlSlug == slug).FirstOrDefault();
int currentIndex = allArticles.IndexOf(article);

        if (currentIndex + 1 > allArticles.Count-1)
            article.Next = allArticles.ElementAt(0);
        else
            article.Next = allArticles.ElementAt(currentIndex + 1);

        if (currentIndex - 1 >= 0)
            article.Previous = allArticles.ElementAt(currentIndex - 1);
        else
            article.Previous = allArticles.Last();
return article;

Answer 1

我不认为LINQ提供“下一个或第一个”操作。不妨使用模数：

article.Next = allArticles[(currentIndex + 1) % allArticles.Count];
article.Previous = allArticles[(currentIndex + allArticles.Count - 1) % allArticles.Count];

（第二行中的+ allArticles.Count用于纠正%在应用于负数时的数学错误行为。）

Answer 2

完全同意Aasmund Eldhuset的回答。

请确保您没有获得空例外：

var article = allArticles.FirstOrDefault(x => x.UrlSlug == slug);
var currentIndex = allArticles.IndexOf(article);

if (article == null) return;
article.Next = allArticles[(currentIndex + 1) % allArticles.Count];
article.Previous = allArticles[(currentIndex + allArticles.Count - 1) % allArticles.Count];

当List索引超出范围时，Linq获取第一个或最后一个元素

2 个答案: