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

ASP.NET、C++/CLI、Java 何でもどうぞ

C# と VB.NET の入門サイト

Re[3]: C# でExcelデータを一行ずつ

分類:[C#]　

C# でExcelデータを１行ずつ取得して処理する方法

初めまして！C# とVSの初心者です。
急ぎで作りたいものがあって
サンプルを参考にしているのですが解決できず困って投稿させていただきます。

言語はC#
VisualStudio2008
compornentOneを入れてExcel操作をしようとしています。

やりたいことはこうです。

①Excelブック（DATA_IN.xlsx）のSheet1の(A1)セルにあるデータを取得
②取得したデータに変換をかける
③変換をかけたデータを別のExcel(DATA_OUT.xlsx)の取得ファイルと同じセル（Sheet1の(A1)セル）に出力
④A2セルに移る→①からまたループでデータ行がなくなるまで１行ずつ処理する

これを以下のように書いてみましたが
１回のループで、INファイルのシートごとデータ取得し、
そのままOUTファイルにコピーされているような動きになっているように見えます。

このソースで、１行ずつの処理にさせるには
どこをどう直したらよいのでしょうか？

//Excel読み込み
C1XLBook wb = new C1XLBook();
int i = 0;

wb.Load(@"C:\DATA_IN.xlsx");

XLSheet sheet1 = wb.Sheets["Sheet1"];
string data = "";

i = 1;
do
{
i += 1;
data = (String)sheet1[i, 0].Value;
if (data != null)
{
XLSheet xlSheet = wb.Sheets["Sheet1"];

if (i == 0)
{
return;
}

//excel出力
sheet1[i, 0].Value = data;
wb.Save(@"C:\DATA_OUT.xlsx");

}
} while (data != null);

簡単な内容を質問してすみませんが、時間なく急いでおりまして、
どなたか回答くださると助かります。

よろしくお願いいたします。

2015/03/08(Sun) 21:37:39 編集(投稿者)

■No75243 (にこにこたん さん) に返信
> 簡単な内容を質問してすみませんが、時間なく急いでおりまして、
> どなたか回答くださると助かります。

まずは厳しいことを書いておきますが、この内容で質問をされてしまう状況で「時間なく急いで」作ると、失敗することが多いです。
アルゴリズムが思いつかない、たどり着かないということは、基礎の理解が不足しているか、応用できるほどに力をつけられていないと考えられるためです。


--- 以下修正済み(3/8 21:36) ----

> １回のループで、INファイルのシートごとデータ取得し、
> そのままOUTファイルにコピーされているような動きになっているように見えます。

この話は振る舞いから発言されているのですか？
それとも、ソースコードの見た目の話ですか？


> このソースで、１行ずつの処理にさせるには
> どこをどう直したらよいのでしょうか？

どのようにうまく動いていないかを説明してください。
とりあえず、ワークブックの保存は do-while ループの外に書いた方がよいでしょう。
(i == 0 になることがないとか、xlSheet という変数が使われていないとかツッコミどころはありますが…）

ご指摘&回答ありがとうございます！

> まずは厳しいことを書いておきますが、この内容で質問をされてしまう状況で「時間なく急いで」作ると、失敗することが多いです。
> アルゴリズムが思いつかない、たどり着かないということは、基礎の理解が不足しているか、応用できるほどに力をつけられていないと考えられるためです。
まさにご指摘通りです。
基礎からテキスト読みましたが、いざ組むとなると
つまづいてばかりです。でも仕上げなくてはいけない状況で
せめてヒントでも、とおもい、投稿しました。

> この話は振る舞いから発言されているのですか？
> それとも、ソースコードの見た目の話ですか？
動き、と書いてますので振る舞いのことです。

> どのようにうまく動いていないかを説明してください。
とりあえずは1行ずつ取得したいのですが
1度目で全行取得してしまうため、②を組み込めてません。
、うまくいってない箇所です。

> とりあえず、ワークブックの保存は do-while ループの外に書いた方がよいでしょう。
> (i == 0 になることがないとか、xlSheet という変数が使われていないとかツッコミどころはありますが…）
保存処理をループの外に、ですね。たしかに。。ヒントをありがとうございます！

2015/03/08(Sun) 23:35:49 編集(投稿者)

■No75249 (にこにこたんさん) に返信
>>どのようにうまく動いていないかを説明してください。
> とりあえずは1行ずつ取得したいのですが
> 1度目で全行取得してしまうため、②を組み込めてません。
> 、うまくいってない箇所です。

「1度目で全行取得してしまうため」と書かれていますが、そのように判断した理由を述べてください。
提示されているコードを読む限り、私はその見方に同意できませんので。

-----
提示されているコードは、１セルのデータを取り出して data 変数に代入し、同じセルに data 変数の内容を代入し、ワークブックの保存を繰り返して全行処理するものです。
あなたの要求に近い形のループになっていますが、そのことはご理解いただいていますか？

（ループの中にデータを書き換える処理を置くという発想に持って行けていない？）

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

ASP.NET、C++/CLI、Java 何でもどうぞ

C# と VB.NET の入門サイト

Re[3]: C# でExcelデータを一行ずつ

過去ログには書き込み不可