フォーム展開時のFileStream での FileShare.None（アクセス制限）の維持

Question

前提・実現したいこと

編集用Ｆｏｒｍを開いている間、管理用データ（xmlファイル）へのアクセスを禁止(FileShare.None)したい
20台ほどの処理端末から管理用データにアクセスしているが、排他性の確保から、データ編集時は
アクセス禁止を行いたい

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

1回目は出来ますが、2回目は例外になってしまう

該当のソースコード

public partial class TestClass
{
　　 //変数
     FileStream objFstream;

     public TestForm()
     {
         InitializeComponent();

         string filePath = "\C:Test\testdata.xml"
         
　　　　 //セットする
         objFstream = new FileStream(filePath,FileMode.Open,FileAccess.ReadWrirte,FileShare.None);
     }

　　 //データを読み込む
     private bool DataUpdate()
     {
        Using(DataSet ds = new DataSet())
        {
            ds.ReadXml(objFstream)
        }
     }

　　 private bool DataSave()
     {
        Using(DataSet ds = new DataSet())
        {
            ds.ReadXml(objFstream)　　　//例外が発生してしまう

            *******データ処理*******

            ds.WriteXml(objFstream)
        }
     }     
}

今までは一旦、　

objFstream.Dispose();
A
objFstream = new FileStream(filePath,FileMode.Open,FileAccess.ReadWrirte,FileShare.None);

してましたが、処理用端末からのアクセスが****A*****の部分にあったのか、フリーズしてしまい
やり直しやデータ破損が発生しました。

通常であれば、処理端末の方ではFileShare.Noneの時は例外処理を行っていますが、一旦、ロックが外れた一瞬にアクセスされてしまうと不都合が発生してしまいます

運用で編集時は処理用端末を触らないとか、処理用端末の運用時間外で実施は難しいのが現状です

Comments

[dodox86]

2回目、というタイミングが分かりませんが、編集用FormであるTestFormを開いたまま、DataSave()メソッドを2回呼んだら例外発生、ということでしょうか。

[RC46]

一旦、DataTabelに読込、Datagridviewで表示し、修正箇所を書き換えて、保存します
DataSave()で再度、ds.ReadXml(objFstream)を行い、必要な個所だけ書き換えて、
ds.WriteXml(objFstream)で保存したいのです。
その際の、ds.ReadXml(objFstream)で例外が出てしまいます

Answer 1

フォームを開いている間、他端末からのアクセスを禁止するのであれば、FileStream は閉じてはいけません。

読むときは Position = 0 に、書くときは SetLength(0) とすれば良いです。

using (DataSet ds = new DataSet())
{
    objFstream.Position = 0;
    ds.ReadXml(objFstream);

    // *******データ処理*******

    objFstream.SetLength(0);
    ds.WriteXml(objFstream);
}

書き込む際に異常終了した場合、バックアップから戻すような処理も考えておく必要がありそうです。

Comments

[RC46]

ご教授いただいたコードを挿入記述したら、例外がなくなり思った通りの処理ができました
ありがとうございました

Answer 2

別ファイル名で編集し、編集が終わったらリネーム処理して元のファイルと差し替えたらどうでしょう。
また、FileStreamを保持せず、File.ReadAll系のメソッドで内容を全てメモリに読み込んでしまって、極力ファイルを開いている時間を短くするといいと思います。