3 ms·
This is a disaster if the senior programmers are less skilled than the junior programmers. It happens a lot in big corporations. "You have more than one return
by sofal 18y ago
This is a disaster if the senior programmers are less skilled than the junior programmers. It happens a lot in big corporations. "You have more than one return statement! Go change that."
Of course if you work in a shop like that, it's advisable to leave. And I did.
- madhan 18y agoI 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.
- Tamerlin 18y agoOne 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.
- edw519 18y agoYou 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.
- sofal 18y agoJust 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?
- edw519 18y agoI 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.