- lobut ( @lobut@lemmy.ca ) 87•10 months ago
I remember when I was working with .NET and I saw some web service code and I saw there was no try catches. They didn’t have a global catch in the asax either or anything. I just wrapped each call into a try catch and log.
I got the same treatment where my manager wanted to know what happened with the increase in errors. I told him what I did. My manager got another developer to go through my commits regardless. I was a bit upset at the lack of trust.
- magikmw ( @magikmw@lemm.ee ) 16•10 months ago
A manager that can’t read a simple try catch commit? Why am I surprised.
- dark_stang ( @dark_stang@beehaw.org ) English9•10 months ago
I joined a team years ago where everyone would catch exceptions then throw a different exception in the catch, swallowing the original. Sometimes these were nested many layers. Troubleshooting was a nightmare.
I spent a week deleting all of them and told everybody that “try” was now a forbidden word outside of entry points.
- lobut ( @lobut@lemmy.ca ) 5•10 months ago
I remember doing that as a junior because everyone in our codebase did it!
My new team lead came in and asked me why. I said it’s to reformat it due to the layer it was in. He said “…and what did you really accomplish with that?” All we did was bury our real error really well. It made me think about these things and to question convention more
- phorq ( @phorq@lemmy.ml ) 44•10 months ago
I would normally put an if-statement before that to verify if IGNORANCE still equals BLISS
- db2 ( @db2@sopuli.xyz ) 29•10 months ago
The second panel would be “No, I enabled error reporting. Those errors belong to just you.”
- katy ✨ ( @cupcakezealot@lemmy.blahaj.zone ) 18•10 months ago
need a blame.css but for code
- SapientLasagna ( @SapientLasagna@lemmy.ca ) 2•10 months ago
I’ll see your blame.css and raise you git blame somebody else
- Digital Mark ( @mdhughes@lemmy.ml ) English17•10 months ago
“My project” doesn’t exist in any team. It’s everyone’s project. A manager needs to have a long conversation with Pink Pants.
If you build your project at anything but highest error level,
clang -Wall
etc., you’re letting errors in, relying at best on coincidence to work the way you think it does.Commit it and don’t revert it!
- pkulak ( @pkulak@beehaw.org ) 2•10 months ago
- Digital Mark ( @mdhughes@lemmy.ml ) English1•10 months ago
You’d think, but there’s a lot of Pink Pants stans downthread.
- Crul ( @Crul@lemm.ee ) 16•10 months ago
Source: https://www.commitstrip.com/2015/04/27/the-eye-opener-commit/
Also on twitter:
- Twitter @CommitStrip
- Nitter link
- Nitter RSS Feed: https://nitter.cz/CommitStrip/rss
- Bear_with_a_hammer ( @Bear_with_a_hammer@lemmy.ml ) 5•10 months ago
That’s why PHP team gave us null coalescing (??), oh the misery we went through writing if / else
- lseif ( @lseif@sopuli.xyz ) 1•10 months ago
who doesnt love ??