C# と VB.NET の質問掲示板

■No76429 (ゆーきゃん さん) に返信
> ストリームを閉じていなくても特に不具合等確認できなかったのですが、
> 閉じないと不具合等が何か起こる場合があるということですね。

それでは実験。
ボタンとリストボックスを 2 つずつ用意し、下記を実行してみて下さい。

button1 は、data1.txt のストリームを Close していません。
button2 は、data2.txt を ReadAllLines で読み取っているコードです。


private void button1_Click(object sender, EventArgs e)
{
    listBox1.BeginUpdate();
    listBox1.Items.Clear();

    var stm = File.OpenText(@"c:\data\data1.txt");
    string line;
    while ((line = stm.ReadLine()) != null)
    {
        listBox1.Items.Add(line);
    }
    // stm.Close(); // 実験のため、あえて Close も Dispose も呼ばない

    listBox1.EndUpdate();
}

private void button2_Click(object sender, EventArgs e)
{
    var lines = File.ReadAllLines(@"c:\data\data2.txt");
    listBox2.DataSource = lines;
}

それぞれのボタンを押した後、エクスプローラーで、
data1.txt と data2.txt を、それぞれ別の名前に
リネームしてみてください。

閉じ忘れている data1.txt の方は、リネームに失敗してしまうかと思います。
(アプリを終了するなどすれば、リネームできるようになりますが)



> また、File.ReadAllLines 、File.ReadLines メソッドについても調べさせていただきます。
これらのメソッドでは、ファイルの開閉が自動的に行われますので、
Close を呼び出す必要がありません（というか呼び出せない）。


ReadAllLines メソッドは、ファイルの内容すべてを一度に読み取り、即座に閉じる機能。
その他の ReadAll 系メソッドも同様に、ファイル全体を一度に読み取り、即座に閉じます。


一方 ReadLines メソッドは、それを呼び出しただけではファイルは開かれません。(遅延実行)
呼び出した結果を foreach で列挙したり、上記のように .DataSource に割り当てたりすると
その時初めてファイルが開かれ、そして列挙が終了した時点で自動的に閉じられる仕組みです。


> すいません。記入ミスしていました
> string[] buzai = File.ReadAllLines(@"c:\data\data.txt");
> でした。
そうすると、StreamReder で開いていた
　StreamReader tanbu = File.OpenText(@"c:\data\data.txt");
の方は不要になりますね。

■No76430 (魔界の仮面弁士 さん) に返信
> ■No76429 (ゆーきゃん さん) に返信
> 
> それでは実験。
> ボタンとリストボックスを 2 つずつ用意し、下記を実行してみて下さい。
> 
> button1 は、data1.txt のストリームを Close していません。
> button2 は、data2.txt を ReadAllLines で読み取っているコードです。
> 
> 
> private void button1_Click(object sender, EventArgs e)
> {
>     listBox1.BeginUpdate();
>     listBox1.Items.Clear();
> 
>     var stm = File.OpenText(@"c:\data\data1.txt");
>     string line;
>     while ((line = stm.ReadLine()) != null)
>     {
>         listBox1.Items.Add(line);
>     }
>     // stm.Close(); // 実験のため、あえて Close も Dispose も呼ばない
> 
>     listBox1.EndUpdate();
> }
> 
> private void button2_Click(object sender, EventArgs e)
> {
>     var lines = File.ReadAllLines(@"c:\data\data2.txt");
>     listBox2.DataSource = lines;
> }
> 
> それぞれのボタンを押した後、エクスプローラーで、
> data1.txt と data2.txt を、それぞれ別の名前に
> リネームしてみてください。
> 
> 閉じ忘れている data1.txt の方は、リネームに失敗してしまうかと思います。
> (アプリを終了するなどすれば、リネームできるようになりますが)
> 

なるほど。
確かに症状の方を確認できました。
.Close()の重要性が分かりました。



> これらのメソッドでは、ファイルの開閉が自動的に行われますので、
> Close を呼び出す必要がありません（というか呼び出せない）。
> 
> 
> ReadAllLines メソッドは、ファイルの内容すべてを一度に読み取り、即座に閉じる機能。
> その他の ReadAll 系メソッドも同様に、ファイル全体を一度に読み取り、即座に閉じます。
> 
> 
> 一方 ReadLines メソッドは、それを呼び出しただけではファイルは開かれません。(遅延実行)
> 呼び出した結果を foreach で列挙したり、上記のように .DataSource に割り当てたりすると
> その時初めてファイルが開かれ、そして列挙が終了した時点で自動的に閉じられる仕組みです。
> 

基本的には、File.OpenTextより、ReadAllLines メソッド等を使った方が間違えにくそうですね。とても参考になりました。
ありがとうございます。