AddRangeに関するエラーメッセージの解決方法について

Question

前回の質問ではお世話になりました。
C#歴半年＆MVC勉強したてですが、質問させていただきます。
今回はエラーメッセージの解決方法がわからず、質問させていただきたいと思います。

###前提・実現したいこと
C#(ASP.NET Framework4.5.1 MVC)で履歴をExcel帳票出力するウェブアプリケーションを作っています。
コレクションに対してAddRangeを実装したいのですがうまく動きません。

###発生している問題・エラーメッセージ
エラー 1 'System.Linq.IOrderedQueryable<A.Models.TEAM_HIS>' に 'AddRange' の定義が含まれておらず、型 'System.Linq.IOrderedQueryable<A.Models.TEAM_HIS>' の最初の引数を受け付ける拡張メソッドが見つかりませんでした。using ディレクティブまたはアセンブリ参照が不足しています

###ソースコード

namespace A.Models
{
    using System;
    using System.Data.Entity;
    using System.ComponentModel.DataAnnotations.Schema;
    using System.Linq;

    public partial class DefaultContext : DbContext
    {
        public DefaultContext()
            : base("name=DefaultContext")
        {
        }
        /// <summary>
        /// プロジェクト経歴
        /// </summary>
        public virtual DbSet<TEAM_HIS> TEAM_HIS { get; set; }
        /// <summary>
        /// 変更履歴
        /// </summary>
        public virtual DbSet<CHANGE_HIS> CHANGE_HIS { get; set; }
    }
}

namespace A.Models
{
    using System;
    using System.Collections.Generic;
    using System.ComponentModel;
    using System.ComponentModel.DataAnnotations;
    using System.ComponentModel.DataAnnotations.Schema;
    using System.Data.Entity.Spatial;
    public partial class TEAM_HIS
    {
        [Key]
        [Column(Order = 0)]
        [StringLength(10)]
        [DisplayName("team名")]
        public string team { get; set; }

        [Key]
        [Column(Order = 1)]
        [StringLength(10)]
        [DisplayName("プロジェクト")]
        public string project { get; set; }

        [Key]
        [Column(Order = 2)]
        [DisplayName("開始日時")]
        public DateTime start { get; set; }

        [Key]
        [Column(Order = 3)]
        [DisplayName("終了日時")]
        public DateTime end { get; set; }
    }
}



using System;
using System.Collections.Generic;
using System.Linq;
using System.text;
using System.Threading.Tasks;
using System.Web;
using System.Web.Mvc;
using System.Data.Entity;
using A.Models;

namespace A.Report.History
{
    /// <summary>
    /// 個人履歴(Excel)出力
    /// </summary>
    public class HistoryGenerator : BaseReportGenerator
    {
        /// <summary>
        /// コンストラクタ
        /// </summary>
        public ReportGeneratorService(DbSet dbcontext)
            : base(dbcontext)
    {}
        public void Set()
        {
            DateTime start = new DateTime(1900, 1, 1);
            DateTime next = new DateTime(1900, 1, 1);

            int i_c = 1;
            // 対象を設定します（今回は仮にAAAとします）
            var target = (from t in DbContext.CHANGE_HIS
                          where t.name == "AAA"
                          orderby t.date ascending
                          select t).FirstOrDefault();
            // CHANGE_HISの異動数を取得します
            var m_c = (from m DbContext.CHANGE_HIS
                          where m.name == target.name
                          && m.eve != "out"
                          select m).Count();
            // outした対象のdateを取得します
            var loss = (from l in DbContext.CHANGE_HIS
                        where l.name = target.name 
                        && l.eve = "out"
                        orderby l.date ascending
                        select l.date).FirstOrDefault();
            // 履歴のコレクションで対象に異動がない場合の宣言
            var col = from c in DbContext.TEAM_HIS
                      where c.team == target.team
                      orderby c.date, c.team ascending
                      select c;
            // 一度データを消します
                col = null;


            // 履歴のコレクションを作成
            foreach (var forlist in DbContext.TEAM_HIS)
            {
                start = (from s in DbContext.CHANGE_HIS
                        where s.team = forlist.team
                        && s.date >= start
                        orderby s.date, no ascending
                        select s.date).FirstOrDefault();
                if (m_c >= i_c)
                {
                    next = (from n in DbContext.CHANGE_HIS
                            where n.team == forlist.team
                            && n.date >= start
                            orderby n.date, n.team ascending
                            select n.date).FirstOrDefault();
                }else{
                    next = DateTime.Now;
                }

                // TEAM毎の個人履歴をセット
                var addcol = from a in DbContext.TEAM_HIS
                              where a.team == forlist.team
                              && a.start < next
                              && a.start >= start
                              && a.start < loss
                              orderby a.start ascending
                              select a;
                // 個人履歴に追加
                addcol.AddRange(col);
                ++i_c;

                foreach (var his in col)
                {
                    var hist = new History(target);
                    hist.Name = target.name;
                    hist.Team = his.team;
                    hist.Project = his.project;
                    hist.Start = his.start;
                    hist.End = his.end;
                    target.Histories.Add(hist);
                }
                target.ColStartPos = cols;
                cols += target.OutputWidth() + 1;
            }
        }
    }
}

