Unzip problem with images

[sham]

Hi to all,

I am using ICSharpCode.Sha rpZipLib.Zip.

My zip file contains a number of images which when they get unzipped only
contain a partial image. The filesize looks correct.

The unziping is done below it is not clear where the problem is. For int
numberOfBytesTo Read = (int) zipEntryObject. Size; I initially had the
int numberOfBytesTo Read = (int) readStream.leng th but this always gave me
0.

foreach (ZipEntry zipEntryObject in zipFileObject)
{
System.IO.Strea m readStream =
zipFileObject.G etInputStream(z ipEntryObject);
int numberOfBytesTo Read = (int) zipEntryObject. Size;

byte[] bytes = new byte[numberOfBytesTo Read];

readStream.Read (bytes, 0, numberOfBytesTo Read);
System.IO.FileS tream outputStream =
new System.IO.FileS tream(@Director y + zipEntryObject. Name,
System.IO.FileM ode.Create);

outputStream.Wr ite(bytes, 0, numberOfBytesTo Read);
outputStream.Fl ush();
outputStream.Cl ose();

}

Sham.

[Marc Gravell]

When reading streams, the better approach (and it does it in the ziplib
docs) is to iterate under Read returns <=0; Read is only guaranteed to
return *some* data; I'm guessing this is the problem. You can also save on
the memory footprint by only needing a small buffer, rather than the entire
file size...

The following function (and override) correctly copiesbetween two streams;
use it instead.
Oh, and consider "using" the streams so that they get disposed instead of
finalised - but since you are closing them this will only make a (genuine)
difference when exceptions are thrown.

Marc

/// <summary>
/// Read all data from one stream into another, in user-defined chunks
/// </summary>
/// <param name="input">St ream to read from</param>
/// <param name="output">T arget stream</param>
/// <param name="bufferSiz e">Chunk size (bytes)</param>
/// <returns>The total number of bytes copied</returns>
private long CopyStream(Stre am input, Stream output, int bufferSize) {
byte[] buffer = new byte[bufferSize];
int bytesRead;
long totalBytes = 0;
while ((bytesRead = input.Read(buff er, 0, bufferSize)) > 0) {
output.Write(bu ffer, 0, bytesRead);
totalBytes += bytesRead;
}
output.Flush();
return totalBytes;
}

/// <summary>
/// Read all data from one stream into another
/// </summary>
/// <param name="input">St ream to read from</param>
/// <param name="output">T arget stream</param>
/// <returns>The total number of bytes copied</returns>
private long CopyStream(Stre am input, Stream output) {
return CopyStream(inpu t, output, 1024);
}