r/csharp • u/YouAreMyFire00 • 6h ago
About codebase conventions, who is actually right and who is wrong here?
Let's say you're a Technical Founder like Mark Zuckerberg or Bill Gates. You have a CS degree worked as mid level developer.
You've written some code (as show in the pic) and added comments to remind yourself and other developers what the code does, so nobody forgets in the future.
Now, you have a Senior Developer on your team who worships Uncle Bob's Clean Code principles like a cult!
she reviews your PR and reject it, saying
"Good code should be self-explanatory. The code itself should explain what it's doing.
Anyone should be able to look at it on the first time and understand it right away.
So go back and remove those comments. They're not clean! God damn it!"
--
What would you do in this situation?
Would you trust the senior dev and delete the comments because they have more experience than you, even though you're the founder and have less development experience than they do?
Ps. We have to move fast