-
Mark Olesen authored
- seems to work more reliably than the previous indented colons
5636df2b
OpenFOAM C++ style guide
#
OpenFOAM C++ style guide
General
- 80 character lines max
- The normal indentation is 4 spaces per logical level.
- Use spaces for indentation, not tab characters.
- Avoid trailing whitespace.
- The body of control statements (eg, if, else, while, etc). always delineated with brace brackets. A possible exception can be made with ‘break’ or ‘continue’ as part of a control structure.
- stream output
-
<<
is always four characters after the start of the stream, so that the<<
symbols align, i.e.
-
Info<< ... os << ...
so
WarningIn("className::functionName()") << "Warning message"
NOT
WarningIn("className::functionName()") << "Warning message"
- no unnecessary class section headers, i.e. remove
// * * * * * * * * * * * * * Private Member Functions * * * * * * * * * * * // // Check // Edit // Write
if they contain nothing, even if planned for ‘future use’
- class titles are centred
/*---------------------------------------------------------------------------*\ Class exampleClass Declaration \*---------------------------------------------------------------------------*/
NOT
/*---------------------------------------------------------------------------*\ Class exampleClass Declaration \*---------------------------------------------------------------------------*/
The .H
Files
- header file spacing
- Leave two empty lines between sections
(as per functions in the
.C
file etc)
- Leave two empty lines between sections
(as per functions in the
- use “//- Comment” comments in header file
- add descriptions to class data and functions
- destructor
- If adding a comment to the destructor - use //- and code as a normal function:
//- Destructor ~className();
- inline functions
- Use inline functions where appropriate in a separate classNameI.H file. Avoid cluttering the header file with function bodies.
The .C
Files
- Do not open/close namespaces in a
.C
file- Fully scope the function name, i.e.
Foam::returnType Foam::className::functionName()
NOT
namespace Foam { ... returnType className::functionName() ... }
EXCEPTION
When there are multiple levels of namespace, they may be used in the .C
file, i.e.
namespace Foam { namespace compressible { namespace RASModels { ... } // End namespace RASModels } // End namespace compressible } // End namespace Foam
- Use two empty lines between functions
Coding Practice
- passing data as arguments or return Pass bool, label and scalar as copy, anything larger by reference.
- const Use everywhere it is applicable.
- variable initialisation using “=”
const className& variableName = otherClass.data();
NOT
const className& variableName(otherClass.data());
- virtual functions If a class is virtual - make all derived classes virtual.
Conditional Statements
if (condition) { code; }
OR
if ( long condition ) { code; }
NOT (no space between “if” and “(“)
if(condition) { code; }
for
and while
Loops
for (i = 0; i < maxI; i++) { code; }
OR
for ( i = 0; i < maxI; i++ ) { code; }
NOT (no space between “for” and “(“)
for(i = 0; i < maxI; i++) { code; }
Note that when indexing through iterators, it is often slightly more
efficient to use the pre-increment form. Eg, ++iter
instead of iter++
forAll
, forAllIter
, forAllConstIter
, etc. loops
like for
loops, but
forAll(
NOT
forAll (
Using the forAllIter
and forAllConstIter
macros is generally
advantageous - less typing, easier to find later. However, since
they are macros, they will fail if the iterated object contains
any commas.
The following will FAIL!:
forAllIter(HashTable<labelPair, edge, Hash<edge> >, foo, iter)
These convenience macros are also generally avoided in other container classes and OpenFOAM primitive classes.
Splitting Over Multiple Lines
Splitting return type and function name
- split initially after the function return type and left align
- do not put “const” onto its own line - use a split to keep it with the function name and arguments.
so
const Foam::longReturnTypeName& Foam::longClassName::longFunctionName const
NOT
const Foam::longReturnTypeName& Foam::longClassName::longFunctionName const
NOR
const Foam::longReturnTypeName& Foam::longClassName::longFunctionName const
NOR
const Foam::longReturnTypeName& Foam::longClassName:: longFunctionName const
- if it needs to be split again, split at the function name (leaving behind the preceding scoping “::”s), and again, left align, i.e.
For example,
const Foam::longReturnTypeName& Foam::veryveryveryverylongClassName:: veryveryveryverylongFunctionName const
Splitting long lines at an “=”
Indent after split
variableName = longClassName.longFunctionName(longArgument);
OR (where necessary)
variableName = longClassName.longFunctionName ( longArgument1, longArgument2 );
NOT
variableName = longClassName.longFunctionName(longArgument);
NOR
variableName = longClassName.longFunctionName ( longArgument1, longArgument2 );
Maths and Logic
- operator spacing
- a + b, a - b
- a*b, a/b
- a & b, a ^ b
- a = b, a != b
- a < b, a > b, a >= b, a <= b
- a || b, a && b
- splitting formulae over several lines
Split and indent as per “splitting long lines at an “=”” with the operator on the lower line. Align operator so that first variable, function or bracket on the next line is 4 spaces indented i.e.
variableName = a * (a + b) - exp(c/d) * (k + t);
This is sometime more legible when surrounded by extra parentheses:
variableName = ( a * (a + b) - exp(c/d) * (k + t) );
- splitting logical tests over several lines
outdent the operator so that the next variable to test is aligned with the four space indentation, i.e.
if ( a == true && b == c )
Documentation
General
- For readability in the comment blocks, certain tags are used that are translated by pre-filtering the file before sending it to Doxygen.
- The tags start in column 1, the contents follow on the next lines and indented by 4 spaces. The filter removes the leading 4 spaces from the following lines until the next tag that starts in column 1.
- The ‘Class’ and ‘Description’ tags are the most important ones.
- The first paragraph following the ‘Description’ will be used for the
brief description, the remaining paragraphs become the detailed
description.
For example,
Class Foam::myClass Description A class for specifying the documentation style. The class is implemented as a set of recommendations that may sometimes be useful.
- The class name must be qualified by its namespace, otherwise Doxygen will think you are documenting some other class.
- If you don’t have anything to say about the class (at the moment), use the namespace-qualified class name for the description. This aids with finding these under-documented classes later.
Class Foam::myUnderDocumentedClass Description Foam::myUnderDocumentedClass
- Use ‘Class’ and ‘Namespace’ tags in the header files. The Description block then applies to documenting the class.
- Use ‘InClass’ and ‘InNamespace’ in the source files. The Description block then applies to documenting the file itself.
InClass Foam::myClass Description Implements the read and writing of files.
Doxygen Special Commands
Doxygen has a large number of special commands with a ‘' prefix or a (alternatively) an ‘@’ prefix.
The ‘@’ prefix form is recommended for most Doxygen specials, since it has the advantage of standing out. It also happens to be what projects like gcc and VTK are using.
The ‘' prefix form, however, looks a bit better for the ‘\n’ newline command and when escaping single characters - eg, ‘\@’, ‘\<’, ‘\>’, etc.