Coding Standards

This page describes the coding standard of OpenSn.

File names

Directory and file names should use snake style.

some_directory/file_name1.ext
some_directory/file_name2.ext
some_other_directory/another_file

C++ conventions

Macros

Macro names should use Pascal case, macro parameters should use snake style

#define MacroDefinition(macro_parameter)

Namespaces

Namespace names should use snake style.

namespace ns_one {

...

}

Enums

Enum names should use Pascal style, enum values upper case.

enum OurEnumType {
  ENUM_VALUE_1,
  ENUM_VALUE_2,
  ...
}

Static constants

Static constants should use upper case.

const int MY_CONSTANT = 10;

Classes and Structs

Class names should use Pascal style. Member variables should use snake style. Private and protected member variables should have trailing _ (underscore). Member functions should use Pascal style. Member function parameters should use snake style.

class ThisIsAClassName {
public:
   int public_member_var;
protected:
   int my_member_variable_;
   double another_member_variable_;

   void MyCoolMemberFunction();
   void MemberFunctionWithAnArgument(int argument_name);
};

struct ThisIsAStructName {
   int public_member_var;

   void MyCoolMemberFunction();
   void MemberFunctionWithAnArgument(int argument_name);

protected:
   int my_member_variable_;
   double another_member_variable_;
};

Getters and Setters

Getters should not include the prefix Get, however setters should use the Set prefix.

class MyClass
{
public:
  Type Member() { return member_; }
  void SetMember(Type type) { member_ = type; }

private:
  Type member_;
};

Numbers

Decimal numbers are written with both whole and fraction part. Examples: 1234.12, 1.0, 0.0, 1.0e-15.

Boolean operators

Boolean operators or, and and not should be used instead of ||, && and !.

Pointers

Shared pointers (std::shared) are preferred over raw pointers. Exception to this rule is when the code interacts with a 3rd party library like PETSc where shared pointers simply don’t exist.

Comments

In-code comments should use //.

// in-code comment
call();

For doxygen style comments /** */ should be used.

/**
 * In-code doxygen-style documentation
 */
void SomeMemberFunction()

Include directives

Preprocessor #include directives should be ordered as follows:

Header for this compilation unit (.h file that corresponds to .cc/.cpp file)
Other OpenSn headers
Non-standard, non-system libraries
C++ headers
C headers

There should not be empty lines separating the groups.

Example:

#include "modules/cfem_diffusion/cfem_diffusion_solver.h"
#include "framework/data_types/varying.h"
#include "petsc.h"
#include <string>
#include <map>

Command-line parameters

Command line parameters used by the OpenSn binary or any

References

Snake style: this_is_snake_style
Kebab style: this-is-kebab-style
Pascal style: ThisIsPascalStyle