How to master at PR Reviews for Senior Devs
Code Review Smells and Heuristics
Nowadays, one of the most important skills that companies demand from Software Engineers (mainly from Senior level…) is to deliver high quality code reviews on iterative and incremental manner. That means with each PR Review Developers are asked to improve quality of the to-be-merged code again and again. In this post I will try to point out the basic principles that each Developer should have in mind when doing refactoring or review. If you want to see practical implementation of these points on code piece, check my this other blog:
Ultimate Guide to Clean Code with over 30 Java Refactoring Examples
Let us have a look at these points topic by topic:
1.Meaningful Names
Use intention revealing names: the method or variable names should explain itself what its intention is before even looking at code implementation.
Class names should be noun or noun phrase.
Method names should be verb.
Pick one word for each concept: get, retrieve, fetch are interchangeable, choose one and use only it for all same actions.
Use Computer Science terms: AccountAdapter would mean adapter pattern for programmer, if there is not…
