[
{
"Id": "168642",
"ThreadId": "50354",
"Html": "Hello,
\r\n
\r\nI am using DotNetZip version 1.7.2.12 for .NET 2.0. When I try to run the operation below, it throws an IOException (especially after the 2nd run). I have tried several things including explicitly closing the file and calling GC collection. This IOException only happens if I run the unzip function. If I comment out the "Using" statement block, it runs successfully. Is the unzip function somehow hanging onto the file? Is my code correct?
\r\n
\r\nThanks in advance,
\r\n
\r\nError:
\r\n{"The process cannot access the file 'D:\\\\test\\\\dest\\\\Temp_Backup\\\\source.zip' because it is being used by another process."}
\r\n
\r\nCode:
\r\n
\r\n\r\n
\r\n\r\nstatic void TestNewUnzipping()\r\n
\r\n{
\r\n
\r\n
\r\nstring src = @"D:\\test\\source\\source.zip";\r\n
\r\n
\r\n
\r\nstring dest = @"D:\\test\\dest\\Temp_Backup\\source.zip";\r\n
\r\n
\r\n
\r\nstring outputFolder = @"D:\\test\\dest\\Temp_Backup\\";\r\n
\r\n
\r\n
\r\nFile.Copy(src, dest, true);\r\n
\r\n
\r\n
\r\nusing (Ionic.Zip.ZipFile zip = Ionic.Zip.ZipFile.Read(dest))\r\n
\r\n{
\r\n
\r\n
\r\nforeach (Ionic.Zip.ZipEntry e in zip)\r\n
\r\n{
\r\n
\r\n
\r\n
\r\n//e.Extract(outputFolder, true);\r\n
\r\n
\r\n\r\ne.ExtractWithPassword(outputFolder,
\r\n\r\ntrue, "1234");\r\n
\r\n}
\r\n
\r\n
\r\n//zip.Dispose();\r\n
\r\n
\r\n\r\n
\r\n\r\n//GC.Collect();\r\n
\r\n
\r\n\r\n}
\r\n
\r\n\r\nFile.Delete(dest);\r\n
\r\n}
\r\n
\r\n",
"PostedDate": "2009-03-16T10:53:01.847-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "170485",
"ThreadId": "50354",
"Html": "In theory what you are doing should work, but there are some possibilities for confusion or errors.
\r\nCan you post the stacktrace that you get with the exception? You have said it is an exception, but you haven't said WHERE you get the exception.
\r\nOn the File.Delete() ? On the ZipFile.Read()? on the File.Copy() ?
\r\n
\r\nWhen you extract files, you appear to be extracting to the same directory that contains the zipfile you are reading.
\r\nIf there is contained within the zipfile, a file with the same name as the one you are reading, then you will likely get an exception.
\r\nIn other words if "source.zip" contains an entry by the name of "source.zip", then the e.Extract() will try to overwrite a file that is in use.
\r\nYou should not have to explicitly call zip.Dispose() nor GC.Collect(). In fact, as a rule, NEVER EVER EVER call GC.Collect(). You likely don't need it. The runtime is almost always smarter than you are about when it needs to do garbage collection!
\r\n
\r\nWhy are you copying the file anyway? Why not just read it from the original location?
\r\nWhy copy the file, only to read and then delete it?
\r\n",
"PostedDate": "2009-03-21T22:53:45.727-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "179520",
"ThreadId": "50354",
"Html": "Same error here, the exception raises on File.Delete(). Im using the Compact Framework version on a PDA
\r\n
\r\n
\r\nZipFile zip = ZipFile.Read(pathFileZIP);
\r\n foreach (ZipEntry e in zip)
\r\n {
\r\n if (password.Length != 0)
\r\n {
\r\n e.ExtractWithPassword(directorio, password); // overwrite == true
\r\n }
\r\n else
\r\n {
\r\n e.Extract(directorio);
\r\n }
\r\n }
\r\n zip.Dispose();
\r\n zip = null;
\r\nSystem.IO.File.delete(pathFileZIP); // IOException The process cannot access the file 'xxxx.zip' because it is being used by another process.
\r\n
\r\nI've tried too with using() and the same exception raises
\r\n",
"PostedDate": "2009-04-15T04:15:13.33-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "179888",
"ThreadId": "50354",
"Html": "Though the exception here is the same as the original message, I don't think it's the same situation. The first message had a zip file that contained a zip file of the same name. It seemed that the extraction was extracting over the original zip file. OR at least there was a chance of that. In any case, the code didn't make a ton of sense to me. I never got a reply.
\r\n
\r\nThe key thing for you is the use of the library on the CF. There is no Dispose() on a Stream in .NET CF. So, when you call ZipFile.Dispose(), it does not translate into a Dispose() on the corresponding stream that is used to read the zip file. This is just the way the compact framework was designed. It seems that the file pointer is not yet closed when you try to delete the file.
\r\n
\r\nWhat I suggest is this: try a back-off interval. Something like this:
\r\nbool retry = true;\r\nwhile (retry)\r\n{\r\n try \r\n {\r\n System.IO.File.Delete(pathFileZIP);\r\n retry = false;\r\n }\r\n catch (Exception e1)\r\n {\r\n retry = WantAnotherRetry();\r\n }
if (retry) Thread.Sleep(delayInterval); \r\n}\r\n\r\nThe criteria for continuing to retry could be, the number of attempts, or a time interval (you'd have to get the time before you start trying to delete, and then subtract it from the time of the current attempt), or something else. Up to you.
\r\n",
"PostedDate": "2009-04-15T18:15:02.82-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "180008",
"ThreadId": "50354",
"Html": "well, I think too is not the same situation but...the title of the discussion is better fitting my issue i think.
\r\n
\r\nWith the back-off interval i finally can delete the file , but at the same time makes the library unusable, it takes at least 40 seconds to delete a 3k zip file.
\r\n
\r\nSo...
\r\nConnecting FTP ..less than 1 second
\r\ndownloading file ..less than 1 second
\r\nextracting files .. less than 0.5 seconds
\r\ndeleting file... 40-60 seconds....???
\r\n
\r\nI'm actually using another library to manage the unzip and i can delete the file inmediatly (but it has other issues xd) . So i think it must be an issue of DotNetzip.
\r\n
\r\nThanks in advance.
\r\n
\r\n
\r\n
\r\n
\r\n
\r\n",
"PostedDate": "2009-04-16T01:33:59.987-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "463822",
"ThreadId": "50354",
"Html": "I had the same problem as diesalher
\r\n
\r\nNow I use:
\r\n
\r\n System::IO::FileStream^ fs = System::IO::File::OpenRead(fileName);
ZipFile^ zip = ZipFile::Read(fs);
for each (ZipEntry^ e in zip)
{
e->Extract(Folder, ExtractExistingFileAction::OverwriteSilently);
}
fs->Close();
\r\n
",
"PostedDate": "2010-07-06T09:00:23.38-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
}
]