###補足情報
TEAM_HISというプロジェクトの履歴を管理しているテーブルから、CHANGE_HISという個人の異動経歴を管理しているテーブルを結合させて、個人ごとのproject履歴を表示したExcelを出力したいというのが、実現したい機能です。
ソースはその一部を表示させています。

注意しなくてはいけないのは、対象者(name)が、TEAMを移ることがあること、TEAMに所属している間にlossする（管理対象外になる）ことがあること。

以上を考慮したところ、対象者があるTEAMに所属していた期間、start, nextで取得し、その期間のproject履歴を抽出する。そして、それを異動回数（CHANGE_HISのレコード数-1)だけ繰り返し、コレクションに追加していく。ただし、CHANGE_HISのレコードにoutが出てきたら、その日付までとする。
このような観点でプログラムを作成しています。

ところが、AddRangeを使用すると、表記のエラーが出てしまい解決できませんでした。
エラー自体の意味が理解できていなく、可能でありましたらエラーの意味と解決方法を教えていただけますでしょうか。
よろしくお願いいたします。

###補足２
AddRangeをどう利用したいかですが、

// 1回目(本当はこれはaddcolではなく、colの形が望ましい）
addcol = 
{
 {T1, P01, 2013-5-1, 2013-8-10},
 {T1, P02, 2013-8-21, 2013-12-20},
 {T1, P03, 2014-1-11, 2014-3-30}
};
// 2回目(addcolは毎回上書きする)
addcol = 
{
 {T2, P01, 2014-5-1, 2014-8-10},
 {T2, P08, 2014-8-21, 2015-3-30}
};
// 3回目
addcol =
{
 {T3, P09, 2015-4-11, 2015-12-30}
};
// 合算後(addcolを3回目まで足しこむ)
col = 
{
 {T1, P01, 2013-5-1, 2013-8-10},
 {T1, P02, 2013-8-21, 2013-12-20},
 {T1, P03, 2014-1-11, 2014-3-30},
 {T2, P04, 2014-5-1, 2014-8-10},
 {T2, P08, 2014-8-21, 2015-3-30},
 {T3, P09, 2015-4-11, 2015-12-30}
};

上記のような2次元配列を足しこんでいくようにしていきたいです。

※2つのテーブルと出力結果をアップローダーにあげておきます。
リンク内容

Comments

[pinnkoro]

2点だけ質問を。
１．「AddRangeを実装したい」というのはTEAM_HISクラスにAddRangeメソッドを新しく作成したいということでしょうか？それともaddcolの後ろに要素をくっつけるようなListでいうAddRangeと同じ処理をさせたいのか。
２．DbSet の定義が書かれていないので分からないのですが、DbContext.TEAM_HISはどういう定義をされていますか？Listで書かれていますか？

[pinnkoro]

自己解決

[FitD]

