diff options
author | einhverfr <einhverfr@4979c152-3d1c-0410-bac9-87ea11338e46> | 2006-09-09 06:55:13 +0000 |
---|---|---|
committer | einhverfr <einhverfr@4979c152-3d1c-0410-bac9-87ea11338e46> | 2006-09-09 06:55:13 +0000 |
commit | 43629eb68af5e8af74b72576f56f359bf975bd07 (patch) | |
tree | c1b4af7b8d0d915b79e4b9276e8df079398078b9 | |
parent | 1036b3068092e00d59c21c4538fc841a1a24b1a0 (diff) |
Added coding standard
git-svn-id: https://ledger-smb.svn.sourceforge.net/svnroot/ledger-smb/trunk@44 4979c152-3d1c-0410-bac9-87ea11338e46
-rw-r--r-- | doc/coding-standard.pdf | bin | 0 -> 44552 bytes | |||
-rw-r--r-- | doc/coding-standard.tex | 66 |
2 files changed, 66 insertions, 0 deletions
diff --git a/doc/coding-standard.pdf b/doc/coding-standard.pdf Binary files differnew file mode 100644 index 00000000..ec407ccf --- /dev/null +++ b/doc/coding-standard.pdf diff --git a/doc/coding-standard.tex b/doc/coding-standard.tex new file mode 100644 index 00000000..424dbe24 --- /dev/null +++ b/doc/coding-standard.tex @@ -0,0 +1,66 @@ +\documentclass{article} +\usepackage{metatron} +\title{LedgerSMB Coding Standards} +\author{The LedgerSMB Development Team} +\begin{document} +\maketitle + +\tableofcontents + +\section{Introduction} +Consistant coding style contributes to readability of code. This contributes in +turn to fewer bugs and easier debugging. + +While consultants which implement custom solutions for their customers are not +required to follow these coding guidelines, doing so will likely contribute to +efficiency of the project and, if desired, the likelihood of having the changes +accepted in the main source tree. + +\section{File Organization and Whitespace} + +Files should be organized according to three principles: atomicity, performance, +and readibility. While there is no maximum file size, there is no reason to +group large amounts of code together when only a small subset of that code will +be executed on any given run of the program. Similarly, core API for a single +data structure entity may be grouped together even if each run may touch only a +small part of the code so that these functions can be maintained together in a +logical way. + +Nested blocks of code should be indented with a single tab. This way, +programmers can adjust the tab width of their editors to their preferences. + +Lines longer than 79 characters are not handled well by many terminals and +should generally be avoided. Continued lines should be indented by one more tab +than either the line above or below (i.e. if the line above is indented by two +tabs and the line below by three, indent the continued segment by four). + +Lines longer than 79 characters cause problems in some terminals and should be +avoided. + +\section{Comments} + +In an ideal world, the code should be sufficiently readible to be entirely +understandable without comments. Unfortunately such an ideal is rarely +attained. Comments should be used to annotate code and add information that is +not already a part of code or programming logic itself. + +Comments may include arguments and return values of functions (for easy +reference), a summary as to why a particular design choice was made, or a note +to alert future programmers that a particular chunk of code deserves additional +attention. Comments should not, however generaly tell what the program is doing +or how it does that. If such comments are required, it is a good indication +that a block of code requires rewriting. + +Additionally it is a good idea to provide a brief description at the top of each +file describing, in general terms, what its function is. + +\section{Function Organization} + +Functions should be atomic. While there is no maximum length to functions, long +functions may be an indication that a function may be non-atomic. + +In general, when more than one line of code is being copied and pasted, it +should instead be moved into its own function where it can be called by all +entry points. + +\end{document} |