File.OpenRead（）からストリームを返す

Question

私はASP.Net Webサイトが（this articleに基づいて）ファイルを取得できるようにするWCFサービスを作成中です。私の問題は、ストリームを返すときは空白だということです。

は簡単にするために、私は試してみて、問題がストリームを返すにあるものを見つけるために、単純なリサイズアプリにコードを分離してきたし、これはコードです：

private Stream TestStream() 
    { 
     Stream fs = File.OpenRead(@"c:\testdocument.docx"); 
     return fs; 
    } 

    // This method converts the filestream into a byte array so that when it is 
    // used in my ASP.Net project the file can be sent using response.Write 
    private void Test() 
    {    
     System.IO.MemoryStream data = new System.IO.MemoryStream(); 
     System.IO.Stream str = TestStream(); 

     str.CopyTo(data); 
     byte[] buf = new byte[data.Length]; 
     data.Read(buf, 0, buf.Length);      
    } 

このコードの結果ということですbufは12,587バイト（ファイルの正しい長さ）ですが、0だけを含んでいます。

私は試してみるとWord文書が問題なく開きますが、何か明白なものがありませんか？

Answer 1

あなたが求めることを忘れてしまった：

str.CopyTo(data); 
data.Seek(0, SeekOrigin.Begin); // <-- missing line 
byte[] buf = new byte[data.Length]; 
data.Read(buf, 0, buf.Length); 

Answer 2

はこれにあなたのコードを変更してみてください：

private void Test() 
{    
    System.IO.MemoryStream data = new System.IO.MemoryStream(TestStream()); 

    byte[] buf = new byte[data.Length]; 
    data.Read(buf, 0, buf.Length);      
} 

Answer 3

あなたTest()方法は、それがあるべきクライアントを模倣しているので、あなたが

str.CopyTo(data); 
    data.Position = 0; // reset to beginning 
    byte[] buf = new byte[data.Length]; 
    data.Read(buf, 0, buf.Length); 

を必要とし、 〜Close()またはDispose()str SおしゃべりそしてmemoryStreamも、まさにプリンシパルの外です。

Answer 4

オプション：

• 使用data.Seek ken2k

• により示唆されるように、やや単純Positionプロパティを使用します開始するにはあなたの人生をより簡単にするためにMemoryStreamでToArray呼び出しを使用し

  data.Position = 0; 
  

• をwith：

  byte[] buf = data.ToArray(); 
  

第3の選択肢が私の好みのアプローチです。

あなたは（MemoryStreamのために、必要に応じてと）自動的にファイルストリームを閉じますusing文を持つべき注意、と私はあなたのコードのクリーナーを作るためにSystem.IOのためのusingディレクティブを追加したい：

byte[] buf; 
using (MemoryStream data = new MemoryStream()) 
{ 
    using (Stream file = TestStream()) 
    { 
     file.CopyTo(data); 
     buf = data.ToArray(); 
    } 
} 

// Use buf 

あなたはStreamに拡張メソッドを作成して、1か所でこれを行うこともできます。このがは、入力ストリームを閉じないことを

public static byte[] CopyToArray(this Stream input) 
{ 
    using (MemoryStream memoryStream = new MemoryStream()) 
    { 
     input.CopyTo(memoryStream); 
     return memoryStream.ToArray(); 
    } 
} 

は注意してください。

Answer 5

あなたはメモリストリームの位置をリセットするのを忘れ：

private void Test() 
{    
    System.IO.MemoryStream data = new System.IO.MemoryStream(); 
    System.IO.Stream str = TestStream(); 

    str.CopyTo(data); 
    // Reset memory stream 
    data.Seek(0, SeekOrigin.Begin); 
    byte[] buf = new byte[data.Length]; 
    data.Read(buf, 0, buf.Length);      
} 

更新：

を注意すべきもう一つのことがあります：それは、通常のメソッドの戻り値を無視しないで支払います。より堅牢な実装では、呼び出しが返された後に読み取られたバイト数を確認する必要があります。

private void Test() 
{    
    using(MemoryStream data = new MemoryStream()) 
    { 
     using(Stream str = TestStream()) 
     { 
      str.CopyTo(data); 
     } 
     // Reset memory stream 
     data.Seek(0, SeekOrigin.Begin); 
     byte[] buf = new byte[data.Length]; 
     int bytesRead = data.Read(buf, 0, buf.Length); 

     Debug.Assert(bytesRead == data.Length, 
        String.Format("Expected to read {0} bytes, but read {1}.", 
         data.Length, bytesRead)); 
    }      
}