質問ありがとうございます
１．はい、後者です。おぼろげながら自分が勘違いしているのがわかったのですが、AddRangeメソッドはコレクションにコレクションを加えることはできないのでしょうか？
○リストA.AddRange(コレクションB)
×コレクションA.AddRange(コレクションB)
もしそうであれば、コレクションにコレクションを加えるような処理を実装するか、あるいは設計しなおすかになるかと思います。

2.DbSetの定義については、ご指摘いただいたTEAM_HISを本文に加えてみます。

Answer 1

単に各対象(人)ごとに集計を行ってそれをListに突っ込んでおいて

  class result
  {
    public string Name;
    public string Team;
    public string Project;
    public date Start;
    public date End;
  }

List<result> results = new List<result>();

それを各対象ごとに横展開すればいいだけじゃないんですか？
→レポート側の処理？

Comments

[FitD]

回答ありがとうございます。

はい、質問の仕方が悪いのかと思いますが、集計の仕方でAddRangeを使っていて、それのエラー解決ができなかったのが一番の問題点です。
次点として、解決できないなら別なやり方で集計できないかと思ってます。

[dojikko]

AddRangeがダメのならForEachでAddすればいいのでは？
というか今のアプローチだとAddもできないとかないですよね？

[FitD]

はい、残念ながら問題はList<>型でないため、AddRangeやAddは使えないことがわかりました。
この設計だと集計ができないことがわかりましたので、一度設計しなおすことにします。
お付き合いいただきありがとうございました。

Answer 2

長くなりそうなので新しく回答します。

こちらで実装しようとしたAddRangeというメソッドは、System.collections.Genericに存在していますが、こちらではなく、A.Models.TEAM_HISにないか見に行っているということでしょうか。

オブジェクト指向をあまり理解されていないですかね？
と言うより、C#歴半年って書いてましたね。。。申し訳ないです。

System.collections.Genericをusingで呼び出したからと言ってそこに記述されているAddRangeがどこででも使用できるというわけではないです。
というよりも、System.Collections.Genericは名前空間なので中にメソッドは無いです。
リンクを参照してみてください。

すみません、理解が追い付いていませんが、addcolはTEAM_HISのコレクションであるから、この場合はAddRangeを使えないということになるのでしょうか。

コレクションという言い方がピンと来ないのですが、Linqをした結果を格納してますので
addcolの型はSystem.Linq.IOrderedQueryable<A.Models.TEAM_HIS>だと思われます。
前述したとおりSystem.Linq.IOrderedQueryableにAddRangeはないので使えないです。

colはaddcolでコレクションを追加していく空の入れ物として作りました。

元々のListなどのAddRangeでもNULLを追加することになり変ではないですか？

すみません、colとaddcolをLINQでの結合の仕方がわからないです。
もし可能でしたら教えていただけますと、AddRangeの問題ではない部分で解決できそうです。

下記コードを参照してみてください。
Dateの時間計算で長くなってしまいそうですが、1人1回のLinqで可能ではないでしょうか。

using System;
using System.Linq;

class Program
{
  class 商品情報
  {
    public int Id;
    public string 名前;
  }

  class 商品販売価格
  {
    public int Id;
    public int 価格;
    public string 店名;
  }

  static void Main(string[] args)
  {
    商品情報[] 商品情報データ =
    {
      new 商品情報() { Id = 1, 名前="PC-8001" },
      new 商品情報() { Id = 2, 名前="MZ-80K" },
      new 商品情報() { Id = 3, 名前="Basic Master Level-3" },
    };

    商品販売価格[] 商品販売価格データ =
    {
      new 商品販売価格() {Id=1, 価格=168000, 店名="BitOut"},
      new 商品販売価格() {Id=1, 価格=148000, 店名="富士山音響"},
      new 商品販売価格() {Id=2, 価格=178000, 店名="富士山音響"},
      new 商品販売価格() {Id=3, 価格=298000, 店名="マイコンセンターROM"},
      new 商品販売価格() {Id=3, 価格=229000, 店名="富士山音響"},
    };

    var query = from x in 商品情報データ
                join y in 商品販売価格データ
                                  on x.Id equals y.Id into z
                select new { Name = x.名前, 商品データ = z };

    foreach (var 商品 in query)
    {
      Console.WriteLine("{0}", 商品.Name);

      foreach (var 価格情報 in 商品.商品データ)
      {
        Console.WriteLine("\t{0} {1:C}",
                                価格情報.店名, 価格情報.価格);
      }
    }
  }
}

