Q&A
超初心者なので、丸投げの質問になります。
なにが出来なくて、実現できないのかの切り分けもできません。。。
ご容赦頂きたく。
以下のコードで、CSVファイルを配列に格納して、
別クラスで、配列を使用したいのですが、
どのようにすればよいのでしょうか？
ｃ＃歴１か月の初心者ですので、○○を参照というよりは、
コードを書いていただけると助かります。
よろしくお願いいたします。
using System; using System.Windows.Forms; namespace ｘｘｘｘｘｘ { public partial class CSVmasterRaead { public Form1() { stArrayData; //読み込めないです・・・・ } } using System; using System.Windows.Forms; namespace csvedit { public partial class Form1 : Form { public Form1() { InitializeComponent(); } private void button1_Click(object sender, EventArgs e) { MessageBox.Show("処理を開始します"); ReadCsv(); } static void ReadCsv() { try { // csvファイルを開く System.IO.DirectoryInfo dirPath = new System.IO.DirectoryInfo(@"E:/Desktop/Study/StudyProject/IN/"); System.IO.FileInfo[] files = dirPath.GetFiles("test_*.csv", System.IO.SearchOption.AllDirectories); // 指定フォルダからCSVを取得し、配列に格納する。 foreach (System.IO.FileInfo filePath in files) { using (var readCsv = new System.IO.StreamReader(filePath.FullName)) { //ヘッダを読み捨てる。 readCsv.ReadLine(); // ストリームの末尾まで繰り返す while (!readCsv.EndOfStream) { // ファイルから一行読み込む string line = readCsv.ReadLine(); // カンマ区切りで分割して配列に格納する string[] stArrayData = line.Split(','); } } } } catch (System.Exception e) { // ファイルを開くのに失敗したとき MessageBox.Show(e.ToString()); } } } }
回答5件
0
ベストアンサー
手っ取り早いのは、publicなstaticフィールドとしてList<string[]>を定義して、そこにCSVファイルの内容を格納していけば良いと思います。
C#
1// stArrayDataをpublicなstaticフィールドとして定義すれば、別クラスから参照可能 2public static List<string[]> stArrayData = new List<string[]>();
ファイル読み込み部分はこのようにしてください。
C#
1// カンマ区切りで分割して配列に格納する 2string[] record = line.Split(','); 3stArrayData.Add(record);
追記
Form1のコードを載せます。★マークが追加・変更部分です。
C
1public partial class Form1 : Form 2{ 3 public Form1() 4 { 5 InitializeComponent(); 6 } 7 8 private void button1_Click(object sender, EventArgs e) 9 { 10 MessageBox.Show("処理を開始します"); 11 ReadCsv(); 12 } 13 14 public static List<string[]> stArrayData = new List<string[]>(); // ★ 15 16 static void ReadCsv() 17 { 18 try 19 { 20 // csvファイルを開く 21 System.IO.DirectoryInfo dirPath = 22 new System.IO.DirectoryInfo(@"E:/Desktop/Study/StudyProject/IN/"); 23 System.IO.FileInfo[] files = 24 dirPath.GetFiles("test_*.csv", System.IO.SearchOption.AllDirectories); 25 26 // 指定フォルダからCSVを取得し、配列に格納する。 27 foreach(System.IO.FileInfo filePath in files) 28 { 29 using(var readCsv = new System.IO.StreamReader(filePath.FullName)) 30 { 31 //ヘッダを読み捨てる。 32 readCsv.ReadLine(); 33 // ストリームの末尾まで繰り返す 34 while(!readCsv.EndOfStream) 35 { 36 // ファイルから一行読み込む 37 string line = readCsv.ReadLine(); 38 // カンマ区切りで分割して配列に格納する 39 string[] record = line.Split(','); // ★ 40 stArrayData.Add(record); // ★ 41 } 42 } 43 } 44 } 45 catch(System.Exception e) 46 { 47 // ファイルを開くのに失敗したとき 48 MessageBox.Show(e.ToString()); 49 } 50 } 51}
投稿2016/06/19 07:57
編集2016/06/19 09:18
総合スコア5944
catsforepawさま
アドバイスありがとうございます。
しかし、私は、低レベルな人間であるため、
上記ヒントだけだと、わからないのです・・・
LISTにする意味はいまでも理解できませんが、
Staticにする意味はわかりました。
しかし、コードを記載している箇所が悪いようで、
ビルドエラーから抜け出せません。
そんなレベルなのです。。。
_|￣|○
Form1のコードは回答の方に追記します。
CSVmasterRaeadクラスの中からstArrayDataを参照する際は`csvedit.Form1.stArrayData`と書いてください。
CSVは複数レコードで構成されているので、そのすべてのレコードを保存するためにListを使います。1レコードは複数フィールドで構成されているためstring[]にデータを保存しますが、それが複数レコードあるので、List<string[]>となります。
catsforepawさま
ご回答ありがとうござます。
LISTの意味もよくわかりました。
しかしビルドエラーになります。。。_|￣|○
別クラスで、LIST変数を読み込みたいのと、
別クラスでメソッドを呼べるかの実験もしたかったので、
Form1のclassのメソッド部分をpublicに変更
旧：static void ReadCsv()
新：public static void ReadCsv()
そして、別classに以下のようにコーディング
をしても、static変数の箇所でビルドエラーになります。。。
”割り当て、呼び出し、インクリメント、デクリメント、新しいオブジェクトの式のみがステートメントとして使用できます。”
ググってもさっぱりです。。。
//名前空間は同じ
static class Program
{
/// <summary>
/// アプリケーションのメイン エントリ ポイントです。
/// </summary>
[STAThread]
static void Main()
{
Application.EnableVisualStyles();
Application.SetCompatibleTextRenderingDefault(false);
Application.Run(new Form1());
// staticメソッド
Form1.ReadCsv();
// static変数が読み込めない？？
Form1.stArrayData;
}
}
static変数を読み込むところで、
LIST型定義をしてあげたら、読み込めました。
ありがとうございました。
たったこんだけのソースで丸一日かかりました。。。
情けない。。。
catsforepawさまの支援がなければ、徹夜でした。
ありがとうございました。
// static変数が読み込めない
List<string[]> csvedit = new List<string[]>(); //追加
csvedit = Form1.stArrayData;
0
初心者ならばこそ結果だけもらうのではなく勉強しましょう。
投稿2016/06/19 03:24
総合スコア28669
問題点を数点あげます。
1 デフォルトエンコード(Windows の日本語環境なら Shift_JIS が多いが必ずではない)にしか対応していない
2 abc,def,"ghi,jkl",mns のような形式の CSV に対応していない
3 データがループの中で使い捨てられている
解決策
1 エンコードを指定し、オプションで変えられるようにしましょう。可能ならエンコードの自動判別に対応しましょう。
2 解析部をグレードアップしましょう。
3 読み込み部分を一つの関数にまとめ、yield return で返しましょう。
IEnumerable<string[]> ReadCsv(string fileName)
{
while (...) {
...
yield return strArrayData;
}
}
Zuishinさま
ヒントを頂きまして、ありがとうございます。
結果を頂こうとしているのは、意図がございまして、
Zuishinさまのように優秀ではないので、↑のヒントを頂いても
理解できないのです。。。
動いているコードをデバックして理解、
そのあと、改造してみて理解というスタンスでないと
頭が悪いので。。。だめなのです。
優秀な方の、理解するための脳内イメージのプロセスを
知りたいです。
stArrayData は ReadCsv のローカル変数なので、これを外からアクセスすることはできません。ローカル変数を外から読めるようにするには、おおまかに次の四つの方法があります。
1 戻り値として出力する、つまり return で返す。
2 out を使った参照渡しの引数として返す。
3 グローバル変数に入れる。
4 ファイルなどにシリアライズして読み込む。
このうち、推奨されるのは 1 2 です。
まず、ここまで理解してください。
わからないところはキーワードを Google で検索してください。
Zuishinさま
日曜日なのにお付き合い頂きまして、ありがとうございます。
↑のヒントは理解できました。
参照渡しというのも理解が乏しいので、頂いたヒントをもとに
２の方法で実験してみます。
また、初めのご指摘の
CSVの仕様ですが、とりあえず
SJISで、”編集なしのカンマ区切り、ヘッダーあり（ヘッダーチェックはしない）
でやろうと思います。
こちらの味付けは、ある程度あたりがついているので、後回しと思っています。
お付き合い頂きまして感謝しております。
Zuishinさま
グローバル変数を参照する形で、実現できました。
ただ参照渡しは、必要なので、勉強してみます。
ありがとうございました。
0
参照渡しのコードについて、
foreachでループしているところを
stArrayData = new List<string>();
stArrayData.Add("data1");
stArrayData.Add("data2");
のように手入力するとうまく参照できるのですが、
以下のようにforreachを使うとうまくいきません。
リスト変数の定義場所だと思っているのですが、
場所をいろいろ変えてもうまくいきません。
↑まったく見当ちがいのところを見ているのでしょうか？
List<string[]> stArrayData
using System; using System.Collections.Generic; using System.Windows.Forms; namespace StudyProject { public partial class Form1 : Form { public Form1() { InitializeComponent(); } private void button1_Click(object sender, EventArgs e) { MessageBox.Show("処理を開始します"); // out を使った場合、変数を初期化不要 List<string> stArrayData; CsvEdit.ReadCsv(out stArrayData); MessageBox.Show("ここでブレークポイントでチェック"); } } public static class CsvEdit { public static void ReadCsv(out List<string> stArrayData) { try { // csvファイルを開く System.IO.DirectoryInfo dirPath = new System.IO.DirectoryInfo(@"E:/StudyProject/IN/"); System.IO.FileInfo[] files = dirPath.GetFiles("test_*.csv", System.IO.SearchOption.AllDirectories); // 指定フォルダからCSVを取得し、配列に格納する。 foreach (System.IO.FileInfo filePath in files) { using (var readCsv = new System.IO.StreamReader(filePath.FullName)) { //ヘッダを読み捨てる。 readCsv.ReadLine(); // ストリームの末尾まで繰り返す while (!readCsv.EndOfStream) { // ファイルから一行読み込む string line = readCsv.ReadLine(); // カンマ区切りで分割して配列に格納する string[] record = line.Split(','); public static List<string[]> stArrayData = new List<string[]>(); stArrayData.Add(record); } } } MessageBox.Show("読み込み完了しました。"); } catch (System.Exception e) { // ファイルを開くのに失敗したとき MessageBox.Show(e.ToString()); } } } }
投稿2016/06/19 12:52
編集2016/06/19 13:30総合スコア14
0
'stArrayData' はコントロールが現在のメソッドを抜ける前に割り当てられる必要があります。
のエラーのコードになります。
get setを指定するような記述のサイトもありましたが、
うまくいきませんでした。
using System; using System.Collections; using System.Windows.Forms; namespace StudyProject { public partial class Form1 : Form { public Form1() { InitializeComponent(); } private void button1_Click(object sender, EventArgs e) { MessageBox.Show("処理を開始します"); string[] stArrayData; ReadCsv(out stArrayData); } public static void ReadCsv(out string[] stArrayData) { try { // csvファイルを開く System.IO.DirectoryInfo dirPath = new System.IO.DirectoryInfo(@"E:/Desktop/Study/StudyProject/IN/"); System.IO.FileInfo[] files = dirPath.GetFiles("test_*.csv", System.IO.SearchOption.AllDirectories); // 指定フォルダからCSVを取得し、配列に格納する。 foreach (System.IO.FileInfo filePath in files) { using (var readCsv = new System.IO.StreamReader(filePath.FullName)) { //ヘッダを読み捨てる。 readCsv.ReadLine(); // ストリームの末尾まで繰り返す while (!readCsv.EndOfStream) { // ファイルから一行読み込む string line = readCsv.ReadLine(); // カンマ区切りで分割して配列に格納する stArrayData = line.Split(','); } } } } catch (System.Exception e) { // ファイルを開くのに失敗したとき MessageBox.Show(e.ToString()); } } } }
投稿2016/06/19 06:55
総合スコア14
途中経過ですが、
out パラメーター 'stArrayData' はコントロールが現在のメソッドを抜ける前に割り当てられる必要があります。
といったエラーがでて、MSDNをみても意味が分かりません。
_|￣|○
もう少し、調べてみます。
0
Microsoft.VisualBasic.FileIO > TextFieldParserがC#からも使えます
方法 : Visual Basic でコンマ区切りのテキスト ファイルを読み取るをC#で書きなおしてみてはどうでしょう
投稿2016/06/19 05:39
総合スコア3939
dojikkoさま
いつも回答をありがとうございます。
上記のVBのやり方は既に知っていました。
なんとかC#でのみの解決を探っています。
後学の為に、参考にさせていただきます。
CCcambodia
あなたの回答
tips
プレビュー
