[
{
"Id": "158942",
"ThreadId": "47472",
"Html": "Hi,
\r\nThanks for the last message and it seems to work wonderful. The last hurdle i have is that once i have extracted the file, i would like to delete the original zip file which seems to be locked by ZipFile for some reason. I have tried Dispose, and setting it as null but neither seems to work.
\r\n
\r\nPlease help.
\r\n
\r\n
\r\n\r\n\r\nIonic.Zip.ZipFile zip = Ionic.Zip.ZipFile.Read(zipName);
\r\n//zip.ExtractProgress
\r\nzip.ExtractProgress += new EventHandler<ionic.zip.extractprogresseventargs>(zip_ExtractProgress);
\r\nzip.ExtractAll(targetDir, true);
\r\nzip.Dispose();
\r\nzip = null;
\r\n\r\n",
"PostedDate": "2009-02-18T04:51:19.823-08:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "159230",
"ThreadId": "47472",
"Html": "Alag, I don't see in that code where you are trying to delete the file. what do you mean by "seems to be locked." What happens when you run your code? Do you get an exception? where?
\r\n
\r\nI coded up a test of my own that
\r\n - creates a zip file
\r\n - reads the file and extracts all entries
\r\n - deletes the original zip file
\r\n
\r\nMy code works. No exceptions.
\r\n// CreateZipSimple.cs\r\n// ------------------------------------------------------------------\r\n//\r\n// Test out create/read+Extract/delete of a zip file. \r\n// \r\n// Author: Dinoch\r\n// built on host: DINOCH-2\r\n// originally created Wed Feb 18 17:54:44 2009\r\n//\r\n// last saved: \r\n// Time-stamp: <2009-February-18 18:23:49>\r\n// ------------------------------------------------------------------\r\n//\r\n// compile with:\r\n// csc /debug+ /target:exe /r:Ionic..Zip.dll /out:CreateZipSimple.exe CreateZipSimple.cs \r\n//\r\n//\r\n// Copyright (c) 2008 by Dino Chiesa\r\n// All rights reserved!\r\n// \r\n//\r\n// ------------------------------------------------------------------\r\nusing System;\r\nusing System.IO;\r\nusing Ionic.Zip;\r\n\r\n\r\nnamespace Ionic.Zip.Samples\r\n{\r\n\r\n public class CreateZipSimple\r\n {\r\n // ctor\r\n public CreateZipSimple () {}\r\n\r\n // ctor\r\n public CreateZipSimple (string[] args)\r\n {\r\n for (int i=0; i < args.Length; i++)\r\n {\r\n switch (args[i])\r\n {\r\n default:\r\n if (DirToZip != null)\r\n {\r\n Usage();\r\n return;\r\n }\r\n DirToZip = args[i];\r\n break;\r\n }\r\n }\r\n }\r\n\r\n string TargetDir = null;\r\n string ZipFileToCreate = null;\r\n string DirToZip = null;\r\n\r\n private void Setup()\r\n {\r\n if (DirToZip== null) { Usage(); System.Environment.Exit(1);}\r\n TargetDir = String.Format("unpack.{0}", System.Diagnostics.Process.GetCurrentProcess().Id);\r\n ZipFileToCreate = "CreateZipSimple.zip";\r\n RemoveZipFile();\r\n }\r\n\r\n\r\n private void RemoveDirectory()\r\n {\r\n Console.WriteLine("=======================================================");\r\n Console.WriteLine("deleting {0}...", TargetDir);\r\n if (Directory.Exists(TargetDir))\r\n {\r\n Directory.Delete(TargetDir, true);\r\n }\r\n else\r\n Console.WriteLine("directory '{0}' does not exist...", TargetDir);\r\n }\r\n\r\n\r\n\r\n private void RemoveZipFile()\r\n {\r\n Console.WriteLine("=======================================================");\r\n Console.WriteLine("deleting {0}...", ZipFileToCreate);\r\n if (File.Exists(ZipFileToCreate))\r\n {\r\n File.Delete(ZipFileToCreate);\r\n }\r\n else\r\n Console.WriteLine("file '{0}' does not exist...", ZipFileToCreate);\r\n }\r\n\r\n\r\n\r\n private void Run()\r\n {\r\n Setup();\r\n CreateZipFile();\r\n ReadZipFile();\r\n RemoveZipFile();\r\n RemoveDirectory();\r\n }\r\n \r\n\r\n private void CreateZipFile()\r\n {\r\n Console.WriteLine("=======================================================");\r\n Console.WriteLine("zipping dir '{0}' into zipfile '{1}'...", DirToZip, ZipFileToCreate);\r\n\r\n using (ZipFile zip = new ZipFile())\r\n {\r\n // note: this does not recurse directories!
String[] filenames = System.IO.Directory.GetFiles(DirToZip);\r\n foreach (String filename in filenames)\r\n {\r\n Console.WriteLine("Adding {0}...", filename);\r\n zip.AddFile(filename);\r\n }\r\n\r\n zip.Save(ZipFileToCreate);\r\n }\r\n }\r\n\r\n\r\n private void ReadZipFile()\r\n {\r\n Console.WriteLine("=======================================================");\r\n Console.WriteLine("Reading and Extracting zip '{0}' to directory {1}", ZipFileToCreate, TargetDir);\r\n\r\n Ionic.Zip.ZipFile zip = Ionic.Zip.ZipFile.Read(ZipFileToCreate);\r\n //zip.ExtractProgress += new EventHandler<ionic.zip.extractprogresseventargs>(zip_ExtractProgress);
zip.ExtractProgress += zip_ExtractProgress;\r\n zip.ExtractAll(TargetDir, true);\r\n zip.Dispose();\r\n zip = null;\r\n Console.WriteLine();\r\n }\r\n\r\n bool firstReport = true;\r\n\r\n public void zip_ExtractProgress(object sender, ExtractProgressEventArgs e)\r\n {\r\n if (firstReport)\r\n Console.Write("Extract");\r\n Console.Write(".");\r\n firstReport= false;\r\n }\r\n\r\n public static void Usage()\r\n {\r\n Console.WriteLine("CreateZipSimple: test bed for creating zip files.\\n");\r\n Console.WriteLine("Usage:\\n CreateZipSimple <DirToZip> ");\r\n }\r\n\r\n\r\n public static void Main(string[] args)\r\n {\r\n try \r\n {\r\n CreateZipSimple me = new CreateZipSimple(args);\r\n me.Run();\r\n }\r\n catch (System.Exception exc1)\r\n {\r\n Console.WriteLine("Exception: {0}", exc1.ToString());\r\n Usage();\r\n }\r\n }\r\n }\r\n}\r\n\r\n",
"PostedDate": "2009-02-18T18:35:03.153-08:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "159312",
"ThreadId": "47472",
"Html": "Hi Cheeso,
\r\nSorry I didn't realize that i missed the code snippet above. I have the last line as File.Delete(zipName) which throws an exception that says file is currently in use and hence cannot be deleted.
\r\n
\r\nI can try your code again - but that will be in a few hours as i am off to a few meetings as of now:)
\r\n
\r\nUpdate
\r\n----------
\r\nThanks for your help and i upgrade to latest 1.7 as per your comments in the other thread but that still seems to have the same issue
\r\n
\r\nThis has now been resolved - Thank you.
\r\n",
"PostedDate": "2009-02-19T01:45:01.337-08:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "159946",
"ThreadId": "47472",
"Html": "Getting the same issue but with a file move command, actually.
\r\n
\r\nZipFile zip = new ZipFile(downloadedFile);
\r\n foreach (ZipEntry entry in zip)
\r\n {
\r\n if (regexCSV.IsMatch(entry.FileName))
\r\n {
\r\n ImportData(new StreamReader(entry.OpenReader()));
\r\n }
\r\n }
\r\n
\r\ntry
\r\n {
\r\n File.Move(downloadedFile, @"processed\\" + Path.GetFileName(downloadedFile) + DateTime.Now.ToFileTime().ToString());
\r\n }
\r\n catch (System.IO.IOException ioError)
\r\n{
\r\n}
\r\n
\r\nI've verified that the stream is being closed, but it still throws an IOException saying the file is still in use.
\r\n {
\r\n Console.WriteLine("Couldn't copy file to processed folder. File already exists there!");
\r\n }
\r\n
\r\n
\r\n
\r\n\r\n",
"PostedDate": "2009-02-19T16:40:56.537-08:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "159948",
"ThreadId": "47472",
"Html": "Actually using "using()" fixes it. Thanks.
\r\n",
"PostedDate": "2009-02-19T16:47:55.277-08:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "160061",
"ThreadId": "47472",
"Html": "For others who are having similar problem, you can also add zip.dispose(); zip= null; which unlocks the file if you dont want to use "using()".
\r\n",
"PostedDate": "2009-02-20T00:53:45.547-08:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "180413",
"ThreadId": "47472",
"Html": "Hi all,
\r\nI have the same issue under ".NET Compact Framework 2.0 ( Windows Mobile ) ". however the code have no problem under "Full Framework ( Windows ).
\r\n
\r\nif (File.Exists(zippath))
\r\n{
\r\n try
\r\n {
\r\n using (Ionic.Zip.ZipFile f = new Ionic.Zip.ZipFile(path))
\r\n {
\r\n f.ExtractAll(targetDir, true);
\r\n }
\r\n }
\r\n catch (Exception ex)
\r\n {
\r\n MessageBox.Show(ex.Message);
\r\n }
\r\n
\r\n try
\r\n {
\r\n File.Delete(path);
\r\n }
\r\n catch (Exception ex)
\r\n {
\r\n MessageBox.Show(ex.Message);
\r\n }
\r\n}
\r\n
\r\n
\r\n
\r\n",
"PostedDate": "2009-04-16T19:28:44.893-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "180524",
"ThreadId": "47472",
"Html": "+1\r\n",
"PostedDate": "2009-04-17T02:45:33.35-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "180527",
"ThreadId": "47472",
"Html": "+1? What's?
\r\n",
"PostedDate": "2009-04-17T02:52:52.917-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "180988",
"ThreadId": "47472",
"Html": "I think what he is saying is, "I also have this problem on the .NET CF".
\r\n
\r\nI recognize this is a problem, with the DotNetZip library on .NET CF.
\r\nI've been in the process of moving out of my house, and so unable to look into this problem.
\r\nI will get to work on it as soon as possible.
\r\nI'll let you all know what I find.\r\n",
"PostedDate": "2009-04-19T08:49:04.513-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "181156",
"ThreadId": "47472",
"Html": "yes,sorry, uaually when in a forum you post "+1" that means that you agree with the commentary inmediatly above you without posting innecesary words.
\r\n",
"PostedDate": "2009-04-20T00:32:09.043-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "181825",
"ThreadId": "47472",
"Html": "I've found a solution to this problem, well not a "real" solution, only a circunvent
\r\n
\r\ninstead of uncompressing the file directly using somethig similar to this....
\r\n
\r\n ZipFile zip = ZipFile.Read(myZipfile);
\r\n foreach (ZipEntry e in zip)
\r\n {
\r\n e.Extract(path);
\r\n }
\r\n zip.Dispose();
\r\n zip = null;
\r\n
\r\nuse instead
\r\n
\r\n
\r\n FileStream fs2 = new FileStream(myZipfile, FileMode.Open);
\r\n byte[] stream= new byte[fs2.Length];
\r\n fs2.Read(stream, 0, (int)fs2.Length);
\r\n fs2.Close();
\r\n
\r\nand then
\r\n
\r\n ZipFile zip = ZipFile.Read(stream);
\r\n
\r\n foreach (ZipEntry e in zip)
\r\n {
\r\n e.Extract(path);
\r\n }
\r\n zip.Dispose();
\r\n zip = null;
\r\n
\r\n
\r\nand then u magically can delete the file myZipfile
\r\n
\r\nu can use using() if you prefer.
\r\n
\r\nNote that this is not a real solution, only a way to keep the library working while Cheeso finds the real solution . Hope this workaroung helps to find the real problem
\r\n
\r\n
\r\n
\r\n
\r\n
\r\n
\r\n
\r\n
\r\n
\r\n
\r\n
\r\n
\r\n",
"PostedDate": "2009-04-21T08:52:40.113-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "182277",
"ThreadId": "47472",
"Html": "Thanks a lot!!!
\r\nI can delete the zip file after extracting.
\r\n",
"PostedDate": "2009-04-22T05:17:31.853-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "184622",
"ThreadId": "47472",
"Html": "Thanks for the tip. I still have to find a real solution.\r\n",
"PostedDate": "2009-04-28T10:42:16.147-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "185750",
"ThreadId": "47472",
"Html": "I believe this is now fixed in the DotNetZip library for .NET CF.
\r\nIf you are on v1.7, you need v1.7.2.16.
\r\nIf you are on v1.8, you need v1.8.2.11.
\r\n
\r\nLet me know if it is not fixed!
\r\n",
"PostedDate": "2009-04-30T17:39:40.333-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "186442",
"ThreadId": "47472",
"Html": "Seems to work fine in v1.7.2.16.
\r\n
\r\nThanks
\r\n",
"PostedDate": "2009-05-04T01:32:50.713-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
},
{
"Id": "192708",
"ThreadId": "47472",
"Html": "Thanks - this works in 1.7.2.16 onwards.
",
"PostedDate": "2009-05-21T02:28:59.553-07:00",
"UserRole": null,
"MarkedAsAnswerDate": null
}
]