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

開発環境:VCS2008
言語:C#

いつも参考にさせていただいてます。Dotです。

FreeMemoというフォームを開いたときにその実行アプリフォルダ内にあるFreeMemo.iniを読み込み、
FreeMemoを閉じた時にFreeMemo.iniにその時、FreeMemoBoxに書かれていた内容を保存したいのですが、
保存がうまくいきません。どこが間違っているか分からないのでご教授お願いします。


 private void FreeMemo_Load(object sender, EventArgs e)
        {
            string AppliPass = Application.StartupPath;
            string FreeMemoText = AppliPass + "\\FreeMemo.ini";
            FreeMemoBox.Text = FreeMemoText;
            string line, str = "";
            StreamReader sr = new StreamReader(FreeMemoBox.Text, Encoding.Default);
            if (File.Exists(FreeMemoBox.Text))
            {
                while ((line = sr.ReadLine()) != null) //テキストファイルを一行づつ読み込む
                {
                    str = str + line + "\r\n";
                }
                FreeMemoBox.Text = str;
            }
            sr.Close();
        }
        private void Form1_Closing(object sender, System.ComponentModel.CancelEventArgs e)
        {
            string AppliPass = Application.StartupPath;
            string FreeMemoText = AppliPass + "\\FreeMemo.ini";
            StreamWriter sr = new StreamWriter((new FileStream(FreeMemoText, FileMode.Create)), Encoding.Default); //書込みモード
            sr.Write(FreeMemoBox.Text);
            sr.Close();
        }

■No42755 (Dot さん) に返信

> 理解力不足で申し訳ありません。
> やってみたところ、write-openに失敗してるようです。

理解力不足というか…
「write-openに失敗している」というのはどうやって判断しましたか？
意図した形でファイルに書き込まれていないから？
そのあたりを書いてもらわないと、回答者側がコードを元に自分で実験して
試してみる等の手間が必要になります。
質問者側は回答者側にそういった情報を提供する必要があるんじゃないかな。
επιστημηさんが
>だから調べてよ。write-openに失敗? それとも書き込みに失敗?
と書かれているのも、つまりはそういうことなわけで。

> 追記：
> Fom1_Closing() に飛び込んでる?
> 
> すいません、書き込みﾐｽです。
> 正しくは
> FreeMemo_Closing
> でした。

FreeMem_Closing だろうが Form1_Closing だろうがどっちでもいいのだけれど。
これも意図としては「正しく Closing 処理を実行していますか？」ということですね。
write-open に失敗したとあるので、Closing 処理（というかイベント）を実行している
のではないかと思いますが、これまた疑問。そこンところを書いてもらわないと。
もしかしたら単に FreeMem_Closing メソッドが Closing イベントに結びついていなかった
だけということも考えられます。

あと、せっかくなので少しコードを見てみました。

>  private void FreeMemo_Load(object sender, EventArgs e)
>         {
>             string AppliPass = Application.StartupPath;

Appli "Pass" ってなんでしょ。StartupPath は Path ってなってますよ。パスワードのパスとキャリアパスのパスは違うパス…ってわかってますか？

>             string FreeMemoText = AppliPass + "\\FreeMemo.ini";

パス名とファイル名をつなげるときは Path.Combine を使うようにするといいです。

>             FreeMemoBox.Text = FreeMemoText;

あとでファイルから読み込んだ内容で FreeMemoBox.Text を書き換えるんだから、これ不要じゃないですか？

>             string line, str = "";
>             StreamReader sr = new StreamReader(FreeMemoBox.Text, Encoding.Default);
>             if (File.Exists(FreeMemoBox.Text))

ここで FreeMemoBox.Text をファイル名として使っているのが間違い。FreeMemoText があるじゃないですか。
あと、File.Exists と StreamReader の順番が逆のような気がしますよ。ファイルを開いてから存在確認するのはおかしいかと。

>             {
>                 while ((line = sr.ReadLine()) != null) //テキストファイルを一行づつ読み込む
>                 {
>                     str = str + line + "\r\n";

string クラスに足していくのはあまり利口ではないです。こういう時は StringBuilder クラスが便利。
その前に書き込むときは StreamWriter.Write 一発だけど、読むときはなぜ ReadLine で一行ずつ読んでるんでしょう？

>                 }
>                 FreeMemoBox.Text = str;
>             }
>             sr.Close();
>         }
>         private void Form1_Closing(object sender, System.ComponentModel.CancelEventArgs e)
>         {
>             string AppliPass = Application.StartupPath;
>             string FreeMemoText = AppliPass + "\\FreeMemo.ini";
>             StreamWriter sr = new StreamWriter((new FileStream(FreeMemoText, FileMode.Create)), Encoding.Default); //書込みモード
>             sr.Write(FreeMemoBox.Text);
>             sr.Close();
>         }