.NETでのバイト配列の読み取りと書き込みWindowsフォーム

Question

次のコードは動作しません（Word文書とPDFファイルを使用してテストされます）。

保存されたファイルは正しいアプリケーションを起動しますが、破損していて開けません。

OpenFileDialog openFileDialog1 = new OpenFileDialog(); 
if (openFileDialog1.ShowDialog() != DialogResult.OK) 
    return; 

string filename = openFileDialog1.FileName; 
FileStream stream = File.OpenRead(filename); 
byte[] array = new byte[stream.Length]; 

SaveFileDialog saveFileDialog1 = new SaveFileDialog(); 
saveFileDialog1.FileName = filename; 
if (DialogResult.OK != saveFileDialog1.ShowDialog()) 
    return; 

FileInfo fi = new FileInfo(saveFileDialog1.FileName); 
using (FileStream fs = fi.OpenWrite()) 
{ 
    fs.Write(array, 0, array.Length); 
} 

Answer 1

より良い結果を得るには、ファイルバイト配列メソッドを使用してください。

また、使い捨てオブジェクトを使用している場合はいつでもusingを使用してください。

 byte[] array; 
     string filename; 

     using (OpenFileDialog openFileDialog1 = new OpenFileDialog()) 
     { 
      if (openFileDialog1.ShowDialog() != DialogResult.OK) 
       return; 
      filename = openFileDialog1.FileName; 
      array = File.ReadAllBytes(filename); 
     } 

     using (SaveFileDialog saveFileDialog1 = new SaveFileDialog()) 
     { 
      saveFileDialog1.FileName = filename; 
      if (DialogResult.OK != saveFileDialog1.ShowDialog()) 
       return; 
      File.WriteAllBytes(array); 
     } 

Answer 2

FileStream stream = File.OpenRead(filename); 
byte[] array = new byte[stream.Length]; 

あなたが実際にあなたがちょうどいいサイズに配列を初期化しているが、データを記入していない、バイト単位で読み込むことはありません。つまり、あなたは新しいファイルに0バイトの束を書いています。

代わりにあなたの利点にフレームワークを使用する - Fileクラスはすでに、すべてのバイトを読み取るための静的メソッドがあります。

byte[] array = File.ReadAllBytes(filename); 

あるいは、上記本当にあなただけの使用をやろうとしているものの全範囲であればFile.Copy()。

Answer 3

私の推測では、あなたが空の配列を作成しているということでしょう。

byte[] array = new byte[stream.Length]; 

そして、このライン：あなたが実際にReadingによってすべてのバイトを二番目のデータソースとして、あなたの最初のFILESTREAMを使用することができます

fs.Write(array, 0, array.Length); 

この行を参照してください。