【C#】dialog.FileNameで取得した文字列で作成ファイル名を決定したい

Question

前提・実現したいこと

お世話になります。
aes暗号化した際に出力されるファイル名をdialog.FileNameで取得した文字列で決定したいと考えています。

aes暗号化はできているため、作成ファイルの文字列を自由に設定できるようにする部分をご教授いただけますと幸いです。

試したこと

aes暗号化した際の出力ファイル設定を下記のように試しております。

1. 出力パスを"C:/Users/Documents/test.aes"に設定

→問題なくtestのaesファイルが作成される（ファイル名が可変ではないため要修正）

2 . 出力パスを System.IO.Path.GetFileNameで入力された文字列を読み取り、var aesName = filename.Replace("zip", "aes");で拡張子をaesに変更
→暗号化ファイルが作成されない

文字列自体は1と同じだと思うので、ファイルが作成されない理由が分かりません。

該当コード

//AES暗号化
        private void AesSaveAsButton_Click(object sender, RoutedEventArgs e)
        {
            var dialog = new SaveFileDialog();
            dialog.Filter = "zip|*.zip";
            if (dialog.ShowDialog() == true)
            {
                var filename = System.IO.Path.GetFileName(dialog.FileName);

                var entryName = filename.Replace("zip", "json");
                //var aesName = filename.Replace("zip", "aes");　//作成されないため一旦コメントアウト

                var json = JsonConvert.SerializeObject(_data, Formatting.Indented);
                // JSON 文字列からエンコーディング UTF-8 でバイト列を作成
                var bytes = Encoding.UTF8.GetBytes(json);
                using (var ms = new MemoryStream())
                {
                    // メモリストリーム上にZipArchiveを作成する
                    using (var zipArchive = new ZipArchive(ms, ZipArchiveMode.Create, true))
                    {
                        var entry = zipArchive.CreateEntry(entryName);
                        using (var es = entry.Open())
                        {
                            // エントリにバイナリを書き込む
                            es.Write(bytes, 0, bytes.Length);
                        }
                    }
                    // MemoryStream からバイト列を取得してファイルを作成
                    byte[] zipBytes = ms.ToArray();
                    File.WriteAllBytes(dialog.FileName, zipBytes);
                    Encode(dialog.FileName, "C:/Users/ユーザ名/Documents/test.aes");
                    //Encode(dialog.FileName, aesName); //出力されない
                }
            }
        }
        //AES複合化
        private void AesOpenAsButton_Click(object sender, RoutedEventArgs e)
        {
            var dialog = new OpenFileDialog();
            if (dialog.ShowDialog().Value)
            {
                Decode(dialog.FileName, "C:/Users/ユーザ名/Documents/test.aes");//←こちらも可変にする予定
                var file = dialog.FileName;
                try
                {
                    var reader = new StreamReader(file);
                    var json = File.ReadAllText(dialog.FileName);
                    reader.Close();
                    _data = JsonConvert.DeserializeObject<ObservableCollection<ManagementUnitData>>(json);
                }
                catch (Exception error)
                {
                    if (error.Message != null)
                        Debug.WriteLine(error.Message);
                }
                if (_data != null)
                {
                    DataContext = _data;
                }
            }
        }
        //aes暗号化
        /// <summary>
        /// 共有キー
        /// </summary>
        private const string AES_KEY = "-SENJU MURAMASA-";
        /// <summary>
        /// バッファサイズ
        /// </summary>
        private const int BUFFER_SIZE = 1024 * 4;
        /// <summary>
        /// 共有キービットサイズ
        /// </summary>
        private const int KEY_SIZE = 128;
        /// <summary>
        /// 暗号操作のビットサイズ
        /// </summary>
        private const int BLOCK_SIZE = 128;
        /// <summary>
        /// 暗号化
        /// </summary>
        /// <param name="src">入力ファイルパス</param>
        /// <param name="dst">出力ファイルパス</param>
        static public void Encode(string src, string dst)
        {
            //aes暗号化
            using (AesManaged aes = new AesManaged())
            {
                //AESインスタンスのパラメータ設定
                aes.KeySize = KEY_SIZE;
                aes.BlockSize = BLOCK_SIZE;
                aes.Mode = CipherMode.CBC;
                aes.Key = Encoding.UTF8.GetBytes(AES_KEY);
                //初期ベクターの生成
                aes.GenerateIV();
                //暗号化オブジェクトの生成
                ICryptoTransform ct = aes.CreateEncryptor(aes.Key, aes.IV);
                //出力ファイルストリーム
                using (FileStream outFs = new FileStream(dst, FileMode.Create, FileAccess.Write))
                {
                    //初期ベクター書き込み
                    outFs.Write(aes.IV, 0, aes.IV.Length);
                    //暗号変換のストリーム(暗号化)
                    using (CryptoStream cs = new CryptoStream(outFs, ct , CryptoStreamMode.Write))
                    {
                        //Deflateアルゴリズムを使用したストリーム(圧縮)
                        using (DeflateStream ds = new DeflateStream(cs, CompressionMode.Compress))
                        {
                            //入力ファイルストリーム
                            using (FileStream inFs = new FileStream(src, FileMode.Open, FileAccess.Read))
                            {
                                byte[] buf = new byte[BUFFER_SIZE];
                                for(int size = inFs.Read(buf, 0, buf.Length); size>0; size = inFs.Read(buf, 0, buf.Length))
                                {
                                    //出力ファイルへの書き込み
                                    ds.Write(buf, 0, size);
                                }
                            }
                        }
                    }
                }
            }
        }
        /// <summary>
        /// 複合化
        /// </summary>
        /// <param name="src">入力ファイルパス</param>
        /// <param name="dst">出力ファイルパス</param>
        static public void Decode(string src, string dst)
        {
            // 高度暗号化標準(AES)
            using (AesManaged aes = new AesManaged())
            {
                // AESインスタンスのパラメータ設定
                aes.KeySize = KEY_SIZE;
                aes.BlockSize = BLOCK_SIZE;
                aes.Mode = CipherMode.CBC;
                aes.Key = Encoding.UTF8.GetBytes(AES_KEY);
                // 入力ファイルストリーム
                using (FileStream inFs = new FileStream(src, FileMode.Open, FileAccess.Read))
                {
                    // 初期化ベクター(IV)読込
                    byte[] iv = new byte[aes.IV.Length];
                    inFs.Read(iv, 0, iv.Length);
                    aes.IV = iv;
                    // 複合化オブジェクト生成
                    ICryptoTransform ct = aes.CreateDecryptor(aes.Key, aes.IV);
                    // 暗号変換のストリーム（複合化）
                    using (CryptoStream cs = new CryptoStream(inFs, ct, CryptoStreamMode.Read))
                    {
                        // Deflateアルゴリズムを使用したストリーム（圧縮解除）
                        using (DeflateStream ds = new DeflateStream(cs, CompressionMode.Decompress))
                        {
                            // 出力ファイルストリーム
                            using (FileStream outFs = new FileStream(dst, FileMode.Create, FileAccess.Write))
                            {
                                // 複合化した結果を書き込む
                                byte[] buf = new byte[BUFFER_SIZE];
                                for (int size = ds.Read(buf, 0, buf.Length); size > 0; size = ds.Read(buf, 0, buf.Length))
                                {
                                    outFs.Write(buf, 0, size);
                                }
                            }
                        }
                    }
                }
            }
        }

