C# と VB.NET の質問掲示板

■No87961 (河童 さん) に返信

data_inpにどういう規則で値を入れるのか
わかんないですけど、結果をリストにまとめて
ぐるぐるやったらいんじゃないでしょうか

var sheetNames = new List<string>() { "A病室", "B病室" };
var values = new List<string[,]>();

foreach (var sheetName in sheetNames)
{
    var value = Sheet_Torikomi(sheetName, workbook);
    values.Add(value);
}

var data_inp = new string[1000, 5];
var row = 0;

foreach (var value in values)
{
    for (var i = 0; i < value.Length; i++)
    {
        data_inp[row++, 0] = value[i, 0];
    }
}

■No87963 (河童 さん) に返信

おはよーございますー

外側のループが病室をぐるぐるしますので
value[i, 0]==nullのときに内側のループをbreakで抜けたら
次の病室を処理するようにできます

ちょっと私のLengthの使い方がおかしかったので
GetLengthに修正しました、二次元配列の次元を指定しないとダメでした

> data_inp[row++, 0] = value[i, 0];
> data_inp[row++, 1] = value[i, 1];
> data_inp[row++, 2] = value[i, 2];

ここのところって列ごとに行を変える感じなんですか？
列数が3個あるので同じ行に入れるのが自然なんじゃないかなって
思って↓こちらのコードの出来上がりです

var data_inp = new string[1000, 5];
var destRow = 0;

foreach (var value in values)
{
    for (var srcRow = 0; srcRow < value.GetLength(0); srcRow++)
    {
        if (value[srcRow, 0] == null)
        {
            break;
        }

        for (var field = 0; field < value.GetLength(1); field++)
        {
            data_inp[destRow, field] = value[srcRow, field];
        }

        destRow++;
    }
}