Code Review Checklist
-
1.Are all the conditional paths reachable? 2.Are all the individual conditions in a complex conditions separately evaluated? 3.If there is a nested IF statement, are the THEN and ELSE parts appropriately delimited? 4. In the case of a multi-way branch like SWITCH/CASE statement, is a default clause provided? Are the breaks after each CASE appropriates? 5.Is there any part of code that is unreachable? 6.Are there any loops that will never execute? 7.Are there any loops where the final condition will never be met and hence cause the program to go into an infinite loop? 8.What is the level of nesting of the conditional statements? Can the code be simplified to reduce complexity? http://www.mindfiresolutions.com/Code-Review-Checklist-238.php[^]
Cheers, Eliza
-
1.Are all the conditional paths reachable? 2.Are all the individual conditions in a complex conditions separately evaluated? 3.If there is a nested IF statement, are the THEN and ELSE parts appropriately delimited? 4. In the case of a multi-way branch like SWITCH/CASE statement, is a default clause provided? Are the breaks after each CASE appropriates? 5.Is there any part of code that is unreachable? 6.Are there any loops that will never execute? 7.Are there any loops where the final condition will never be met and hence cause the program to go into an infinite loop? 8.What is the level of nesting of the conditional statements? Can the code be simplified to reduce complexity? http://www.mindfiresolutions.com/Code-Review-Checklist-238.php[^]
Cheers, Eliza
Quote from your profile .... Biography: I am in the marketing team at Mindfire Solution. Is that whay you have not asked a proper question and have a link to your website ....... ? :rolleyes:
Ali