Comments

[Zuishin]

> "C:/Users/Documents/test.aes"

エスケープしてませんが、これで

> →問題なくtestのaesファイルが作成される（ファイル名が可変ではないため要修正）

なのでしょうか？
まあ本題はディレクトリ名部分をなぜか削除しているところだと思いますが。

[dodox86]

提示のコードが
> var entryName = filename.Replace("zip", "json");
> //var aesName = filename.Replace("zip", "aes");　//作成されないため一旦コメントアウト
...
> var entry = zipArchive.CreateEntry(entryName);
...
> Encode(dialog.FileName, "C:/Users/Documents/test.aes");
> //Encode(dialog.FileName, aesName); //出力されない

などとなっていて、ところどころ利用している変数名が合致していないような気がします。
確認用に変更しているだけかもしれませんが、実はそこを間違っていた、なんてことはないですか。

[gontya]

> まあ本題はディレクトリ名部分をなぜか削除しているところだと思いますが。
仰る通り、ディレクトリを一部消去しておりました。（コード上では入っています。）
下記を編集し追加させていただきます。

"C:/Users/ユーザ名/Documents/test.aes"

[Zuishin]

そうではなく、System.IO.Path.GetFileName を使ってディレクトリ名を削除しているという意味です。
filename は結局このコードでは捨てられていますが、おそらく本番では使われているんでしょう？

[Zuishin]

よく読んでいませんでした。entryname として zip に登録するのに使われているのでディレクトリ名は要りませんね。

[gontya]

>Zuishinさん
アドバイスいただきありがとうございます。
System.IO.Path.GetFileNameの仕様を勘違いしておりました。

ディレクトリ名を削除しないGetDirectoryName()メソッドを使用するべきでした。

[dekaaki]