LINQ応用編

Comments

[FitD]

・オブジェクト指向に関して
正直あまり理解できていないと思います。今回のエラーの発端である、
「using ディレクティブまたはアセンブリ参照が不足しています」
を解決するためとして、ファイルにusing System.Collection.Genericがあるのになぜだめなのかいまだに理解できていません。
初期ではusingをC言語でのIncludeみたいなものだと思っていたのですが、先頭のusingは名前空間を指定しているだけで、他にも使い方があるみたいなことを書いてあった気がします。

・結合について
内部結合はわかるのですが、今回AddRangeを使えなくて問題になっていいるのは列の追加ではなくて、複数行の追加という意味です。

質問欄にもう１つソースを記述してみます。

[pinnkoro]

エラー文が確実に正しいと思ってはいけないです。
ないはずのAddrangeというメソッドを参照しようとしてるので、コンパイラ的には
「たぶん書いた人がusing書き忘れてたりDLLの参照ミスってんだろうな」
と思ってこういうエラー文が返ってきます。

>>内部結合はわかるのですが、今回AddRangeを使えなくて問題になっていいるのは列の追加ではなくて、複数行の追加という意味です。

複数行を追加したいのであれば、foreach文で1行ずつ取得して1行ずつAddしてみてはどうでしょうか。

>>・オブジェクト指向に関して
すごく大雑把に説明しますが、C#の基礎をもう一度勉強しなおしたほうが良いです。

下記ソースがあった時にメインのソースにusing NameAと書くとしましょう。
するとこれはNameAに含まれるclassを使いますよーっていう宣言になります。
もし、中のメソッドを使いたいときはインスタンスを作成して呼び出します。
ClassA　classA = new ClassA();
classA.MethodA();

このとき呼び出してるのはClassAのMethodAです。ClassBのMethodAはClassBのインスタンスを作って呼び出すしかないです。
もちろんClassCでMethodAを呼び出そうとしてもClassCでは作成していないので
「using ディレクティブまたはアセンブリ参照が不足しています」
のエラー文が出てくると思います。

すなわちメソッドはそれが作成されたクラスにしか適応されないと思ってください。
参考：ネームスペースとクラス
http://www.atmarkit.co.jp/fdotnet/csharp_abc/csharp_abc_002/csharp_abc01.html

namespace NameA
{
public class ClassA
{
public void MethodA()
{
　　　// 処理A
}
}
public class ClassB
{
public void MethodB()
{
// 処理B
}
}
public class ClassC
{

}
}

[FitD]

C#もですが、オブジェクト指向についてはもう少し勉強しなおしてみます

var addcol　→ List<object> addcol

とすることで、AddRangeのエラー自体はなくなりました。
もちろん、LINQの部分でエラーが発生してしまうので、この箇所は何とかしないといけませんが。
このことから、AddRangeがList<>型でないと使えないことがようやく理解できました。

問題が複雑化してしまったので、いったんこれにて本質問は解決とします。
ありがとうございました。

Answer 3

なんとなく理解が出来た気がするのでこちらに書かせていただきます。

最終的にされたいこととして、添付されてるエクセルの「出力データ」のように
4カラム＋空白1カラム計5カラムで1人分で人数分を横につなげたい ということでよろしいでしょうか。

ごめんなさい、とんでもない勘違いをしていたようです。
下記のような感じでご理解いただけますでしょうか。
参照URL:DbContextを動的に使用する

using (var db = new DynamicDbContext("DynamicDbContext", type))
{
  // DbSet<T>プロパティは無いのでSet()メソッドでDbSetインスタンスを取得
  var dbSet = db.Set(type);
   
  Console.WriteLine(dbSet.ElementType); // --> Person
 
  dbSet.AddRange(newEntities);
  db.SaveChanges();
}

