DataTableに新しい列を値ごと結合したい

Question

ID	名前	身長
1	田中	170
2	木村	160

というテーブルAに

ID	体重
1	50
2	40

というテーブルBを結合して

ID	名前	身長	体重
1	田中	170	50
2	木村	160	40

というテーブルにしたいです。

今は行ごとのループで列に値を追加していますが、数万行あるため非常に時間が掛かっています。
パフォーマンスの良い方法はないでしょうか。

追記
今の処理

1. テーブルAへ列：体重を追加（Columm.Add)
2. テーブルAのRowでループ(ForEach)
   2.1. TableA(i)("体重")=TableB(i)("体重")

です。

Comments

[Zuishin]

今はどのようにしているのかコードを記述してください。

[Taishi2002]

フレームワーク上で処理を作っているためコードを記載することはできないのですが、処理の流れを追記致します。

[Zuishin]

ID が昇順で 1 ずつ増え、抜けがないことが両テーブルで保証されていますか？

[Taishi2002]

はい。保証されています。両テーブルのIDは完全に一致します。

Answer 1

元になる DataTable はどのように作っているのですか？

数万行ということは SQL Server などのデータベースから ADO.NET を利用して作成しているのではないかと想像していますが、もしそうであれば JOIN 句を使った SELECT クエリで直接データベースから目的の DataTable を作ってはいかがですか？

Comments

[Taishi2002]

残念ながらExcelを読み込んでいます。
C#で作成されたフレームワークのようなものの上で動作しており、ExcelをDataTable型で読み込む処理はフレームワークの部品を使用しています。

[退会済みユーザー]

EXCEL からでも ADO.NET と JOIN した SELECT クエリを使ってできると思うのですが？　質問者さんの方でやってみてできなかったということでしょうか？

[退会済みユーザー]

Excel ファイルが違うとダメかもしれませんね。ADO.NET + JOIN 句の SELECT クエリがダメだとすると、質問者さんが今使っているコードより早いかどうかは分かりませんが、LINQ を使って JOIN するという方法があります。検討されてはいかがですか？

Answer 2

状況が分からないので何とも言えないけど、
0. 手作業か何かでExcelからDBに各テーブルのデータを移行する
0. DBからはSQLで頑張る
正直DataTableは重いのでお勧めできないし、Excelからデータを拾ってってのも厳しいと思うので、可能ならこの手段が良いとは思う
それが難しいならZuishinさんの提案通りにDataTableからList<T>へ作り直してからLinqでJOINの方が速いと思う
DataTableは内部がobjectなので一々キャストして処理してるので

なお、DBから取得する場合は個人的にはDapperさんがおすすめ

IDが重複しないのなら以下のやり方でもありかもしれないけど、
作ったデータもDataTableにしないといけないとなるとListにしてPersonのIDのコメントを外して
.First(r => r.ID == row.Field<int>("ID"))で対象のインスタンスを拾って体重を設定した後、
最後にList<>を.ToDataTable()する感じかな？

//以下のクラスを定義しておいて
public class Person
{
    //public int ID{get;set;}
    public string 名前{get;set;} = string.Empty;
    public double 身長{get;set;} = 0.0;
    public double 体重{get;set;} = 0.0;
}

//Dictionaryに設定していく
var dic = new Dictionary<int, Person>();

foreach(var row in tableA)
{
    dic[row.Field<int>("ID")] = new Person
        {
            名前 = row.Field<string>("名前"),
            身長 = row.Field<double>("身長")
        };
}

foreach(var row in tableB)
{
    var p = dic[row.Field<int>("ID")];
    p.体重 = row.Field<double>("体重");
}

Answer 3

カラム名「体重」でのアクセスを序数でのアクセスに変えれば速くなると思いますが、DataTable を使う限りそれ以上にはならないと思います。
あとカウンタ変数 i を使っているので foreach を for に直すことで少し最適化できるような気もしますが、これはほとんど気休め程度でしょう。
DataTable よりももっと軽量なクラスを自作して使うと改善すると思います。

意外と遅い DataTable 、なので List を使うと 5 倍早くなる