How to deal with developers who lack standards?

https://news.ycombinator.com/item?id=17100445

by hotz • 8 years ago

9 14 8 years ago

news.ycombinator.com

How do you deal with developers that don't comment their code? They're fully aware that they should and that it's the decent thing to do or when they update functions and move things around, they actually remove the comments.

Things like PEP8 or PSR-2 mean nothing to them.

I could also mention changing functionality, but not making sure that the tests are updated.

onion2k• 8 years ago

Tools not rules!

Use hooks to check quality automatically on your repos and/or CI server. Change the environment so a developer can't push code unless linting and tests pass. If you configure the remote to use receive.denyNonFastForwards they won't even be able to use --force. Then add linting rules so you have to comment things (eg https://eslint.org/docs/rules/valid-jsdoc). It won't enforce quality, but that comes later once people habitually know to write comments.

Expect a lot of resistance to this sort of solution though.

hotz2• 8 years ago

I've tried that. I was seen as having a stick up my a*. I've given up on enforcing it on that level.

jerry40• 8 years ago

Why do they remove comments? Do they delete commented code or textual comments?

hotz2• 8 years ago

Your standard docblocks, even if the docblock explains what the function does if it's not clear. You'll basically have a function with zero comments. It's painful, I've never been given a reason as to why.

chatmasta• 8 years ago

Why can’t you just read the function itself?

Maybe the real problem is people writing code that is so hard to understand you need a comment to explain what it does.

gary__2• 8 years ago

I'll hazard a guess that it's because it's quicker to delete than update them.

camhenlin• 8 years ago

Is the code readable without comments? The team that I am on tends to (at least attempt!) write verbose code with long variable and function names, to obviate the need for comments. We do use comments for pointing out particularly difficult pieces of code or code that exists in a specific way for a specific reason, but we tend to think that most comments that people write are unnecessary

gary__2• 8 years ago

I took this approach after reading Clean Code, though I must say, I often regret not adding the comment as well when I return to the code much later. Still try to make the code as readable as possible though.

ajeet_dhaliwal• 8 years ago

How do you know they are fully aware? May be they are not. I’ve been programming for almost 20 years and I’d never heard of PEP8 or PSR-2 until you mentioned them, granted I don’t do much python or php work.

May be the environment they are working on constricts them. May be they are pushed to deliver ASAP and given no time to think about maintainability because management doesn’t care about technical debt.

If you want higher standards drive it through by giving talks to your team and suggesting a style guideline be adopted to whoever the tech lead and introduce code reviews to ensure this adhered to.

seattle_spring• 8 years ago

What are the comments? Usually people that complain that I don't comment enough are also the ones that add extremely useless comments like:

    // Return this
    return this;

timojaask• 8 years ago

Have you discussed it with them? Did you ask them, without judging, why do they feel that it's better to delete the comment rather than update it? I'm wondering what kind of answers would come from them.

Who knows, maybe they have a legit reason for the behavior. Or maybe they simply did not give it enough thought. Either way, the humane appreach, talking, could help here.

tucaz• 8 years ago

Fire them. If you are specifically asking for a task to be done (add comments) and they are not doing it anyway it is the same as not doing some other task.

If you don’t fire them even after talking and asking them to change their behavior then stop complaining and accept it as a fact of life.

How to deal with developers who lack standards?

Related Stories