Hi!<p>I work as a Front End developer in Dublin and I was wondering the following:<p>- All companies that I worked, I never saw a programmer saying good things about someone else's code.<p>Since I realised this thing, I always look at someone else's code in a different way. I try not to thing of how bad the code actually is but I try to criticise architecture problems and things that might be optimised. Not the code itself.<p>What I say about not the code itself would be something like:<p>Trying to avoid to criticise micro optimisations and/or things that I know it is bad but I do it anyway for reasons that might be due to time constraints.<p>The thing is, It is very easy to criticise someone's code but it is very hard to at the same time have a great code quality AND meet all constraints to deliver that.<p>And the point is, at the end, someone will look and still think it is bad.<p>I would like to know your thoughts about this problem and how to address them in a better way.
<i>"All companies that I worked, I never saw a programmer saying good things about someone else's code."</i><p>I think that this may have to do with the fact that reading code (even code that you yourself wrote a while ago) is harder than writing code. When you're writing code, you have an image fresh in your mind of what you want to create, but when you're reading code, you're struggling to build up a mental model of what that code does. And if, because of this, you perceive the other person's code as being harder to understand, it might bias you into thinking that it's bad code.<p>People who have a lot of programming experience become better at reading other peoples' code - it's a learnable skill, like sight-reading music.<p>I read a lot of other peoples' code, and I find code that's all over the spectrum: worse than mine, about the same as mine and better than mine (sometimes a lot better). I have co-workers who write sloppy code and other co-workers who write very clean code.<p>A programmer who has never seen code that they can say good things about is probably fairly inexperienced (hasn't read very much code in their life).
Levels of experience vary all over the map. Orders of magnitude differences. When experienced people get called in to deal with problem code, it's most likely been written by someone with less experience. There is very often room for critical comments. Just statistics.
There are multiple different psychological theories that might explain why we dislike code that we don't write. I do agree it is an interesting thing to think about.<p>Two that I can immediately think of are<p>1. Ikea Effect(1) - We place a higher value on things we create vs. things we didn't create<p>How it applies: We place more value on code we wrote compared to what others wrote.<p>2. Fundamental Attribution Error (2) - We tend to look at our own accomplishments as evidence of our own ability, and our failures as events caused by external influence. When it comes to others it's the reverse, we look at others accomplishments as the result of external events, and their failures as evidence of their internal lacking of ability<p>How this applies: Let's face it, if you're working as a software developer, you don't always have the time to write 100% perfect code (which is not an excuse to write bad code) but a reason why we often settle for "good enough" code. We probably accept this rationalization when reading our own code but are less lenient when reading others code. This then frames their judgment of the code in a negative light. "I hate this code because the developer took shortcuts/didn't do things the right way... " Even though we do the same.<p>3. Mere Exposure Effect (3) - We like things we're familiar with.<p>How it applies: Since we wrote our code, know how it works (most of the time right...) and what we were thinking when we wrote it, that familiarity is pleasurable vs code that we don't know how it works (yet), and have less of an idea of the mindset that the developer had when writing it.<p>Those are just a few... there are probably more theories you can apply to this problem but I do agree it's a real thing.<p>[1] <a href="https://en.wikipedia.org/wiki/IKEA_effect" rel="nofollow">https://en.wikipedia.org/wiki/IKEA_effect</a><p>[2] <a href="https://en.wikipedia.org/wiki/Fundamental_attribution_error" rel="nofollow">https://en.wikipedia.org/wiki/Fundamental_attribution_error</a><p>[3] <a href="https://en.wikipedia.org/wiki/Mere-exposure_effect" rel="nofollow">https://en.wikipedia.org/wiki/Mere-exposure_effect</a>