c#におけるFileStreamの記述について

Question

前提・実現したいこと

Windows XPや Windows10上において計測プログラムがログを出しながら動いており、
そのログを用いて1秒間隔でグラフを描画したいと考えております。

発生している問題・エラーメッセージ

ただ、そのログを作成しているプログラム上から参照しようとすると、OpenFileDialogを開き、選択ボタンを押した段階で「別のプロセスが使用している」とのエラーが出ます。
拙いコードなのは重々承知なのですが、間違えている点はどのあたりにあるのでしょうか？？
あるいはほかに良い方法があるのでしょうか？？

もしよろしければ、ご教授お願いしたいです。
よろしくお願いいたします。

該当のソースコード

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.IO;
using System.Timers;
using System.Windows.Forms;
using System.Windows.Forms.DataVisualization.Charting;

namespace DigitShowBasic_Monitor
{
    public partial class Form1 : Form
    {
        String FilePass;
        String[] InitialCol;
        private int XColNum;
        private int YColNum;

        public Form1()
        {
            InitializeComponent();
            XMax_NUD.Value = 100;
            XMin_NUD.Value = 0;
            YMax_NUD.Value = 10;
            YMin_NUD.Value = -10;

            

        }

        private void FileChange_Button_Click(object sender, EventArgs e)
        {
            OpenFileDialog ofDialog = new OpenFileDialog();
            ofDialog.InitialDirectory = @"C:";
            ofDialog.Title = "Select output file";
            ofDialog.Filter = "Output Files (*.dat, *.out, *.vlt)|*.dat;*.out;*.vlt";
            ofDialog.RestoreDirectory = true;


            if (ofDialog.ShowDialog() == DialogResult.OK )
            {
                FilePass = ofDialog.FileName;
                FileName.Text = ofDialog.SafeFileName;
            }
            ofDialog.Dispose();

            if (FilePass != null)
            {
                Update_ChartControls();
            }
         
        }

        private void Update_ChartControls()
        {
            FileStream fs = new FileStream(FilePass, FileMode.Open, FileAccess.Read, FileShare.Read);
            StreamReader sr = new StreamReader(fs);
            InitialCol = sr.ReadLine().Split();
            XAxis_Combo.Items.Clear();
            YAxis_Combo.Items.Clear();

            sr.Close();

            for (int i = 0; i < InitialCol.Count() - 1; i++)
            {
                XAxis_Combo.Items.Add(InitialCol[i]);
                YAxis_Combo.Items.Add(InitialCol[i]);
            }
            XAxis_Combo.SelectedIndex = 0;
            YAxis_Combo.SelectedIndex = 1;
        }

        private void StartDrawing_button1_Click(object sender, EventArgs e)
        {
            if (FilePass != null)
            {
                timer1.Start();
            }
        }

        private void timer1_Tick(object sender, EventArgs e)
        {
            chart1.Series.Clear();
            chart1.ChartAreas.Clear();

            Series seriesLine = new Series();
            seriesLine.ChartType = SeriesChartType.Line;

            FileStream fs = new FileStream(FilePass, FileMode.Open, FileAccess.Read, FileShare.Read);
            StreamReader sr = new StreamReader(fs, Encoding.UTF8);

            string[] Data = new string[0];

            int j = 1;
            while (sr.Peek() > -1)
            {
                Array.Resize(ref Data, j + 1);
                Data[j] = sr.ReadLine();
                j++;
            }

            sr.Close();

            int NumLines = Data.Length;
            var Step = 1;

            XColNum = XAxis_Combo.SelectedIndex;
            YColNum = YAxis_Combo.SelectedIndex;

            if (NumLines >= 2000)
            {
                Step = NumLines / 1000;

            }
            else
            {
                Step = 1;
            }

            for (int i = 2; i < NumLines; i += Step)
            {
                var x = float.Parse(Data[i].Split()[XColNum]);
                var y = float.Parse(Data[i].Split()[YColNum]);
                DataPoint dp = new DataPoint((double)x, y);
                seriesLine.Points.Add(dp);
            }
            ChartArea ChartArea1 = new ChartArea();

            chart1.ChartAreas.Add(ChartArea1);
            chart1.Series.Add(seriesLine);

            ChartArea1.AxisX.Maximum = (double)XMax_NUD.Value;
            ChartArea1.AxisX.Minimum = (double)XMin_NUD.Value;
            ChartArea1.AxisY.Maximum = (double)YMax_NUD.Value;
            ChartArea1.AxisY.Minimum = (double)YMin_NUD.Value;
        }

