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

  • ipkpjersi@lemmy.one
    link
    fedilink
    arrow-up
    2
    ·
    2 years ago

    Actually, exception rethrowing is a real thing - at least in Java. You may not always want to handle the exception at the absolute lowest level, so sometimes you will instead “bubble” the exception up the callstack. This in turn can help with centralizing exception handling, separation of concerns, and making your application more modular.

    It seems counter-intuitive but it’s actually legit, again at least in Java. lol

    • TwilightKiddy@programming.dev
      link
      fedilink
      arrow-up
      1
      ·
      2 years ago

      Rethrowing caught exception in C# is just throw;, not throw ex;. This will delete old stack trace, which is very punishable if someone debugs your code later and you’re still around.

        • TwilightKiddy@programming.dev
          link
          fedilink
          arrow-up
          1
          ·
          1 year ago

          throw ex; treats ex as a new exception, so, it starts a new stack trace for it from itself and deletes stack trace that was saved in ex.StackTrace. On the other hand, throw; takes already present exception in the scope and throws it without modifying the stack trace, preserving the original method that threw ex in the stack trace.

          I feel like I wrote the same thing twice. I’m a bit bad with explaining stuff, feel free to ask more specific questions if you still don’t understand the difference.

    • grimmi@feddit.de
      link
      fedilink
      arrow-up
      1
      ·
      2 years ago

      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
        link
        fedilink
        arrow-up
        1
        ·
        edit-2
        2 years ago

        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
          link
          fedilink
          arrow-up
          1
          ·
          2 years 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.

    • Jaloopa@beehaw.org
      link
      fedilink
      arrow-up
      1
      ·
      2 years ago

      Why wouldn’t it? It’s syntactically valid C#, with the added bonus of destroying the stack trace