C#、エクセルシートのデータを配列に入れる際のループの回し方について

Question

var objWsAaa = (object[,])wsAaa.Range[wsAaa.Cells[12,2],wsAaa.Cells[r,c]].Cells.Value2;

上記ワークシートのデータをリストに入れる際に
大枠のループでは列数、
中のループでは行数で処理にあてはめなければいけません。
可能でしょうか。
また、どのようにリストに入れることができますか。

var listObjWsAss = new double[r];
v = 0;
for(k = 1; k < c + 1; k++)
{
  for(n = 1; n < r + 1; r++)
  {
  listObjWsAss[n][v] = double.Parse((string)objWsAaa[n,k]);
  }
} 

今の状況は上記です。
よろしくお願いいたします。

Answer 1

コメント踏まえて、シートA→シートBへのデータ反映部分を組んでみました。

//列毎に処理
for(int currentCol = 1; currentCol <= c - 2; currentCol++)
{
    //シートA対象列の全行取得
    var rowsData = new List<object>();
    for(int currentRow = 1; currentRow <= r - 12; currentRow++)
    {
        rowsData.Add(objWsAaa[currentRow, currentCol]);
    }

    //対象列をシートBにコピーするか判定（１つでも条件を満たすセルがあればコピーしない）
    var isExist = rowsData.Any((x) =>
        {
            //True:セルが対象外とする条件を満たす
            return true;
        });

    //シートBデータへ反映
    if(!isExist)    //対象列のみ
    {
        for(int currentRow = 1; currentRow <= r - 12; currentRow++)
        {
            //便宜上、上書きしてる。加算とか、なんでも良い。
            objWsB[currentRow, currentCol] = rowsData[currentRow - 1];
        }
    }
}
//※シートAのデータ行数≦シートBのデータ行数を前提としたのでチェックしてない。
//※doubleの必要性がよくわからなかったのでobjectのまま処理している

未検証です。ループが1回多く回るとか、凡ミスがあるかも。

Comments

[tomi0416]

ありがとうございます。
一点質問があります。
"var isExist = rowsData.Any((x) =>{});"
上記の"(x)"って何ですか？！

[Tak1wa]

rowsDataの１要素です。
rowsDataはList<object>なのでobjectですね。

[tomi0416]

もう少しでできそうです。
ありがとうございました！

Answer 2

こんにちは。

tomi0416さんのやりたいことが、ちゃんと理解出来ているか自信が無いですが、
おそらく、案２ですか？

//案１：行列の番号でリスト作りたいなら
var result = new Dictionary<Tuple<int, int>, double>();
for (k = 1; k < c + 1; k++)
{
    for (n = 1; n < r + 1; n++)
    {
        //TODO:Cast可否チェックしたほうが良いです。
        result.Add(
            new Tuple<int, int>(n, k),
            double.Parse((string)objWsAaa[n, k])
            );
    }
}

//案２：Excelの行列番号ではなく0始まりで配列作るのであれば
const int startCol = 12;    //wsAaa.Cells[12,2]の12
const int startRow = 2;     //wsAaa.Cells[12,2]の2
var listObjWsAss = new double[r,c];
//v = 0;    vって固定で0ですか？つかいどころがイマイチわかりませんでした。
for (k = 1; k < c + 1 - startCol; k++)
{
    for (n = 1; n < r + 1 - startRow; n++)
    {
        listObjWsAss[n - 1, k - 1] = double.Parse((string)objWsAaa[n, k]);
    }
}

未検証！

Comments

[tomi0416]

ありがとうございます！！！
案２です！
しかし、案２を記載した際に、”[n - 1, k - 1]”でエラーになります。
エラー内容は、”Cannot apply indexing to an expression of type 'object'”です。

[tomi0416]

エラー内容を間違えました。

正、"Indexer has 1 paramerter(s) but is invoked with 2 argument(s)"
誤、"Cannot apply indexing to an expression of type 'object'"

[Tak1wa]

うーん、実装したコードを見せてもらえますか？

[tomi0416]

var listObjWsAss = new double[r];
v = 0;
for(k = 1; k < c + 1; k++)
{
for(n = 1; n < r + 1; r++)
{
listObjWsAss[n-,v-] = double.Parse((string)objWsAaa[n,k]);
}
}

