One situation I was even worse; the "senior" programmers were actually juniors with nearly no experience, but who had office politics (i.e. nepotism -- one was the CTO's son) on their side.
Needless to say, the company didn't go far.
I used to work where I would get skewered for misplacing the ; or about the formatting of my comments. After surviving that, I think I can survive anywhere.
You have more than one return statement! Go change that.
Yep, that's one of the 38 reasons for my original rejection. Violations of "single entry/single exit" were not promoted into production. I argued and lost. Now I'm glad I did.
Just curious: Are you now a true believer in the "single exit always" doctrine (as well as whatever the other 37 rules were) or are you just saying that the benefits of enforcing discipline on the developers' output outweighed the disadvantages of having cargo-culted and dogmatic code standards?
I became a true believer once I got past my personal prejudices and actually made an attempt to understand "why" some rules were the way they were. I wasn't as smart as I thought. Still not.
Enforcing discipline for the sake of enforcing discipline? See military.
Of course if you work in a shop like that, it's advisable to leave. And I did.