General The code works The code is easy to understand Follows coding conventions Names are simple and if possible short Names are spelt correctly Names contain units where applicable There are no usages of magic numbers No hard coded constants that could possibly change in the future All variables are in the smallest scope possible There is no commented out code There is no dead code (inaccessible at Runtime) No code that can be replaced with library functions Variables are not accidentally used with null values Variables are immutable where possible Code is not repeated or duplicated There is an else block for every if clause even if it is empty No complex/long boolean expressions No negatively named boolean variables No empty blocks of code Ideal data structures are used Constructors do not accept null/none values Catch clause...