Wie kann ich meinen BinaryWriter optimieren?

Question

Ich arbeite gerade an einem Programm, das Dateien per FTP überträgt. Ich sende die Dateien binär, weil ich mit ASCII keine Sonderzeichen senden kann.

Hier ist meine Zeit Code:

using(BinaryReader bReader = new BinaryReader(srcStream)) 
    using (BinaryWriter bWriter = new BinaryWriter(destStream)) 
    { 
     Byte[] readBytes = new Byte[1024]; 
     for(int i = 0; i < bReader.BaseStream.Length; i += 1024) 
     { 
      readBytes = bReader.ReadBytes(1024); 
      bWriter.Write(readBytes); 
     } 
    } 

Meine Probleme mit diesem Code sind:

1. Es funktioniert wirklich langsam, ist es eine Möglichkeit, zu optimieren?
2. Die Art und Weise, wie ich nach EOF (EndOfFile) frage, scheint sehr seltsam zu sein, gibt es eine andere Eleganz Option?

Thanks a lot: D

Answer 1

Warum verwenden Sie Binary und Binary alle an? Warum fragst du immer wieder nach der Länge? Hier ist eine Methode, die ich ein paar Mal jetzt geschrieben habe:

public static void CopyStream(Stream input, Stream output) 
{ 
    byte[] buffer = new byte[8192]; 
    int read; 
    while ((read = input.Read(buffer, 0, buffer.Length)) > 0) 
    { 
     output.Write(buffer, 0, read); 
    } 
} 

dass ein 8K-Puffer verwendet, aber Sie können das natürlich ändern. Oh, und es wiederverwendet den Puffer, anstatt jedes Mal ein neues Byte-Array zu erstellen, was Ihr Code tun wird :) (Sie müssen das Byte-Array nicht zum Anfang zuweisen - Sie könnten readBytes an dem Punkt deklariert haben Der Anruf an bReader.ReadBytes.)

Answer 2

Ich denke, dass Ihre Leistungsprobleme von zwei Orten kommen. Sie rufen bReader.BaseStream.Length jedes Mal durch die Schleife und Ihr Aufruf an bReader.ReadBytes() reserviert jedes Mal ein neues Byte-Array.

Ich denke auch nicht, dass die Binary und Binary notwendig ist, da Sie nicht ihre Funktionen zum Lesen und Schreiben Typen werden unter Verwendung von anderen als Byte-Arrays, die in dem zugrunde liegenden Strom durch Stream.Read() und Stream.Write() bereits unterstützt werden.

ich dies als tun würde:

byte [] buffer = new byte[1024]; 
int bytesRead; 
while ((bytesRead = srcStream.Read(buffer, 0, buffer.Length)) != 0) 
{ 
    dstStream.Write(buffer, 0, bytesRead); 
}