Code Review Factors

 

• Misc
• Boxing – Use if ‘ref’ keyword
• Instantiation - Use if ‘new’ keyword
• Loops
• Proper Collection used
• ‘as’ operator
• ConvertToInt – Extension method
• Return statement
• Hard Code Values
• Check for nulls
• Use of Temporary variables
• Avoid Global variables

 

 

 

• Code Compliance – As followed by our earlier Architect. It’s a long list and comprised of lot of factors in brief.
• Others are as below.
• Frequent code paths
• Frequent loops.
• Boxing
• Collections
• Null Checks
• Instantiation checks
• Type Conversion
• Proper use of Resource Files
• TypeCasting
• Access Specifiers
• Proper use of Redirection
• String operations
• Avoid multiple property access calls
• Unnecessary Long Codes
• Naming convention for Resource keys
• JS vs Serverside Validations
• Proper use of DataBinding

 

 

 

Thanks & Regards,

Arun Manglick || Senior Tech Lead