LINQクエリの結果タイプではサポートされません「System.Linq.IQueryable`1 [可能System.Int32]

Question

私は、次の機能があります：私は何をしようとしている

public List<DTO.ArticleDTO> GetRecentArticles(int portalID, int moduleID, int domainID, int rootKnowledgeTypeID, int count) 
{ 
    var articles = 
     (from 
      article in _Context.Articles.LatestArticles(_Context.ArticleVersions, portalID, moduleID) 
      where _GetRootDomainsForArticle(article.ArticleID, article.Version, rootKnowledgeTypeID).Contains(domainID) 
     orderby article.UpdatedOn descending 
      select new ArticleDTO 
      { 
       ArticleID = article.ArticleID, 
       Title = article.Title, 
       Summary = article.Introduction, 
       Content = article.Content, 
       UpdatedOn = article.UpdatedOn, 
       Version = article.Version, 
       KnowledgeTypeText = (from 
             category in _Context.Categories 
            join 
             categoryVersion in _Context.ArticleCategoryVersions 
            on 
             category.CategoryID equals categoryVersion.CategoryID 
            where 
             categoryVersion.Version == article.Version && 
             categoryVersion.ArticleID == article.ArticleID && 
             category.ParentID == rootKnowledgeTypeID 
            select 
             category.Name 
           ).First() 
      }); 

    return articles.Take(count).ToList(); 
} 

をすべて取得することですルートカテゴリ（ドメインID）に属する記事。カテゴリはドメイン/サブドメインとみなされ、各記事は1つ以上のドメインとサブドメインに属します。ですから、私は上記のlinqクエリのwhere節で以下のメソッドを使用しています。

private IQueryable<int> _GetRootDomainsForArticle(int articleID, int version, int rootKnowledgeTypeID) 
{ 
    return from c in _Context.Categories 
        join cv in _Context.ArticleCategoryVersions on c.CategoryID equals cv.CategoryID 
        where cv.ArticleID == articleID 
        where cv.Version == version 
        where c.ParentID == null 
        where c.CategoryID != rootKnowledgeTypeID 
        select c.CategoryID; 
} 

私が手にエラーがある： "比較演算子 'はSystem.Linq.IQueryable`1 [可能System.Int32]' 型に対応していません。"

私は遅延実行のためだと思っていますが、正しい方法を理解していません。

編集：@SivaGopalコメントに基づいて

私は、メイン・クエリの一部になることを_GetRootDomainsForArticleからクエリを移動し、それが働いた：

public List<DTO.ArticleDTO> GetRecentArticles(int portalID, int moduleID, int domainID, int rootKnowledgeTypeID, int count) 
{ 
    var articles = 
     (from 
      article in _Context.Articles.LatestArticles(_Context.ArticleVersions, portalID, moduleID) 
      where (from c in _Context.Categories 
       join cv in _Context.ArticleCategoryVersions on c.CategoryID equals cv.CategoryID 
       where cv.ArticleID == article.ArticleID 
       where cv.Version == article.Version 
       where c.ParentID == null 
       where c.CategoryID != rootKnowledgeTypeID 
       select c.CategoryID).Contains(domainID) 
      orderby article.UpdatedOn descending 
      select new ArticleDTO 
      { 
       ArticleID = article.ArticleID, 
       Title = article.Title, 
       Summary = article.Introduction, 
       Content = article.Content, 
       UpdatedOn = article.UpdatedOn, 
       Version = article.Version, 
       KnowledgeTypeText = (from 
           category in _Context.Categories 
            join 
           categoryVersion in _Context.ArticleCategoryVersions 
          on 
           category.CategoryID equals categoryVersion.CategoryID 
            where 
           categoryVersion.Version == article.Version && 
           categoryVersion.ArticleID == article.ArticleID && 
           category.ParentID == rootKnowledgeTypeID 
            select 
           category.Name 
         ).First() 
      }); 

    return articles.Take(count).ToList(); 
} 

のLINQの私の限られた理解はに私を導きますそのクエリをクエリ構成の形式としてメソッドに分割し、再利用可能にすることができます。このようなことはないと思われますが、確かに、特にクエリの一部が頻繁に再利用されるようなコードを大幅に減らすため、これを行う方法が必要です。

Answer 1

あなたの関数をもっと分割してみてください。長くて深いLinqメソッドはうまくいくかもしれませんが、読みやすくするためにはそれほど良くなく、あなたが話しているようなバグを紹介することができます。私はまだこれをテストしていませんが、それはあなたの問題を解決するのに役立つはずです。

public List<DTO.ArticleDTO> GetRecentArticles(int portalID, int moduleID, int domainID, int rootKnowledgeTypeID, int count) 
{ 
    var allArticles = _Context.Articles.LatestArticles(_Context.ArticleVersions, portalID, moduleID); 
    var filteredArticles = new List<DTO.ArticleDTO>(); 

    foreach (DTO.ArticleDTO article in allArticles) 
    { 
     var domains = _GetRootDomainsForArticle(article.ArticleID, article.Version, rootKnowledgeTypeID); 
     if (domains.Contains(domainID) 
     { 
      filteredArticles.Add(article); 
     } 
    } 

    var articles = filteredArticles.OrderBy(article.UpdatedOn).Descending().Select(article => new DTO.ArticleDTO { 
     ArticleID = article.ArticleID, 
     Title = article.Title, 
     Summary = article.Introduction, 
     Content = article.Content, 
     UpdatedOn = article.UpdatedOn, 
     Version = article.Version, 
     KnowledgeTypeText = (from category in _Context.Categories 
              join categoryVersion in _Context.ArticleCategoryVersions 
              on category.CategoryID equals categoryVersion.CategoryID 
              where 
              categoryVersion.Version == article.Version && 
              categoryVersion.ArticleID == article.ArticleID && 
              category.ParentID == rootKnowledgeTypeID 
              select 
              category.Name 
             ).First() 
    }).Take(count).ToList(); 
} 

はまた、あなたは絶対にあなたの_GetRootDomainsForArticle関数からのIQueryable <>を返すために必要なのですか？あなたがそれを手放すことができれば、IEnumerable <>を返すようにしたいと思います。

Koda