using usage (IDisposable)

[TMattC]

Ok guys, thanks for your help.

[Lost User]

Why are you only catching "FormatException"? You prefer to crash otherwise? In this context, it only makes sense to catch the general "Exception" ... One can always use e.GetType().ToString() or "if (e is FormatException)..." to log any exception specifics if it came down to that and you did not want to use multiple catch blocks.

[Richard Deeming]

You should never catch the general "Exception" type, except in the entry point to your application where you simply log the exception and exit. In the OP's code, the try..catch block isn't needed, since there are TryParse methods which won't throw an exception if the input is invalid.

---

"These people looked deep within my soul and assigned me a number based on the order in which I joined." - Homer

[Richard Deeming]

As others have said, the code you've posted will work as expected. However, you can avoid the try..catch block by using the TryParse methods. You should also check the result of line.Split(';') to avoid an IndexOutOfRangeException, check that the second field is a valid Enum value, and check that the dictionary doesn't already contain the key:

string[] fields = line.Split(';');
if (fields.Length < 5) return false;

// TODO: Give these variables proper names:
long f0;
int f1, f3, f4;

if (!long.TryParse(fields[0], out f0)) return false;
if (!int.TryParse(fields[1], out f1)) return false;
if (!int.TryParse(fields[3], out f3)) return false;
if (!int.TryParse(fields[4], out f4)) return false;

// If the dictionary already contains the key, the Add method throws an ArgumentException:
if (prodFileIndexDic.ContainsKey(f0)) return false;

// Any integer can be cast to any enum without an exception;
// you probably wany to check that it's valid first:
if (!Enum.IsDefined(typeof(ProdKategoriEnum), f1)) return false;

prodList.Add(new Produkt(f0, (ProdKategoriEnum)f1, fields[2], f3, f4));
prodFileIndexDic.Add(f0, lineIndex);
lineIndex++;

---

"These people looked deep within my soul and assigned me a number based on the order in which I joined." - Homer

[Lost User]

You have no idea what he has "higher up". I said: "in this context". You also can't seem to tell the difference between "Parse" and "TryParse". You should "never" shoot your mouth off before thinking. Take you down vote and ....

[Richard Deeming]

Gerry Schmitz wrote:

You have no idea what he has "higher up".

Doesn't matter - swallowing every exception "just-in-case" is still a bad idea.

Gerry Schmitz wrote:

You also can't seem to tell the difference between "Parse" and "TryParse".

What?! I suggest you try reading my comments again.

Gerry Schmitz wrote:

You should "never" shoot your mouth off before thinking.

Talking to yourself is the first sign of madness.

---

"These people looked deep within my soul and assigned me a number based on the order in which I joined." - Homer

[Lost User]

He was using "Parse" ... I never said "every" or "just-in-case" either ...

[Richard Deeming]

Which will throw an exception when the input cannot be parsed. Which is why I suggested using TryParse, which doesn't throw an exception. Which would then mean that he wouldn't need a try..catch block to catch the exception from the Parse method.

---

"These people looked deep within my soul and assigned me a number based on the order in which I joined." - Homer

[Lost User]

I was talking about his post ... not some side post of yours. Give it a rest; I know what TryParse is for.

[Lost User]

Point Nazis.