CodeStyle » History » Version 7
Alex Afanasyev, 01/27/2014 09:29 AM
1 | 1 | Junxiao Shi | # NFD code style guidelines |
---|---|---|---|
2 | |||
3 | NFD adopts [NDN Platform C++, C, C#, Java and JavaScript Code Guidelines](http://named-data.net/codebase/platform/documentation/ndn-platform-development-guidelines/cpp-code-guidelines/), with the following exceptions: |
||
4 | |||
5 | * (amended 10). Global variables should have `g_` prefix |
||
6 | * (amended 11). **All** class variables should have `m_` prefix. Static class variables should have `s_` prefix. |
||
7 | 5 | Alex Afanasyev | * (amended 31). Thrown exceptions can be either suffixed with ``Exception`` (SecurityException) or ``Error`` (SecurityError). Alternatively (and it is a recommended method), one should declare exception class ``Exception`` or ``Error`` as an inner class, from which the exception is thrown. For example, when declaring class Foo that can throw errors, one can write the following: |
8 | |||
9 | #include <stdexcept> |
||
10 | |||
11 | class Foo |
||
12 | { |
||
13 | struct Error : std::runtime_exception |
||
14 | { |
||
15 | Error(const std::string& what) : std::runtime_error(what) {} |
||
16 | }; |
||
17 | }; |
||
18 | |||
19 | In addition to that, if class Foo is a base class or interface for some class hierarchy, then child classes should should define their own ``Error`` or ``Exception`` classes that are inherited from the parent's Error class. |
||
20 | |||
21 | 1 | Junxiao Shi | * (amended 33). We will use only `.cpp` and `.hpp` extensions |
22 | 7 | Alex Afanasyev | * (removed 35). Lines should be within a reasonable range. >100 column-lines should be generally avoided. |
23 | 1 | Junxiao Shi | * (amended 44). Avoid C-style casts. Use `static_cast`, `dynamic_cast`, `reinterpret_cast`, `const_cast` where appropriate instead. |
24 | * (new). Exceptions can be used in the code, but should be used only in **exceptional** cases and not in the primary processing path. |
||
25 | * (new). When declaring/defining function/method, the return type should be put on a separate line before function/method name. |
||
26 | 4 | Junxiao Shi | * (amended 68). All three presented styles ARE acceptable. First and third ARE recommended (these are actually GNU styles). |
27 | 3 | Alex Afanasyev | * (amended 69). The class declarations should have the following form: |
28 | |||
29 | class SomeClass : public BaseClass |
||
30 | { |
||
31 | public: |
||
32 | 4 | Junxiao Shi | ... <public methods> ... |
33 | 3 | Alex Afanasyev | protected: |
34 | 4 | Junxiao Shi | ... <protected methods> ... |
35 | 3 | Alex Afanasyev | private: |
36 | ... <private methods> ... |
||
37 | |||
38 | public: |
||
39 | ... <public data> ... |
||
40 | protected: |
||
41 | ... <protected data> ... |
||
42 | private: |
||
43 | ... <private data> ... |
||
44 | }; |
||
45 | |||
46 | 4 | Junxiao Shi | ``public``, ``protected``, ``private`` may be repeated several times without interleaving (e.g. public, public, public, private, private) if this allows better readability of the code. |
47 | 3 | Alex Afanasyev | |
48 | * (amended 70) Method and function definitions should have the following form: |
||
49 | |||
50 | void |
||
51 | someMethod() |
||
52 | { |
||
53 | ... |
||
54 | } |