        private void Button2_Click(object sender, EventArgs e)
        {
            timer1.Stop();
        }
    }
}

試したこと

こちらの方法を試して、上記のようにFileStreamの引数を変えてみましたが、同じエラーが発生します。
またこちらの方の記述を見ると、FileStreamの引数は少なくてもいいように思われます。

補足情報（FW/ツールのバージョンなど）

Visual Studio 2019 .Net Framework 4.0 c#

追記

ログを作成するプログラムは以下のようになっていました。

fopen_s(&FileSaveData1, (LPCSTR)pFileName1 , "w" );
fprintf(FileSaveData1,"%s	","Time(s)");

Answer 1

そのログを作成しているプログラム上から参照しよう

ログ作成の方のコードはどうなっているでしょうか? そちらが、オープンしたままだと、読む方での対応ではどうしようもありません。

FileStream Class (日本語、怪しいですが) にFileStream の記述がありますが、機能が多すぎて覚えきれないですね。

Comments

[Kashi_wamochi]

>pepperleafさん
回答ありがとうございます。ログの方のプログラムの方を確認してみます。

[pepperleaf]

"w"で　openしているという事は、上書きですね。
同じプログラム内という事なら、わざわざ、ファイルから、読まずに、プログラム内で、データを置いて、それを使いまわした方が早いと思いますが、、。
また、ログファイルと言うと、実行中でも参照できるように、書込みしたら、すぐにクローズする等しないと、他から、参照できません。
(その時は、"w" じゃなくて、"a"オープンですね)

Answer 2

FileShare.ReadWriteを指定しても読み込めないのであれば、書き込む側を変更するしかないかと思います。

ログの出力側でfopen_sで開いているとのことですが、こちらに

Files that are opened by fopen_s and _wfopen_s are not sharable. If you require that a file be sharable, use _fsopen, _wfsopen with the appropriate sharing mode constant—for example, _SH_DENYNO for read/write sharing.

とあります。（日本語訳があやしいので、原文で引用しました）
簡単に言うとfopen_fで開いたファイルは共有できないので、共有したい場合は_fsopenを使いましょうとのことです。

fprintfで書き込んでも即時にファイルに反映されないので、必要に応じてfflushするなりしてください。

Comments

[Kashi_wamochi]

>YAmaGNZさん
回答ありがとうございます。他の既成ソフトで開こうとしても同じメッセージが出たため、ご指摘の通りかなと思います...

[Kashi_wamochi]

確認したところC++で書かれていました。
一応上記に該当すると思われる箇所のコードを添付しました。

Answer 3

FileStream（だけでなくIDisposable実装してるクラスなら一般的に）使ったらDisposeするなりusingブロック使うなりしないとダメな気が。

---

追記
とりあえず

-           FileStream fs = new FileStream(FilePass, FileMode.Open, FileAccess.Read, FileShare.Read);
-           StreamReader sr = new StreamReader(fs);
+           using(FileStream fs = new FileStream(FilePass, FileMode.Open, FileAccess.Read, FileShare.Read))
+           using(StreamReader sr = new StreamReader(fs)) {
                InitialCol = sr.ReadLine().Split();
+           }
             XAxis_Combo.Items.Clear();
             YAxis_Combo.Items.Clear();
    
-            sr.Close();

こんな感じで。（もう一箇所も）

あと、どうでもいいですけどFilePassはFilePathでは。

Comments

[Kashi_wamochi]

返信ありがとうございます。恥ずかしながら、IDisposableについて全く無知でした...明日もう一度トライしてみます！

[Zuishin]

StreamReader をクローズしたらファイルの方もクローズされるんじゃありませんでしたっけ？
タイマーのコールバックが怪しそうです。

[gentaro]

>Zuishinさん
そう思うのですが、質問文に「OpenFileDialogを開き、選択ボタンを押した段階」とあるため、ちょっと不思議な感じはしてます。とりあえず両方直してみたらどうかな、と。

[Zuishin]

どのみちそこは確かに直した方がいいですね。

[Kashi_wamochi]

>Zuishinさん、Gentaroさん
ご回答ありがとうございます。コードを指摘の通りusingステートメントを用いて書き換えましたが、同じタイミングで同じエラーメッセージが発生しました。
試しにメモ帳やNotepad++で開こうとしたところ、「別のプロセスを使用中です」とのエラーが出たため、ほかの方が仰られている通り、ログ作成の方のプログラムが怪しいと感じています。