C#でCSVを生成する際、ユーザがダイアログで指定した場所にうまく保存できません。

Question

概要

下記のようなBuilderを記述の上、ユーザが指定した場所にCSVファイルを保存したいです。下記のようにすると、

C:\Users\[ユーザ名]\Documents\Visual Studio 2013\Projects\[プロジェクト名]\[アプリ名]\bin\Debug

下には保存されるのですが、デスクトップ等ユーザが選択した場所に保存できません。

class CsvBuilder : IDisposable
    {
        /// <summary>
        /// CSVファイルに書き込むストリーム
        /// </summary>
        private StreamWriter stream = null;
 
        /// <summary>
        /// ファイル名を指定して、 <see cref="CsvWriter">CsvWriter</see> クラスの新しいインスタンスを初期化します。
        /// </summary>
        /// <param name="path">書き込む完全なファイルパス。</param>
        public CsvBuilder(string path) :
            this(path, Encoding.Default)
        {
        }
 
        /// <summary>
        /// ファイル名、文字エンコーディングを指定して、 <see cref="CsvWriter">CsvWriter</see> クラスの新しいインスタンスを初期化します。
        /// </summary>
        /// <param name="path">書き込む完全なファイルパス。</param>
        /// <param name="encoding">使用する文字エンコーディング。</param>
        public CsvBuilder(string path, Encoding encoding)
        {
            var stream = new FileStream(path, FileMode.Create, FileAccess.ReadWrite);
            this.stream = new StreamWriter(stream, encoding);
        }
 
        /// <summary>
        /// 使用する文字エンコーディングを取得します。
        /// </summary>
        public Encoding Encoding
        {
            get
            {
                return this.stream.Encoding;
            }
        }
 
        /// <summary>
        /// 現在のストリームで利用される改行文字列を取得または設定します。
        /// </summary>
        public string NewLine
        {
            get
            {
                return this.stream.NewLine;
            }
 
            set
            {
                this.stream.NewLine = value;
            }
        }
 
        /// <summary>
        /// 現在のストリームオブジェクトと基になるストリームをとじます。
        /// </summary>
        public void Close()
        {
            if (this.stream == null)
            {
                return;
            }
 
            this.stream.Close();
        }
 
        /// <summary>
        /// CsvWriter で利用されているすべてのリソースを解放します。
        /// </summary>
        public void Dispose()
        {
            if (this.stream == null)
            {
                return;
            }
 
            this.stream.Close();
            this.stream.Dispose();
            this.stream = null;
        }
 
        /// <summary>
        /// 現在のライターで使用したすべてのバッファーをクリアし、バッファー内のすべてのデータをストリームに書き込みます。
        /// </summary>
        public void Flush()
        {
            this.stream.Flush();
        }
 
        /// <summary>
        /// 現在のライターで使用したすべてのバッファーを非同期的にクリアし、ストリームへ書き込みます。
        /// </summary>
        /// <returns>非同期のフラッシュ操作を表すタスク。</returns>
        public Task FlushAsync()
        {
            return this.stream.FlushAsync();
        }
 
        /// <summary>
        /// ストリームに文字を書き込みます。
        /// </summary>
        /// <typeparam name="T">リストの型。</typeparam>
        /// <param name="data">CSVデータ。</param>
        public void Write<T>(List<List<T>> data, string bankName, string extensiton)
        {
            SaveFileDialog sfd = new SaveFileDialog();
            sfd.FileName = CreateDateTimeFileName(name, extension);
            sfd.InitialDirectory = @"C:\";
            sfd.Title = "ファイルの保存先を指定してください。";
            sfd.Filter = "csv files (*.csv)|*.csv";

            if (sfd.ShowDialog() == DialogResult.OK)
            {
                foreach (var row in data)
                {
                    this.WriteRow<T>(row);
                }
            }
        }
 
        /// <summary>
        /// ストリームに１レコード分の文字列を書き込みます。
        /// </summary>
        /// <typeparam name="T">リストの型。</typeparam>
        /// <param name="row">CSVの１レコード。</param>
        public void WriteRow<T>(List<T> row)
        {
            var sb = new StringBuilder();
 
            foreach(var cell in row)
            {
                var value = cell.ToString();
 
                if (value.Contains(this.NewLine) ||
                    value.Contains(",") ||
                    value.Contains("\""))
                {
                    value = value.Replace("\"", "\"\"");
                    sb.Append("\"");
                    sb.Append(value);
                    sb.Append("\"");
                }
                else
                {
                    sb.Append(value);
                }

                sb.Append(",");
            }
 
            sb.Remove(sb.Length - 1, 1);
         
            this.stream.WriteLine(sb.ToString());
        }
 
        /// <summary>
        /// ストリームに１レコード分の文字列を非同期的に書き込みます。
        /// </summary>
        /// <typeparam name="T">リストの型。</typeparam>
        /// <param name="row">CSVの１レコード。</param>
        /// <returns>非同期の書き込み操作を表すタスク。</returns>
        public Task WriteRowAsync<T>(List<T> row)
        {
            return Task.Factory.StartNew(() =>
            {
                this.WriteRow<T>(row);
            });
        }

        /// <summary>
        /// 日付名のファイル名を返す
        /// </summary>
        /// <param name="extension">拡張子</param>
        /// <returns>拡張子を含んだ、日付のファイル名</returns>
        public static string CreateDateTimeFileName(string bankName, string extension)
        {
            return DateTime.Now.ToString("yyyyMMddhhmmss") + bankName + extension;
        }

    }