[tomi0416]

上記です。

[Tak1wa]

まず、格納する配列は多次元で定義してください。
→var listObjWsAss = new double[r,c];

[Tak1wa]

変数vは結局使うのでしょうか？ずっと0ですけど…

[tomi0416]

var listObjWsAss = new double[r,c];
v = 0;
for(k = 1; k < c + 1; k++)
{
for(n = 1; n < r + 1; r++)
{
listObjWsAss[n-,v-] = double.Parse((string)objWsAaa[n,k]);
}
v = v + 1;
}

[Tak1wa]

とりあえずこちらでは以下で動いており、静的エラーも実行時エラーも発生していません。
var listObjWsAss = new double[r,c];
int v = 0; //便宜上ここで変数定義
for(k = 1; k < c + 1; k++)
{
for(n = 1; n < r + 1; n++)
{
listObjWsAss[n-1,v] = double.Parse((string)objWsAaa[n-1,k-1]);
}
v = v + 1;
}

"Indexer has 1 paramerter(s) but is invoked with 2 argument(s)"
このエラーは listObjWsAss[n-1,v] = …の行で発生しているのでしょうか。

追加で以下を教えてください。

VisualStudioのバージョン、ターゲットフレームワークのバージョン

[tomi0416]

VisualStudioのバージョン、2013
ターゲットフレームワークのバージョン、4.5

すみません、私の質問はTak1waさんが考えてくださった案２なのですが、
案２のvar listObjWsAss = new double[r,c];の部分で配列の要素数を指定しているのですが、要素数は、
for (k = 1; k < c + 1 - startCol; k++){}内の条件で決まります。
なので、宣言する際は要素数がわからない状態です。

var objWsAaa = (object[,])wsAaa.Range[wsAaa.Cells[12,2],wsAaa.Cells[r,c]].Cells.Value2;
var listObjWsAss = new List<List<double>>();
v = 0;
for(k = 1; k < c + 1; k++)
{
var listData = new List<double>();
for(n = 1; n < r + 1; r++)
{
listData [n][v] = double.Parse((string)objWsAaa[n,k]);
}
　listObjWsAss.Add(listData);
}

上記、今の状態です。

やりたいことの詳細を追記させていただきます。
エクセルシートＡにあるデータをエクセルシートＢに貼りつけたいです。
その際に条件があります。
エクセルシートＡの列ごとチェックします。（一個目のfor文）
条件を満たした列は、エクセルシートＢにはりつける対象列になります。
二個目のfor文で対象列の行のデータを取得したいです。

どうぞ、よろしくお願いいたします。

[Tak1wa]

なるほど。ありがとうございます。意味がわかりました。
ふたつ質問です。
１．対象外と判断された列はシートB上ではどうなりますか。空白列ですか？詰めますか？
２．ObjectのままではなくDouble型にしているのは何故ですか？

[tomi0416]

ありがとうございます。
１．対象外と判断された列はシートB上ではどうなりますか。空白列ですか？詰めますか？
→　対象外と判断された列はシートB上ではでは変化しないです。理由は、シートBのデータをオブジェクトに入れています。
var objWsB = (object[,])wsB.Range[wsB.Cells[a,2],wsB.Cells[b,c]].Cells.Value2;
（a,b,cはシート内から出力したlong型のデータです。）

２．ObjectのままではなくDouble型にしているのは何故ですか？
→　シートBのデータがDoubleなのでDoubleにしました。もしかして、関係ないですか？！

[Tak1wa]

objWsBの登場で複雑になってしまいました。
添え字のa, b, c, r などのせいで問題点がわかりにくくなっています…。
シートAでは12, 2, r, cが登場しますね。
シートBではa, 2, b, cが登場しますね。
このそれぞれのインデックスの関連性を教えてください。

[tomi0416]

var objWsAaa = (object[,])wsAaa.Range[wsAaa.Cells[12,2],wsAaa.Cells[r,c]].Cells.Value2;

var objWsB = (object[,])wsB.Range[wsB.Cells[a,2],wsB.Cells[b,c]].Cells.Value2;

関連性は特にないです。
変数にした理由は、固定ではないっということを伝えたかっただけです。
値は各シートのデータ量で変動します。

ご存じの通り、"シートBのデータ量" ＞ "シートAのデータ量"です。