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

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

新規登録して質問してみよう
ただいま回答率
85.50%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Windows Forms

Windows Forms(WinForms)はMicrosoft .NET フレームワークに含まれる視覚的なアプリケーションのプログラミングインターフェイス(API)です。WinFormsは管理されているコードの既存のWindowsのAPIをラップすることで元のMicrosoft Windowsのインターフェイスのエレメントにアクセスすることができます。

Q&A

解決済

3回答

687閲覧

C#でCSVデータを配列に読み込もうとするとNullReferenceExceptionが出る

SATSUKI.

総合スコア21

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Windows Forms

Windows Forms(WinForms)はMicrosoft .NET フレームワークに含まれる視覚的なアプリケーションのプログラミングインターフェイス(API)です。WinFormsは管理されているコードの既存のWindowsのAPIをラップすることで元のMicrosoft Windowsのインターフェイスのエレメントにアクセスすることができます。

0グッド

0クリップ

投稿2017/07/24 05:14

編集2017/07/24 05:45

概要

c#でフォームアプリを作っています。その中でcsvファイルのデータを配列に読み込む処理があるのですが、その処理をしている部分でNullReferenceExceptionが出てしまいます。おそらくインスタンスの生成がうまくいってないと思うのですが、なぜなのかわからず困っています。c#は勉強している最中で、そこら辺がまだよく理解できてないのでお力を貸していただきたいです。

フォームアプリの仕様

8個のcsvファイル(データは数値、行と列のデータの数は揃っていて8個のファイルすべて同じデータ数、空データなし)をopenFileDialogで開きます。calcボタンが押されるとそれらのデータを一旦配列に読み込み、計算処理をした結果を新しい2つのcsvファイルに書き込んで保存します。

ソースコード

以下該当部分の抜粋です。

c#

1using System; 2using System.Collections.Generic; 3using System.ComponentModel; 4using System.Data; 5using System.Drawing; 6using System.IO; 7using System.Linq; 8using System.Text; 9using System.Threading.Tasks; 10using System.Windows.Forms; 11 12 13//namespaceなど中略 14 15 private void buttonCalc_Click(object sender, EventArgs e) //calcボタンが押されたときに発生するイベント 16 { 17 if (openFileDialog1.FileNames.Length == 8) //ファイルが8つ選択されていたら 18 { 19 StreamReader[] readCSVfile = new StreamReader[8]; //ファイルから読み込んでデータを格納するためのstreamオブジェクト 20 for (int n = 0; n < 8; n++) //ファイル8個開く 21 { 22 readCSVfile[n] = new StreamReader(openFileDialog1.FileNames[n], Encoding.GetEncoding("Shift_JIS")); //データが入っているcsvファイルを開く 23 } 24 string calculated1FilePath = textBoxSaveFolder.Text + "\\calculated1.csv"; //計算結果を保存するcsvファイルのパス 25 string calculated2FilePath = textBoxSaveFolder.Text + "\\calculated2.csv"; //計算結果を保存するcsvファイルのパス 26 StreamWriter calculated1CSV = new StreamWriter(calculated1FilePath); //calculated1のデータを書き込むcsvファイルを生成 27 StreamWriter calculated2CSV = new StreamWriter(calculated2FilePath); //calculated2のデータを書き込むcsvファイルを生成 28 29 30 try 31 { 32 for (int i = 0; readCSVfile[0].EndOfStream == false; i++) //行を走査 33 { 34 string[,] dataPixelLine = new string[8, readCSVfile[0].ReadLine().Split(',').Length]; //データ一行分をカンマで区切ったものを8つ(ファイル8つあるので)格納するためのstring型二次元配列//添字の0次元目はファイル番号、1次元目は列番号 35 36 for (int n = 0; n < 8; n++) //ファイルを8個同時に開いて計算する 37 { 38 for (int s = 0; s < readCSVfile[n].ReadLine().Split(',').Length; s++) //列を走査 39 { 40 dataPixelLine[n, s] = String.Copy(readCSVfile[n].ReadLine().Split(',')[s]); //一行分の文字列をカンマで区切って一列ずつstring型2次元配列に格納 -------ここでNullReferenceExceptionが出る------- 41 } 42 } 43 44 45 46 string[] calculated1String = new string[dataPixelLine.GetLength(0)]; //計算したcalculated1の値を一行分記憶するためのstring型配列 47 string[] calculated2String = new string[dataPixelLine.GetLength(0)]; //計算したcalculated2の値を一行分記憶するためのstring型配列 48 49 //ここに計算処理が入る 50 //計算結果がcalculated1Stringとcalculated2Stringに格納される 51 52 string calculated1Line = string.Join(",", calculated1String); //calculated1String[]の各要素に格納されている文字を、カンマ区切って一つの文字列にする 53 string calculated2Line = string.Join(",", calculated2String); //calculated2String[]の各要素に格納されている文字を、カンマ区切って一つの文字列にする 54 calculated1CSV.WriteLine(calculated1Line); //新しいcsvファイルに一行書き込む 55 calculated2CSV.WriteLine(calculated2Line); //新しいcsvファイルに一行書き込む 56 } 57 } 58 finally 59 { 60 for (int n = 0; n < 8; n++) //ファイルクローズ処理 61 { 62 readCSVfile[n].Close(); 63 } 64 } 65 MessageBox.Show("処理終了"); 66 } 67 else 68 { 69 MessageBox.Show("ファイルを8つ選択してください"); 70 } 71 }

