Saturday, July 11, 2009

What is 'good code' & 'good design'?

Good code:
'Good code' is code that works, is bug free, and is readable and maintainable. Some organizations have coding 'standards' that all developers are supposed to adhere to, but everyone has different ideas about what's best, or what is too many or too few rules. There are also various theories and metrics, such as McCabe Complexity metrics. It should be kept in mind that excessive use of standards and rules can stifle productivity and creativity. 'Peer reviews', 'buddy checks' code analysis tools, etc. can be used to check for problems and enforce standards.
For C and C++ coding, here are some typical ideas to consider in setting rules/standards; these may or may not apply to a particular situation:

  • minimize or eliminate use of global variables.
  • use descriptive function and method names - use both upper and lower case, avoid abbreviations, use as many characters as necessary to be adequately descriptive (use of more than 20 characters is not out of line); be consistent in naming conventions.
  • use descriptive variable names - use both upper and lower case, avoid abbreviations, use as many characters as necessary to be adequately descriptive (use of more than 20 characters is not out of line); be consistent in naming conventions.
  • function and method sizes should be minimized; less than 100 lines of code is good, less than 50 lines is preferable.
  • function descriptions should be clearly spelled out in comments preceding a function's code.
  • organize code for readability.
  • use whitespace generously - vertically and horizontally
  • each line of code should contain 70 characters max.
  • one code statement per line.
  • coding style should be consistent throught a program (eg, use of brackets, indentations, naming conventions, etc.)
  • in adding comments, err on the side of too many rather than too few comments; a common rule of thumb is that there should be at least as many lines of comments (including header blocks) as lines of code.
  • no matter how small, an application should include documentaion of the overall program function and flow (even a few paragraphs is better than nothing); or if possible a separate flow chart and detailed program documentation.
  • make extensive use of error handling procedures and status and error logging.
  • for C++, to minimize complexity and increase maintainability, avoid too many levels of inheritance in class heirarchies (relative to the size and complexity of the application). Minimize use of multiple inheritance, and minimize use of operator overloading (note that the Java programming language eliminates multiple inheritance and operator overloading.)
  • for C++, keep class methods small, less than 50 lines of code per method is preferable.
  • for C++, make liberal use of exception handlers


Good Design:
'Design' could refer to many things, but often refers to 'functional design' or 'internal design'. Good internal design is indicated by software code whose overall structure is clear, understandable, easily modifiable, and maintainable; is robust with sufficient error-handling and status logging capability; and works correctly when implemented. Good functional design is indicated by an application whose functionality can be traced back to customer and end-user requirements. (See further discussion of functional and internal design in 'What's the big deal about requirements?' in FAQ #2.) For programs that have a user interface, it's often a good idea to assume that the end user will have little computer knowledge and may not read a user manual or even the on-line help; some common rules-of-thumb include:

  • the program should act in a way that least surprises the user
  • it should always be evident to the user what can be done next and how to exit
  • the program shouldn't let the users do something stupid without warning them.

No comments:

Post a Comment