MSDN C # DeflateStream пример проблемы с кодом - PullRequest
Новая
       62

MSDN C # DeflateStream пример проблемы с кодом

0 голосов
/ 22 февраля 2009

Есть идеи, почему в приведенном ниже примере нам нужно добавить 100 (buffer.Length + 100)? buffer.Length должен быть таким же, как длина распакованного буфера, поэтому нет необходимости добавлять еще 100. : -)

http://msdn.microsoft.com/en-us/library/bc2dbwea.aspx

спасибо заранее, George

Ответы [ 2 ]

3 голосов
/ 22 февраля 2009

Я не знаю, почему они это делают, но в целом это не очень хороший пример - например, вместо этого они используют явные вызовы Close (не в блоке finally) вместо FileStream использования оператора using, а также предположения, что один вызов Read прочитает весь файл. Черт, даже чтение всего файла в буфер - плохая идея - было бы лучше скопировать порцию за раз прямо в DeflateStream.

Эти вещи наводят меня на мысль о том, что не стоит беспокоиться об этом конкретном нечетном бите кода (дополнительные 100 байт) в этом примере. Конечно, не рассматривайте это как "лучшую практику".

Примеры MSDN, как правило, лучше, чем это, хотя есть много других странных примеров.

РЕДАКТИРОВАТЬ: Перечитав код и ответ CodeMelt, ему действительно нужны дополнительные 100 байтов - но только потому, что ReadAllBytesFromStream реализован так плохо. Вот лучшая реализация, которая всегда просит поток заполнить оставшуюся часть буфера: 

public static int ReadAllBytesFromStream(Stream stream, byte[] buffer) 
{
    int offset = 0;
    int bytesRead;
    while ((bytesRead = stream.Read(buffer, offset, 
                                    buffer.Length-offset)) > 0) 
    {
        offset += bytesRead;
    }
    return offset;
}

(Обратите внимание, что для offset и totalCount не было необходимости, поскольку они всегда имели одинаковое значение.)

2 голосов
/ 22 февраля 2009

Он запрещает stream.Read (buffer, offset, 100) превышать его длину в следующем методе, потому что поток продолжает читать, пока ничего не читает. 

public static int ReadAllBytesFromStream(Stream stream, byte[] buffer) 
{
// Use this method is used to read all bytes from a stream.
int offset = 0;
int totalCount = 0;
    while (true) 
    {
    // even it reads to the end, but it will still read the next
    // 100 bytes to see if anything has been read.
    int bytesRead = stream.Read(buffer, offset, 100); 
        if ( bytesRead == 0) 
        {
        break; 
        }
offset += bytesRead;
totalCount += bytesRead; 
    }
return totalCount;
}
...