Comments

[FitD]

質問を補って理解していただいてありがとうございます。

はい、最終的な目標はエクセルの「出力データ」のように、一人ごとに纏まりをつくるということではあります。
ただ、今の質問用として作成したソースではなく、実際のソースではそれを実装するメソッドは別途用意はされています（EPPlus 4.0.4？）残念ながら解析能力が足りなくて、その箇所は表現できませんが。
そのソースを使うためには、一人の履歴コレクションをアウトプットとして必要としています。こちらもソースコードに付け加えます。

一番知りたい内容としては、一人に関してのAddRangeのエラー解決ではあります。
それ以外にコレクションにコレクションを追加していく方法がありましたら、教えていただけると助かります。

[FitD]

修正いただいた内容について拝見させていただきました。
理解力が低いため、もうしばらくお時間をいただければと思います。

[FitD]

応用力が無いようで変換して転記してもうまくいきませんでした。

今回の場合でAddRangeを使う場合は、usingの宛先が動的に変化する？ため、usingの宛先を固定ではなく、プログラムで指定してあげなくてはいけないということでしょうか？
この場合、記述していただいたソースは、
namespace A.Report.Historyの前に挿入すればよいでしょうか？

[pinnkoro]

エラーの理由としてはSystem.Linq.IOrderedQueryable<A.Models.TEAM_HIS>
にAddRangeというメソッドが存在しないということです。
System.Linq.IOrderedQueryable<A.Models.TEAM_HIS>は、
var addcol = from a in DbContext.TEAM_HIS～～
で返ってきているaddcolが何かを見ていただけるとわかるのではないでしょうか。

