Learn More
Resend Email
36
TheSmoun
7y

Just found this piece of code from one of my coworkers:

restMethod() {
try {
// some complicated logic
} catch (WebApplicationException e) {
throw e;
}
}

Why?!

And btw: Hi fellow devRant ppl! 👋

rant

rest api

jpa

java
Favorite
Ranter
Comments
  • 1
    7y
    Maybe he initially planned to log it here before rethrowing.
  • 0
    7y
    @danglingPtr yeah maybe. Usually you could use this for logging or for rollbacks in the EntityManager
  • 0
    7y
    I've seen this, and in most cases it was a placeholder for a to-do: throw another exception from the current namespace (and perhaps log this one)

    This try-catch is redundant, though.
  • 1
    7y
    I use try catch and then throw just to cleanup for the exception, log every important thing then throw
  • 0
    7y
    Ah. The old rethrow and discard stack trace gambit.
  • 0
    2y
    lmao, I bet one day this motherfucker will be catching StackOverflowException
Related Rants
devRant © 2021 Hexical Labs LLC
Privacy Policy  |  Terms of Service
Add Comment