C#で、"CSVファイルよりデータを文字列配列に取り込むソース"を作成したい

Question

タイトル通り、"C#で、"CSVファイルよりデータを文字列配列に取り込むソース"を作成したいです。

色々と参考にしながら、よく見たことのあるソースを作成しました。

しかし素朴な疑問なのですが、下記のソースの場合、例えば"Item.csv"、"Chart.csv"という2つのcsvファイルがあった場合、2つともちゃんと読み込んでくれるのでしょうか？

ファイル名を指定していないため、この2つのcsvファイルをどのようにして特定し、読み込んでくれるか不安になりました。

自分で確かめれば良いことなのですが、とある事情で3日ほどvisual studioが使用できないのですが、もやもやするためご教授願います。

 private void BtnSplit_Click(object sender, EventArgs e)
        {
            List<string[]> sStrList = new List<string[]>();  // CSV分割後格納リスト

            string sCsvPath = null;
            if (CsvSplit(sCsvPath, ',', ref sStrList) == false)
            {
               
                return;
            }

            sStrList.ForEach(sList =>
            {
                Debug.WriteLine(sList);
            });
        }






        public bool CsvSplit(string sCsvFilePath,char chDlimiter,ref List<string[]> sStrList)
        {
            bool bRet = true;

            try
            {
                // CSVファイル読込
                using (StreamReader objReader = new StreamReader(sCsvFilePath,Encoding.GetEncoding("shift_jis")))
                {
                    // ストリームの末尾まで繰り返す
                    while (objReader.EndOfStream != true)
                    {
                        // ファイルから一行読み込む
                        string sLine = objReader.ReadLine();

                        // 読み込んだ一行を区切り文字で分けて配列に格納する
                        string[] sValues = sLine.Split(chDlimiter);

                        // 返却用リストに格納
                        sStrList.Add(sValues);

                    }//End while
                }
            }
            catch (Exception e)
            {
                // ファイルを開くのに失敗したとき
                Debug.WriteLine(e.Message);

                bRet = false;
            }

            return bRet;
        }

Comments

[退会済みユーザー]

前のスレッド https://teratail.com/questions/158354 の続きですよね。前のスレッドでいろいろレスを貰っているのですから、まず前のスレッドでそれらにフィードバックを返せませんか？　そして、方針変更して新たに別のスレッドを立てて質問するということを書いていただければと思います。

[papinianus]

SurferOnWww様と指摘が似ているのですが、前回は、配列の配列(ジャグ配列)のはずでしたよね？仕様変更なんですか？手前味噌みたいになって嫌なんですが、前回の質問で私の書いたのは今回のコード程度のゆるいcsvなら対応していますし、List<string[]>にも容易に対応できるんですが？というかIEnumerable<IEnumerable<string>>でもいいってことですか？

Answer 1

• BtnSplit_Click

string sCsvPath = null; → string sCsvPath = "Item.csv";という風に指定すればいいですし、FileDialogで選択させればいい。

後から、思ったのですが、2つのファイルを比較とか連結とかしたいということですかね？
書かれたコードはrefで引数を渡してaddしているからsCsvPathをかえて2回CsvSplitを呼び出せば、連結させることはできると思います。
比較するとか、結合(RDBのjoin)がやりたいなら、2つ変数を宣言して、LINQでやれば容易にできると思います(IntesectとかExceptとかJoinとかで)。

• C#について

なんども言いますが、C++のことは忘れてください
システムハンガリアンで書かないでください(visual studioを使っているのですよね？)
このパターンでCsvSplitに成功したかどうかをboolで返す必要はないはずです。長さ0のListを返せばよいでしょう。

↓こう書いてますが、sListはsつまりstringではなく、string[]なので、要素の出力はできません。string.Join(",", sList)とかにしなければ。

            sStrList.ForEach(sList =>
            {
                Debug.WriteLine(sList);
            });

Answer 2

うーん、聞きたいことが分からない…

しかし素朴な疑問なのですが、下記のソースの場合、例えば"Item.csv"、"Chart.csv"という2つのcsvファイルがあった場合、2つともちゃんと読み込んでくれるのでしょうか？

読み込みたいファイルをちゃんと指定すればそりゃ読み込めます。
読み込みたいファイルが存在するディレクトリを指定してその中にある.csvファイルだけを読み込むように実装すればそりゃ読み込めます。

ファイル名を指定していないため、この2つのcsvファイルをどのようにして特定し、読み込んでくれるか不安になりました。

ファイル名を指定（正確にはファイルのフルパスかな）すれば読み込めますが、それを指定しなければ当然読み込まれもしません。

ちなみにサンプルコードをそのまま実行したらヌルリで落ちるんじゃないですかね。

Comments

[Zuishin]

catch してるので落ちはしませんがまともに動きそうもないですね。
あと書き方が古すぎて 20 年前のソースかと思いました。

Answer 3

ファイルの読み込みに関しては、既出の回答を読んで下さい。
下記はそれ以外に関してです。

例えば、csvファイルのある行に、
a,b,c,"d,e",f
とあるとします。
普通、この場合この行は5列として扱われますが、現在のコードでは、
6列となります。
ダブルクオーテーションのチェックをしましょう。

あと、CSVファイルを読み込むだけなら、わざわざコードを書かなくても
Microsoft.VisualBasic.FileIO.TextFieldParserを使えば良いと思います。
参考サイト