Offering a different point of view helps either confirm their decisions or open up avenues for improvement of their work. It’s not about telling “this is how I would do it” but rather “Have you thought about other options and approaches? What do you think? Why did you choose this route?” helps the individual weigh the pros and cons with each. It should never be an opportunity for you to chime in with how you would have done it (even if you would have done it that way). You simply want to expose the other possibilities so that they are aware of them and haven’t overlooked a potential savings or killer-feature.
"I like the way you tackeled this" Implies the work has been done successfully.
", have you thought about this approach?" is trying to imply that the work is not sufficient actually without actually saying that. Your message has taken a complete 180 since the first half of the sentence but you haven't given the listener any clues because if you use a word like "but" it might hurt the facade of niceness. It's mixed messaging.
But I think a lot of the issue here is because were talking about a made up example. If we just add a little more detail to the made up response it clears up a lot of the ambiguity and makes it sound less contradictory.
"I really liked how you tackled [X Requirement], have you thought about [Y Requirement + approach]?"
to be clear, I fully agree with this and your broader point. It's the specific verbiage in your first example that I think is problematic, mainly because it feels disingenuous and insincere at best, manipulative at worst. Which is highly unfortunate because I think the vast majority of the time the motives are pure!
It’s funny because I still feel the opposite of you — I think the example where you say that you like their solution is a better approach, as long as it’s not horrible.
It leaves open the possibility that you as the reviewer are actually mistaken, and they possibly have a good reason for doing it that way.
I may be the senior reviewing the junior’s code, but they probably spent longer looking at the problem than me and there is a chance that I’m missing something. By suggesting “have you considered this, it may be able to handle x situation better”, you respect their work more.
It depends on context though, if somebody clearly just didn’t understand something obvious, then I’ll just tell them directly.
Either way, probably a difference in our opinions of feedback, you might prefer more directness than me
> It leaves open the possibility that you as the reviewer are actually mistaken, and they possibly have a good reason for doing it that way.
That's a great point. There must be room in this to cover that the person actually had good reasons that the asker doesn't know about. It's a hard needle to thread.
It feels stilted and weird to me. My first thought was "OK, that sounds unnatural, I'd have to say it different." Good idea, but I could never let it come out of my mouth that was since no actual human being talks that way.
as a manager, I'm not looking to challenge your outcome, I'm looking for you to explain why your solution works best compared to the others that were suggested. "have you thought about X?" is a genuine curiosity so that I may defend your solution to the higher ups when they ask me "what about X?". They are constantly being courted by vendors. Vendors who think they know the problem better than you do. Let's show them that you covered the bases and hit that home run properly.
In this context, is it weird to ask if you've "thought about X"? Is it disingenuine or condescending to you? Servant leadership is about supporting the team that I'm responsible for, to block and shield them from the wrath of VPs who want blood because their napkin-idea-over-cocktails wasn't selected for funding.
I'm trying to prepare you, the presenter of the solution, for the onslaught of questions that will come your way. If you feel these kinds of questions are a praise sandwich, I'm more than happy to present your solution for you.
