Why APIs such "AddEntry" do no try own Exceptions?
\r\nIf something goes wrong while I'm adding elements to a ZipFile, I would like to know what is the element that raise the Exception.
\r\nNow, instead I have to implement a my procedure to check the specific element in the "catch (...)"
\r\n\r\n", "PostedDate": "2010-11-05T08:53:44.667-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "517800", "ThreadId": "233647", "Html": "\r\n
I'm having trouble with your English syntax.
\r\nI don't understand what you mean by "try own Exceptions".
\r\nYou're referring to a design decision in DotNetZip regarding exceptions, but I don't know exactly what the decision or behavior is. Also I don't understand what problem this behavior causes you.
\r\n\r\n", "PostedDate": "2010-11-05T10:14:07.933-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "517822", "ThreadId": "233647", "Html": "\r\n
"try API (of the ZipNet classes) own exception. Sorry for the mistake.
\r\nTo clarify my idea I put here an example
\r\n\r\n
try\r\n{\r\n Using (Zip....)\r\n {\r\n\r\n foreach (var o in list)\r\n Zip.AddEntry(o,......)\r\n } \r\n}\r\ncatch (Exception or ArgumentException or Exception of System NS)\r\n{\r\n // do smt\r\n}\r\n
\r\n
If AddEntry raises an Exception I have to catch it as "System Exception" that doesn't tell me nothing about the object "o" I tried to put with AddEntry.\r\n
So I have to do something like this:\r\n
\r\n
\r\n\r\ncatch (System.Exception....)\r\n{\r\n foreach( var o in list)\r\n check what is the "o" object that has problem\r\n\r\n MessageBox.Show("The object is"+ o)\r\n}\r\n\r\n
Instead, I would like something close to this:\r\n
\r\n
\r\n\r\ncatch (ZipNetException.DuplicatedItem ex)\r\n{\r\n MessageBox.Show("The object probably is "+ e.Message)\r\n}\r\n\r\n
As you can see, in this code I catch YOUR exception and not a generic system exception.\r\n
With your (ZipNet) exception I could have the entry that raised the exception.\r\n
\r\n
Yes I think is a design question.\r\n
\r\n
I hope my idea is more clear now\r\n
\r\n
regards\r\n
\r\n", "PostedDate": "2010-11-05T10:49:46.643-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "517920", "ThreadId": "233647", "Html": "\r\n
ok, it is becoming a little clearer.
\r\nI think what you are saying is that you would like to receive a exception that is specific to DotNetZip in case of a particular error or exception condition.
\r\nCan you tell me, what is the specific exceptional condition you are referring to? what specific exception is raised currently by DotNetZip? Can you show a stacktrace, please? What specific exception would you like to receive?
\r\n\r\n", "PostedDate": "2010-11-05T13:33:58.523-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "517934", "ThreadId": "233647", "Html": "\r\n
I make a Zip with contents and relative entry names. It could happen that some entries are duplicated.
\r\nIn this case I would like to receive an Exception telling me the entry is already present. As the message of the Exception it could
\r\nhave the name of the Entry I try to add.
\r\n\r\n
Monday morning I'm going to send you the stacktrace and the specific exception I get.
\r\n\r\n
thanks a lot
\r\n", "PostedDate": "2010-11-05T14:05:10.327-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "518368", "ThreadId": "233647", "Html": "\r\nThank you - send the stacktrace and we'll see if we can make some progress on that.
\r\n", "PostedDate": "2010-11-07T06:32:08.01-08:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "518672", "ThreadId": "233647", "Html": "\r\nhere is the StackTrace
\r\n\r\nin System.ThrowHelper.ThrowArgumentException(ExceptionResource resource)\r\n in System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)\r\n in System.Collections.Generic.Dictionary`2.Add(TKey key, TValue value)\r\n in Ionic.Zip.ZipFile.InternalAddEntry(String name, ZipEntry entry)\r\n in Ionic.Zip.ZipFile._InternalAddEntry(ZipEntry ze)\r\n in Ionic.Zip.ZipFile.AddEntry(String entryName, Stream stream)\r\n in Ionic.Zip.ZipFile.AddEntry(String entryName, String content, Encoding encoding)\r\n in Ra....Sta.Logic.ProjectManager.SalvaProgetto(String nomeProgetto, List`1 listAg) in C:\\Lavoro\\Lavoro\\...\\Logic\\ProjectManager.cs:riga 232\r\n\r\n
\r\n
I catch the "System.ArgumentException".
\r\nthanks
\r\n\r\n", "PostedDate": "2010-11-08T03:22:41.957-08:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "519427", "ThreadId": "233647", "Html": "\r\n
Ah, yes, the generic Dictionary throws System.ArgumentException when you try to add a duplicate key. \r\nhttp://msdn.microsoft.com/en-us/library/k7z0zy8k.aspx
\r\nok, now... what is the problem with that exception? Can you tell me what you would propose as an alternative? Would you prefer to get something like\r\nIonic.Zip.DuplicateZipEntryException ?
Let me think about that.
\r\n", "PostedDate": "2010-11-09T09:13:04.337-08:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "519461", "ThreadId": "233647", "Html": "\r\nYES!!!
\r\nI would like to have Ionic.Zip.DuplicateZipEntryException and in the payload of the exception the name of the entry I tried to add.
catch (Ionic.Zip.DuplicateZipEntryException ex)\r\n{\r\n MessageBox.Show("The object probably is "+ e.Message)\r\n}\r\n
ok Signor, I think that makes sense.
\r\nThis feature won't be available until I build a new release. I hope that will be soon, but no guarantees.
\r\n\r\n", "PostedDate": "2010-11-09T10:38:17.45-08:00", "UserRole": null, "MarkedAsAnswerDate": null } ]