3 ms·
For someone to see your criticism as objective, the objective standard should be established ahead of time. For design/code reviews this means having coding sta
by vinay_ys 6y ago
For someone to see your criticism as objective, the objective standard should be established ahead of time. For design/code reviews this means having coding standards/guidelines and training everyone to stick to them – rules of the game are known ahead of time to all participants. Then it is much easier to convince someone that it is objective feedback.
A critical skill for a senior engineer in a team is to define and establish such standards for the team. Rather than reacting to each design/code review, taking the time to setup these process mechanisms is a lot more productive and healthy for everyone involved.
This requires strategic thinking (a skill that comes with practice) towards how you choose to spend your time at work to accomplish anything.
- berkes 6y ago> as objective, the objective standard should be established ahead of time. A lot of projects lack that "objective standards". I've been helping to solve exactly this. But code can be messy without having a clear defined "clean code" standard. And a project can perform horrible, without having minimum response times docutented. Often my job was to define all those. And often those definitions already hit a nerve. When suddenly your code is marked as "far too tightly coupled" or "too complex: AB above 34" when you thought it was neat and smart code, that is confronting and, I've seen, is often felt as personal critique.