Programmer Humor

32371 readers

488 users here now

Post funny things about programming here! (Or just rant about your favourite programming language.)

Rules:

Posts must be relevant to programming, programmers, or computer science.
No NSFW content.
Jokes must be in good taste. No hate speech, bigotry, etc.

founded 5 years ago

MODERATORS

AgreeableLandscape@lemmy.ml

cat_programmer@lemmy.ml

Brilliant exception handling I found in an app i had to work on (media.kbin.social)

submitted 1 year ago by Zadyra@kbin.social to c/programmerhumor@lemmy.ml

20 comments fedilink hide all child comments

Brilliant exception handling I found in an app i had to work on

you are viewing a single comment's thread
view the rest of the comments

[–] Shareiff@lemmy.world 2 points 1 year ago (5 children)

Lol what’s wrong with this if the parent function catches it

[–] chillhelm@lemmy.world 3 points 1 year ago

Depending on the language it either does nothing and just adds code bloat or (and this would be much worse) it will catch any exception that can be implicitly cast to type Exception and throw it as type Exception. So the next higher scope would not be able to catch e.g. a RuntimeException or w.e. to handle appropriately. It could only catch a regular Exception even if the original error was a more detailed type.

[–] grimmi@feddit.de 1 points 1 year ago (1 children)

If this is C# (and it looks like it is), this leads to you losing the original stack trace up until this point.

The correct way to do this in C# is to just throw; after you're done with whatever you wanted to do in the catch.

[–] jyte@lemmy.world 1 points 1 year ago* (last edited 1 year ago) (1 children)

wait what ?

So you are saying that the following code will keep throwing e but if I used throw e; it would basically be the same except for the stack trace that would be missing the important root cause ?!

try {
} catch (WhateverException e) {
    // stuff, or nothing, or whatever
    throw; 
}

[–] TwilightKiddy@programming.dev 1 points 1 year ago

Exactly. Aside from deleting your already built stack trace, as a bonus you'll get another stack trace building call, enjoy wasted CPU cycles.

[–] ElmiHalt@sopuli.xyz 1 points 1 year ago

You don't catch it if that's the case

[–] EinfachUnersetzlich@lemm.ee 1 points 1 year ago

You could just not catch it and it'll get thrown up the stack anyway.

[–] Xanvial@lemmy.one 0 points 1 year ago (1 children)

The catch is useless if it's just throwing the exception anyway

[–] Kleysley@lemmy.fmhy.ml 1 points 1 year ago

The catch is mandatory...