I'm a junior engineer at a startup and I was hoping to get some tips on how to reduce the number of comments, number of bugs found, basically reduce the back and forth needed and improve quality of my code during code reviews?
Would love to hear specific, actionable advice from other engineers. Tech stack, if relevant, is RoR, React, PostgreSQL
You mention bugs specifically, for which I think the best remedies are good unit tests and careful consideration of edge cases (which often go hand in hand). Other smaller issues such as deviation from framework conventions and performance problems can be helped by getting to know your languages, frameworks and database better, which comes with time. More generally, there are lots of good resources out there to help you improve the readability and structure of your code - Clean Code by Robert Martin (particularly the earlier chapters) and Sandi Metz’s books and talks are both good starting points. Issues with your general approach to solving problems are harder to address, but reviewing others’ code, learning about design patterns and pair programming can all help here.
In terms of actionable advice, I’d encourage you to briefly discuss your intended approach to a problem with a colleague before starting work: this can help expose issues in advance. Even better, pair program on the implementation with another engineer. Just pairing on the review can also help to reduce the back and forth and get code shipped faster.
On bugs, those are opportunities to write unit tests that reproduce the bug, fix the code, and prove that the bug no longer exists. After a while, you'll be able to recognize how to avoid bugs specific to the type of things you've done in the past. If it helps to know, you'll gradually get better, but even the best/most experienced/smartest people can have bugs in their code.
To summarize, don't feel bad or inadequate for making a mistake. Appreciate the fact that someone cares enough to help you improve and they also care about the company that they take the time to do good code reviews. You'll get better.
“Don’t just read the code guidelines, try to search the ciérnase for a similar case to the problem you are solving and apply the same principles. Generally, look around and make your code blend. Then review your own PR, sometimes it’s easy to spot mistakes when looking at just the diff.”
In addition, I tend to repeat myself a ouch times so they “get it”, but I do expect not to have to point out the same mistake over and over again, so pay attention to what’s pointed out during review and remember it for future reviews!
Full disclosure working on https://www.superadmin.so. Trying to help with making e2e testing easier.