・ 使用する側のコードは下記のようなものです。

using (var csvBuilder = new CsvBuilder(CsvBuilder.CreateDateTimeFileName("Name", ".csv")))
    {
        csvBuilder.Write(list, "Name", ".csv");
    }

動作環境

・.NET Frame Work 4.5
・Visual Studio2013

追加で…

Debugフォルダ下に作成されるCSVファイルは削除する処理を入れるべきものでしょうか？開発時の確認にはあった方が便利ですが、延々とたまっていくのをたまにまとめて削除という運用も微妙かと思っています。

よろしくお願いします。

Answer 1

CsvBuilderクラスのコンストラクタを見ると、第一引数には「書き込む完全なファイルパス」を与えることになっていますね？
一方、「CsvBuilder.CreateDateTimeFileName("SMBC_biz", ".csv")」って、タイムスタンプを付加した「ファイル名」しか返さないのではないでしょうか？

ですから、この実装方法だと、FileStream を New した時点では「path」にはファイル名しか代入されていないので、「カレントディレクトリ」にしかファイルが出力されないのだと思います。

argiusさんがご指摘のように、「SaveFileDialog」を使用して出力先ファイルをユーザーが選択できるようにしたいなら、SaveFileDialog は FileStream を New する前に呼び出して、選択したファイルが出力先としてオープンされるようにする必要があります。

ただし、FileNameプロパティではファイル名しか取得できないので、結局はカレントディレクトリへの出力となってしまいます。

それで、MSDNのページに載っている実装例のように
var stream = sfd.OpenFile()
として、実際に選択されたファイルをオープンするのが正しいと思います。

それから、

Debugフォルダ下に作成されるCSVファイルは削除する処理を入れるべきものでしょうか？

についてですが、テスト環境のお掃除処理（＝本番環境では不要）をプログラム内に実装するのはおかしいと思います。

通常は__「テスト環境維持のための日次バッチ」__のような感じでお掃除を実行しているケースが多いのではないでしょうか？
具体的には、Debugディレクトリ配下のCSVファイルを削除するコマンドを、crontabに設定するなどして１回/日 実行すれば良いのではないでしょうか？

Comments

[toshi0607]

ありがとうございます！ユーザが保存場所選択後、FileNameプロパティでフルパスが取得できたのでそれを踏まえてStream処理するようにしました。

お掃除処理に関する知見もありがとうございます！！

Answer 2

ファイル保存ダイアログから、選択されたファイル名を取得してからファイルオープンしないといけないのではないでしょうか。

sfd.FileNameプロパティーでファイル名を取得し、その後でFileStreamをnewする必要があると思います。

参考リンク：
SaveFileDialog クラス (System.Windows.Forms)

Comments

[pi-chan]

argiusさん、せっかく貼り付けてくださった参考リンクですが、MSDNのURLって、URL自体にカッコ "(", ")" を含んでいるので、URL/リンクを表すマークダウン記法と相性が悪いです。（クリックしても目的のページへジャンプできません。）
お手数ですが、URLを「文字列」として貼り付けて頂けませんか？

[argius]

pi-chanさん
ご指摘ありがとうございます。すみません、ちゃんと確認してませんでした...
リンクを修正しました。

[toshi0607]

下記のように修正することで適正に処理できました！ありがとうございました。

```lang-C#
SaveFileDialog sfd = new SaveFileDialog();
sfd.FileName = CsvBuilder.CreateDateTimeFileName("Name", ".csv");
sfd.InitialDirectory = @"C:\";
sfd.Title = "明細ファイルの保存先を指定してください。";
sfd.Filter = "csv files (*.csv)|*.csv";

if (sfd.ShowDialog() == DialogResult.OK)
{
using (var csvBuilder = new CsvBuilder(sfd.FileName))
{
csvBuilder.Write(wtxns);
}
}
```