Do all the things like ++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatarSign Up
Lyniven446212dCode review shouldn't be based on preferences at all tho. Choosing A or B implementation is a team choice based on how well it fits the product you're making together.
C0D44666712dCode review isn't for selecting a path and making decisions. It's for getting a second or third set of eyes over the chosen solution (this should have already been done as @Lyniven said) and making sure there's no bugs, stand out performance issues, or missed requirements, or just recommendations.
Nanos875012d> or the companies I work for do something
> very very wrong in code review.
Probably a good question to ask in any job interview, eg. interview them about their code review with an simple example you prepared and have with you, and see if they pass the test !
Tell them if they don't. :-)
Or tell them you will, and don't !
Sabro31912dI like code reviews, it's useful. Someone explains to me why they prefer a particular solution == I learn something. Either about them as a person or about my job.
"People" is your problem here. Or "person".
Nanos875012dNowadays I notice a lot of folk like to exercise power because they can.
A previous place I worked, every time a new manager got in, they would change things, if they needed to be changed or not..
I only changed the things that needed to be improved and left the things that had been working fine for hundreds of years alone. :-)
I remember moving a desk that was in the wrong place, doing so saved 18,000 working hours a year.
When a new manager got in charge, they moved it back to its old place, and everyone's workload went up again, even though they knew why I moved it in the first place !
Fast-Nop2007312dSuch an aggressive code review is only warranted if the code actually would run into problems. Like, choice of an n^2 algo because it's fast enough for the small test data sets, but may be faced with large sets now or in the future.
muttley8611d“I don’t like your solution “ WTAF!
PRs are welcome-> RESOLVE