質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.58%

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

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 1,058

poy327

score 16

タイトル通り、"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;
        }
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • SurferOnWww

    2018/11/16 14:28

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

    キャンセル

  • papinianus

    2018/11/16 14:54

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

    キャンセル

回答 3

checkベストアンサー

+1

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

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

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

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/11/16 14:41

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

    キャンセル

+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);
            });

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.58%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る