[
{
"Id": "163759",
"ThreadId": "49024",
"Html": "Hi, I've problem when I save compress file on steam, this is a simple code to reproduce the issue:
\r\n
\r\n 'Save to stream
\r\n Using zip As New Ionic.Zip.ZipFile
\r\n zip.AddFileFromString("file.zip", "", System.IO.File.ReadAllText("c:\\temp\\test.html"))
\r\n Dim s As New System.IO.MemoryStream()
\r\n zip.Save(s)
\r\n System.IO.File.WriteAllText("c:\\temp\\test.html_1.zip", System.Text.Encoding.Default.GetString(s.ToArray))
\r\n End Using
\r\n
\r\n 'Save to file
\r\n Using zip As New Ionic.Zip.ZipFile
\r\n zip.AddFile("c:\\temp\\test.html", "")
\r\n zip.Save("c:\\temp\\test.html_2.zip")
\r\n End Using
\r\n
\r\nThe test.html_1.zip seems corrupted and it's also twice bigger then test.html_2.zip..
\r\n",
"PostedDate": "2009-03-03T02:09:32.483-08:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "163875",
"ThreadId": "49024",
"Html": "Hmmmm, yes I would bet you have experienced a few problems!
\r\n
\r\nFirst, what do you expect to get when calling System.Text.Encoding.Default.GetString() using s.ToArray() as the input? The zip data contained within s.ToArray() is not an encoded string. Calling GetString() on it will not produce a usable string; I suspect if you print out the result of that call, you will get a long series of question marks. And then, writing that string (?????????????????????????) out to a file will not produce a valid zip file. It is just a series of question marks.
\r\n
\r\nWhat are you trying to accomplish, with that code? Can you explain in english?
\r\n
\r\nNext - the MemoryStream is disposable, and should be wrapped in a using clause.
\r\n
\r\nThis code will save a zip to a MemoryStream and then writes the content of that MemoryStream to a disk file. Notice there is no attempt to decode the zip file data into a string with a call to Encoding.GetString().
\r\nDim buffer As Byte() = Nothing\r\n' This will not scale very well!\r\nUsing ms As MemoryStream = New MemoryStream\r\n Using zip As ZipFile = New ZipFile\r\n zip.AddItem(NameOfFileOrDirectoryToZip)\r\n zip.Save(ms)\r\n End Using\r\n buffer = ms.ToArray\r\nEnd Using\r\n\r\nUsing output As FileStream = File.Create(archiveName)\r\n output.Write(buffer, 0, buffer.Length)\r\nEnd Using\r\n\r\n
\r\nBut that approach is not very scalable. Suppose you have a zip file that is 1mb in size. Then you will have a 1mb buffer in memory. Large, but not too large. Now suppose you have a 2gb zip file. Will you retain a 2gb buffer in memory? This approach works for small zip files but doesn't scale up well.
\r\nThe DotNetZip library can save zip files into a stream, or it can save to a file on disk. You seem to be doing something in the middle - you write the data to a stream and then write it to a file.
\r\nWhat is it that you really want to do? What are you trying to accomplish?
\r\n
\r\nThere is a third problem in your code, something I don't understand at all. You have this code:
\r\nzip.AddFileFromString("file.zip", "", System.IO.File.ReadAllText("c:\\temp\\test.html"))\r\n\r\n\r\nThis statement adds a new entry into the ZipFile instance; the entry is named "file.zip". And the contents of that entry are obtained from a file called "c:\\temp\\test.html". This does not make sense to me, for a couple reasons. First, you are inserting what is apparently an HTML file into an entry named "file.zip". As above, using the .zip extension does not make it a zip file. This is very confusing to me. The content is still going to be HTML. Second, why use AddFileFromString() when the entry you want to add to the zip file, is in a file already? Why not just call AddFile? This would be more sensible, to me:\r\nzip.AddFile("c:\\temp\\test.html")\r\n\r\n\r\nWith that line of code, you are telling DotNetZip to add a file ("c:\\temp\\test.html") to the ZipFile instance.
\r\nRegarding that call to AddFileFromString(), I will ask the same question I asked above: what are you trying to accomplish?
\r\nHave you read the documentation for DotNetZip? I've tried to be very thorough with it.
\r\n",
"PostedDate": "2009-03-03T07:12:47.873-08:00",
"UserRole": null,
"MarkedAsAnswerDate": null
}
]