スタックトレース

************** 例外テキスト ************** System.NullReferenceException: オブジェクト参照がオブジェクト インスタンスに設定されていません。 場所 calc_csv.Form1.buttonCalc_Click(Object sender, EventArgs e) 場所 C:¥Users¥myPC¥Visual Studio 2017¥Projects¥C#¥Windows Form Apprications¥calc csv¥calc csv¥Form1.cs:行 77 場所 System.Windows.Forms.Control.OnClick(EventArgs e) 場所 System.Windows.Forms.Button.OnClick(EventArgs e) 場所 System.Windows.Forms.Button.OnMouseUp(MouseEventArgs mevent) 場所 System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks) 場所 System.Windows.Forms.Control.WndProc(Message& m) 場所 System.Windows.Forms.ButtonBase.WndProc(Message& m) 場所 System.Windows.Forms.Button.WndProc(Message& m) 場所 System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m) 場所 System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m) 場所 System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

よろしくお願いします。


バージョンなど

Windows10 Pro 64bit
Visual Studio 2017 community
.NET Framework 4.7
C# 7.0

気になる質問をクリップする

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

skitoy4321

2017/07/24 05:17

スタックトレース含めた例外の詳細内容を記載すれば、適切な回答が得られるかもしれません。
SATSUKI.

2017/07/24 05:46

ご指摘ありがとうございます。追記しました。
koguma98

2017/07/24 05:47

Form1.cs の 77 行目はどの行でしょう?
SATSUKI.

2017/07/24 05:53

ご指摘ありがとうございます。dataPixelLine[n, s] = String.Copy(readCSVfile[n].ReadLine().Split(',')[s]); の行です。ソースコード中にコメントで"-------ここでNullReferenceExceptionが出る-------"と記述してある行です。
guest

回答3

0

とりあえず、一度、splitした情報を配列に入れてから処理しては如何でしょうか?

C#

1string[,] dataPixelLine = new string[8, readCSVfile[0].ReadLine().Split(',').Length]; //データ一行分をカンマで区切ったものを8つ(ファイル8つあるので)格納するためのstring型二次元配列//添字の0次元目はファイル番号、1次元目は列番号 2//1行目の1列目が必ず読み飛ばされますが… 3 4for (int n = 0; n < 8; n++) //ファイルを8個同時に開いて計算する 5{ 6 string line = ""; 7 while ((line = readCSVfile[n].ReadLine()) != null) //ファイルの内容を全て読込む 8 { 9 Array strArray = line.Split(','); 10 for (int s = 0; s < strArray.Length; s++) //列を走査 11 { 12 dataPixelLine[n, s] = (string)strArray.GetValue(s); //一行分の文字列をカンマで区切って一列ずつstring型2次元配列に格納 -------ここでNullReferenceExceptionが出る------- 13 } 14 } 15}

その他諸々、気になったこと(本筋と関係ないので、読み飛ばしてもらってもOKです)

  • 出力ファイルがCloseされていないので、フォームを閉じずに2回目に実行するとエラーになります。
  • コメントにも書きましたが、必ず1行目の1列目が読み飛ばされてしまいます。
  • このロジックだと1ファイルに複数行があった場合でも、最終行のデータのみが変数に保存されます。

投稿2017/07/24 05:51

motuo

総合スコア3027

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

SATSUKI.

2017/07/26 09:10

具体的な改善策まで提示してくださり、ありがとうございます。「その他諸々、気になったこと」もたいへん参考になりました。
guest

0

同時に処理しているつもりなのかもしれませんが、ここで最初のファイルしかチェックしていません。

C#

1for (int i = 0; readCSVfile[0].EndOfStream == false; i++) //行を走査

単純にSplitするだけでいい要件なのであれば、Streamを使うよりもFile.ReadAllLines()で一気に読みこんだほうがいいかもしれません。

投稿2017/07/25 08:49

x_x

総合スコア13749

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

SATSUKI.

2017/07/26 09:11

File.ReadAllLines()というメソッドがあるのですね、初めて知りました。使わせていただきます。
guest

0

ベストアンサー

こんにちは。

終端に到達したStreamでReadLineを呼ぶとnullが返るので、nullに対してsplitを呼んでいるのが直接的な原因です。

ちなみに、C#のStreamはReadLineメソッドを呼ぶ度に次の行にシークするので、

csharp

1for (int n = 0; n < 8; n++) //ファイルを8個同時に開いて計算する 2{ 3 for (int s = 0; s < readCSVfile[n].ReadLine().Split(',').Length; s++) //列を走査 4 { 5 dataPixelLine[n, s] = String.Copy(readCSVfile[n].ReadLine().Split(',')[s]); //一行分の文字列をカンマで区切って一列ずつstring型2次元配列に格納 -------ここでNullReferenceExceptionが出る------- 6 } 7}

この部分のReadLineについて、内部のforループで条件判定を行う度に一行データを読み飛ばしている、カンマでSplitした一つを読む度に残りのデータを捨てているため、意図した動作にはなっていないと思います。

投稿2017/07/24 06:16

tamoto

総合スコア4103

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

SATSUKI.

2017/07/26 09:10

ReadLineを呼び出すたびに次の行に遷移してしまい、意図したのとは違うタイミングでstreamが終端に到達していたのですね。原因がわかってよかったです。ありがとうございます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問