System.NullReferenceException発生

Question

visualstudio2017 C#
1秒タイマーでﾌｧｲﾙの更新時刻を比較し任意の場所を抜き出してＵＤＰで送信するプログラムです。
ビルドして動作させておくとしばらくして
下記タイマー部分のコードの”str1 = str.Substring(9, 2);　”部分で
************** 例外テキスト **************
System.NullReferenceException: オブジェクト参照がオブジェクト インスタンスに設定されていません。
場所 udpSR.Form1.timer1_Tick(Object sender, EventArgs e) 場所 C:\Users\xxxx\Documents\Visual Studio 2017\Projects\xxxx\xxxx\Form1.cs:行 220
場所 System.Windows.Forms.Timer.OnTick(EventArgs e)
場所 System.Windows.Forms.Timer.TimerNativeWindow.WndProc(Message& m)
場所 System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
が発生します。
220行が”str1 = str.Substring(9, 2);なので
string str, str1 の初期化場所、内容を変更してみましたが、再発します。
何かおかしな部分はありますでしょうか？

private void timer1_Tick(object sender, EventArgs e)
        {
            string Tpath = ConfigurationManager.AppSettings["Apath"]; //ファイルのフルパス
            string Wzz = ConfigurationManager.AppSettings["Wxx"];     //数値
            int wxy = int.Parse(Wzz);
            string wls = "";
            //DateTime F2 = DateTime.Now;
            DateTime f1 = File.GetLastWriteTime(Apath);
            int i = 0;
            string str="";
            string str1="";

            if (f1 != F1)   //初期F1にはダミーが入っている
            {

                FileStream fs = new FileStream(Apath, FileMode.Open, FileAccess.Read, FileShare.ReadWrite); //ReadWrite指定
                StreamReader sr = new StreamReader(fs, Encoding.GetEncoding("Shift_JIS")); //

                fs.Seek(784,System.IO.SeekOrigin.Begin);    //任意の位置
                str = sr.ReadLine();
                sr.Close();
                fs.Close();

                str1 = str.Substring(9, 2);　　　　//ここでSystem.NullReferenceException
                str1 += str.Substring(6, 2);

                int wl = Convert.ToInt32(str1, 16);
                wl += wxy;
                wls = wl.ToString();

                button1.Enabled = false;

                byte[] sendBytes = Encoding.UTF8.GetBytes(wls);

                textBox4.Text = wls;

                if (udpclient == null)
                {
                    udpclient = new System.Net.Sockets.UdpClient();
                }



                udpclient.BeginSend(sendBytes, sendBytes.Length, textBox2.Text, int.Parse(textBox3.Text), SendCallBack, udpclient);

                F1 = File.GetLastWriteTime(Apath);

            }
        }

Answer 1

fs.Seek(784,System.IO.SeekOrigin.Begin); //任意の位置

ここで，シーク先がファイルサイズを超えた指定になっているのでは？

Comments

[onGu]

ありがとうございます。
1568バイトあります。

[onGu]

読込ﾌｧｲﾙも１秒更新なのも関係するでしょうか？

[fana]

よくわからんですが，「更新」作業中のタイミングで読みに行ったら，1568バイト「無い」状態だったりしないのでしょうか．

[onGu]

エラーの読み込み内容を確認するのが早いですね。

Answer 2

Apath で示されるファイルが 786 バイト以上ありますか？
無いなら ReadLine は null を返すので、str は null となり、null のメソッド(SubString)を呼ぶことで例外が発生します。

例外が起こる位置で F9 を押してブレークポイントを設定し、実行してください。するとその行で停止するので、Apath を確認してください。
確認したらそのファイルが 786 バイト以上あるかどうか確かめてください。

Comments

[onGu]

ありがとうございます。
1568バイトあります。

[onGu]

読込ﾌｧｲﾙも１秒更新なのも関係するでしょうか？

[Zuishin]

関係するんじゃないでしょうか。更新を止めて確かめてください。

[onGu]

ありがとうございます。
運用時には読み込みﾌｧｲﾙの更新は止められないのでエラー出たら何もしないか
タイマーをやめてファイルが更新されたら送信するよにしたほうがいいですね。

Answer 3

strがnullなのではないでしょうか？
（str = sr.ReadLine();がnullなのだと思うので、読み込み出来ていないのでしょうか？）
strがnullとなることが有り得るのなら、str.Substringを行っている2行を実行しない、ということになりますが、その場合、int wl = Convert.ToInt32(str1, 16);もエラーになりますね。
strがnullとなった場合、異常値として処理した方がよいでしょう。

Comments

[onGu]

ありがとうございます。
毎回おこるわけではないので異常処理をどうしてよいものか。

[kenshirou]

対象のファイルがどのようなものか分かりませんが、次のタイマー処理まで待てばstrのnullが解消されるのであれば、strがnullの場合はreturnでtimer1_Tickを抜ければ良いかと。
strがnullとなる原因は何でしょうか？対象ファイルの更新タイミングと重なった時とか？

[onGu]

ありがとうございます。
タイマーをやめてファイルの更新を監視して実行するコードに変えたところエラーが頻発するようになりました。
読むﾌｧｲﾙの更新とコードで開くタイミングが重なっていると思い、開く前にsleepをいれたところ
解消されました。（まだ短時間ですが）これで様子をみます。

Answer 4

まずその例外が発生してるところで実行を止め、strになにが入ってるのかを確認しましょう。

Comments

[onGu]

ありがとうございます。
例外が発生するまでデバッグで動かしておくのが一番早そうですね

[y_waiwai]

例外の出る行の左側でダブルクリックしたらブレークポイント設定できてそこで実行止まります

[onGu]

ありがとうございました。