ちょっと補足です。
回答者さんのソースで拡張子を変更したいことは分かるのですが、単純な置換を使用すると"zip"という文字が含まれるファイル名(例えばZipper.zip)を選択された場合に誤った変換になると思います。
拡張子の変換メソッドが用意されてますのでそちらを使用する方がいいと思います。
https://dobon.net/vb/dotnet/file/changeextension.html

[gontya]

>dekaakiさん
そうですね、汎用的な形にするのであればこちらの方が有効ですね。
補足ありがとうございます！

Answer 1

全体的にやってることがメチャクチャという感じがするのですが？　そこはとりあえず置いといて・・・

ファイルは作成できているけど自分の期待しているフォルダに無いので見つけられないだけだと思います。

Encode(dialog.FileName, "C:/Users/ユーザ名/Documents/test.aes");

これは第 2 引数にフルパスを設定しているので、設定した通りのパスに作成されるはずです。

Encode(dialog.FileName, aesName); //出力されない

こちらは質問のコードの、

var filename = System.IO.Path.GetFileName(dialog.FileName);
var aesName = filename.Replace("zip", "aes");

で aesName が "test.aes" のようになるので、質問者さんが期待している dialog.FileName で指定されるフォルダではなくて、アプリの実行フォルダにファイルが作成されるはずです。

前のスレッド https://teratail.com/questions/319hxpttp0w7nb でも言いましたが、パス名、ファイル名の操作、文字列の操作という基本的なことの知識が足りないと思います。勉強して話が通じる程度には知識を得てください。

Comments

[gontya]

話が通じないと言われてしまったら議論の余地がなくなってしまいますね。
申し訳ないです。学習が足りてないことは自覚しているのでもっと知識を付けてからこのようなツールを使用させていただこうかと思います。

また、回答のほどありがとうございます。
System.IO.Path.GetFileNameメソッドを使用するのが誤りでしたね。
ファイルパス全体を取得できるものと勘違いしておりました。

GetDirectoryName()メソッドを使用すればうまくいきそうなのでこちらを試させていただきます。

[退会済みユーザー]

> GetDirectoryName()メソッドを使用すればうまくいきそうなのでこちらを試させていただきます。

違います。そこがパス名、ファイル名の操作、文字列の操作という基本的なことの知識が足りないところです。知識を得るために Microsoft の公式ドキュメントを読みましょう。

SaveFileDialog を使ってどのようにできるかと言うと、dialog.FileName で例えば "C:/Users/ユーザ名/Documents/test.zip" という文字列が取得できたとすると、その拡張子だけを .zip から .aes に代えてそのパスでファイルを作るということになるはずです。具体的には以下のようにします（拡張子以外には zip という文字列はないことが条件）。

var aesName = dialog.FileName.Replace("zip", "aes");

上のコードで aesName には "C:/Users/ユーザ名/Documents/test.aes" が代入されるので、それを、

Encode(dialog.FileName, aesName);

と設定すればユーザーが SaveFileDialog で選んだフォルダ C:/Users/ユーザ名/Documents に test.aes という名前のファイルが生成されるはずです。

[gontya]

なるほど、新しいメソッドを使わないでも実装の幅は多くあるんですね。
あくまで文字列の操作にすることによって、ファイルアクセス権限のことを考える必要もなくなりそうですね。

ありがとうございます。勉強になりました。

[退会済みユーザー]

アクセス権はまた別の話かと。

このスレッドの本題は、

(1) SaveFileDialog でユーザーが選んだパス (dialog.FileName) に zip ファイルを作成

(2) 作成した zip ファイルを読んできてそれを暗号化し、同じパスで拡張子だけ .aes に代えて別のファイルを作成する

・・・ということで、(2) でのパスの指定ができてなかったということですよね。

先に (1) でファイルが作成できている（＝フォルダ対して必要なアクセス権がある）はずなので、(2) ではもうアクセス権の話は出てこないはずですけど？

[gontya]

上記で述べていたGetDirectoryNameメソッドの話です。

[退会済みユーザー]

> 上記で述べていたGetDirectoryNameメソッドの話です。

GetDirectoryName メソッドをどのように使うつもりだったのか分かりませんが、少なくとも今回のケースでは出番はないはずです。

無理やり出番を作って、例えば dialog.FileName からディレクトリ名を取得して、別途作成するファイル名と連結してパスを組み立てるということはあるかもしれませんが、やはりアクセス権は関係ないはずです。先に zip ファイルが作成できれば、そのディレクトリにはアクセス権はあるはずなので。

Answer 2

まずはその２番の方法でのファイル名を表示させてみよう
ファイル名では許されない文字とかフォーマットになってませんか

Comments

[退会済みユーザー]

回答でないことを回答欄に書くのは止めましょう。