下記参照
[MSDN参照](https://msdn.microsoft.com/ja-jp/library/bb340178(v=vs.110).aspx)

// 一度データを消します
　col = null;

となっていたため、！？っとなっているのですが、
addcol.AddRange(col);で追加してる「col」についてもよくわかっていません。

Linqでcolとaddcolを結合するのではダメでしょうか？

[FitD]

>>エラーの理由としてはSystem.Linq.IOrderedQueryable<A.Models.TEAM_HIS>
にAddRangeというメソッドが存在しないということです。
　こちらで実装しようとしたAddRangeというメソッドは、System.collections.Genericに存在していますが、こちらではなく、A.Models.TEAM_HISにないか見に行っているということでしょうか。

>>System.Linq.IOrderedQueryable<A.Models.TEAM_HIS>は、
>>var addcol = from a in DbContext.TEAM_HIS～～
>>で返ってきているaddcolが何かを見ていただけるとわかるのではないでしょうか。
　すみません、理解が追い付いていませんが、addcolはTEAM_HISのコレクションであるから、この場合はAddRangeを使えないということになるのでしょうか。

>>//一度データを消します
>>col=null;
　本当はif(m_c > 1) col = null;
でしたが、m_c == 1のときの抽出を省くためにnullを改めて入れました。
colを消しているのは、
// 履歴のコレクションで対象に異動がない場合の宣言
で作成したコレクションに追加していくと、データが重複してしまうためです。
型宣言がif文内でできないため、結果的にデータが空の型宣言をするのがスマートなのかもしれませんが、わからなかったためこのような記述になりました。

>>addcol.AddRange(col);で追加してる「col」についてもよくわかっていません。
　colはaddcolでコレクションを追加していく空の入れ物として作りました。

>>Linqでcolとaddcolを結合するのではダメでしょうか？
　すみません、colとaddcolをLINQでの結合の仕方がわからないです。
もし可能でしたら教えていただけますと、AddRangeの問題ではない部分で解決できそうです。

Answer 4

何とかエラーを出なくして、それっぽい形にはなりました。

using System;
using System.Collections;
using System.Collections.Generic;
using System.Linq;
using System.text;
using System.Threading.Tasks;
using System.Web;
using System.Web.Mvc;
using System.Data.Entity;
using A.Models;

namespace A.Report.History
{
    /// <summary>
    /// 個人履歴(Excel)出力
    /// </summary>
    public class HistoryGenerator : BaseReportGenerator
    {
        /// <summary>
        /// コンストラクタ
        /// </summary>
        public ReportGeneratorService(DbSet dbcontext)
            : base(dbcontext)
        {}
        public void Set()
        {
            DateTime start = new DateTime(1900, 1, 1);
            DateTime next = new DateTime(1900, 1, 1);

            int i_c = 1;
            // 対象を設定します（今回は仮にAAAとします）
            var target = (from t in DbContext.CHANGE_HIS
                          where t.name == "AAA"
                          orderby t.date ascending
                          select t).FirstOrDefault();
            // CHANGE_HISの異動数を取得します
            var m_c = (from m DbContext.CHANGE_HIS
                          where m.name == target.name
                          && m.eve != "out"
                          select m).Count();
            // outした対象のdateを取得します
            var loss = (from l in DbContext.CHANGE_HIS
                        where l.name = target.name 
                        && l.eve = "out"
                        orderby l.date ascending
                        select l.date).FirstOrDefault();
            // 出力対象の経歴コレクションの作成
			var add_col = (from a in DbContext.TEAM_HIS
                        orderby a.date ascending
                        select a)
            // AddRange関数を使用するためにList型にキャスト
            List<TEAM_HIS> col = new List<TEAM_HIS>();

            // 履歴のコレクションを作成
            foreach (var forlist in t_col)
            {
            	// カウントが1の時はnextが設定されていないため
            	if(i_c == 1);
            	{
	                start = (from s in DbContext.CHANGE_HIS
    	                    where s.team = forlist.team
        	                && s.date >= start
            	            orderby s.date, no ascending
                	        select s.date).FirstOrDefault();
                }
                else
                {
                	start = next;
                }
                // カウントがMAXでない場合は計算する
                if (m_c >= i_c)
                {
                	// TEAMの移動が無い場合は次の時間をセット（ただし1回目は除く）
                	if(i_c != 1 & start == next)
                	{
	                    next = (from n in DbContext.CHANGE_HIS
    	                        where n.team == forlist.team
        	                    && n.date > start
            	                orderby n.date, n.team ascending
                	            select n.date).FirstOrDefault();
                	}else{
                		next = (from n in DbContext.CHANGE_HIS
    	                        where n.team == forlist.team
        	                    && n.date >= start
            	                orderby n.date, n.team ascending
                	            select n.date).FirstOrDefault();
                	}
                }else{
                    next = DateTime.Now;
                }
                // TEAM毎の個人履歴をセット
                addcol = from a in DbContext.TEAM_HIS
                         where a.team == forlist.team
                         && a.start < next
                         && a.start >= start
                         && a.start < loss
                         orderby a.start ascending
                         select a;
                // 個人履歴に追加
                if(addcol != null) col.AddRange(addcol);
                ++i_c;
            }


            foreach (var his in col)
            {
                    var hist = new History(target);
                    hist.Name = target.name;
                    hist.Team = his.team;
                    hist.Project = his.project;
                    hist.Start = his.start;
                    hist.End = his.end;
                    target.Histories.Add(hist);
            }
            target.ColStartPos = cols;
            cols += target.OutputWidth() + 1;
        }
    }
}

やり方としては、<TEAM_HIS>をList<>にキャスト？したのですが、
「最も適しているオーバーロード メソッドには無効な引数がいくつか含まれています。 」
というエラーが何度も出てきましたが、無事解決できました。
ただ、実際にデータが正しく出てくるかはまだ未知数なので、1日がかりのテストに取り掛かる予定です。

ご協力ありがとうございました。

Comments

[FitD]

追記ご報告をします

最終的にはAddRangeを使用することができました
また、実際のプログラムで実行してみたところ、所々プログラムの設計ミスがあったりして、検証にはかなり時間を掛かってしまいました

子だとコードが見づらくなってしまうので、親のほうに反映させます。