[3238] | 1 | CODING-STANDARTS FOR ORXONOX |
---|
| 2 | -==============--==---+++++- |
---|
| 3 | In this file it is described how a orxonox-developer should structure his code. |
---|
| 4 | Every developer has to read it and follow the rules, and orxonox will grow. |
---|
[1855] | 5 | |
---|
[3238] | 6 | Contents: |
---|
| 7 | --------- |
---|
[1855] | 8 | 1.Coding Conventions |
---|
| 9 | 2.How to format your Code |
---|
[3238] | 10 | 3.Example |
---|
[1855] | 11 | |
---|
| 12 | 1.Coding Conventions |
---|
[3238] | 13 | ==================== |
---|
[1855] | 14 | ==> If you are beginning a new code-file: copy the proto_class.{cc,h} |
---|
| 15 | ==> and work with these files. |
---|
| 16 | |
---|
[3238] | 17 | 1.1.What has to be there: |
---|
| 18 | ------------------------- |
---|
| 19 | a) in every code file, there must be a GNU copyright header. |
---|
[1855] | 20 | |
---|
[3238] | 21 | b) under the (c) header write your name as main-programmer, if |
---|
| 22 | you are just bugfixing or extending write it under co-programmer. |
---|
[1855] | 23 | |
---|
[3238] | 24 | c) Every element of the must be documented with doxygen-tags. |
---|
[1855] | 25 | |
---|
[3238] | 26 | 1.2.Doxygen Tags: |
---|
| 27 | ----------------- |
---|
| 28 | Doxygen tags in general are comments inside a file, that look just a bit different. |
---|
| 29 | most of the time they extend the /* or // with a second star or a ! |
---|
| 30 | |
---|
| 31 | a) h-files: |
---|
| 32 | 1) every h-file you want to be documented you have to tag. (tag looks like /*! \file somefile */ ) |
---|
| 33 | 2) every class has to be Documented. ( //! what it does) |
---|
| 34 | 3) special variables can be documented. ( //!< wow this member is cool because ) |
---|
| 35 | |
---|
| 36 | b) cc-files: |
---|
| 37 | 1) all the methods must be documented, and all their parameters and return value must be covered. |
---|
| 38 | |
---|
| 39 | c) look out for this: Doxygen parses preprocessor arguments, |
---|
| 40 | so if some comments is excluded by a #ifdef, #endif |
---|
| 41 | doxygen will not parse it. And ther will be nothing in the docu. |
---|
| 42 | |
---|
| 43 | |
---|
[1855] | 44 | 2.How to format your Code |
---|
[3238] | 45 | ========================= |
---|
[1853] | 46 | We use the GNU conding convention (which is also used in xemacs etc.): |
---|
| 47 | |
---|
| 48 | -- Put a space after every comma. |
---|
| 49 | -- Put a space before the parenthesis that begins a function call, |
---|
| 50 | macro call, function declaration or definition, or control |
---|
| 51 | statement (if, while, switch, for). (DO NOT do this for macro |
---|
| 52 | definitions; this is invalid preprocessor syntax.) |
---|
| 53 | -- The brace that begins a control statement (if, while, for, switch, |
---|
| 54 | do) or a function definition should go on a line by itself. |
---|
| 55 | -- In function definitions, put the return type and all other |
---|
| 56 | qualifiers on a line before the function name. Thus, the function |
---|
| 57 | name is always at the beginning of a line. |
---|
| 58 | -- Indentation level is two spaces. (However, the first and following |
---|
| 59 | statements of a while/for/if/etc. block are indented four spaces |
---|
| 60 | from the while/for/if keyword. The opening and closing braces are |
---|
| 61 | indented two spaces.) |
---|
| 62 | -- Variable and function names should be all lowercase, with underscores |
---|
| 63 | separating words, except for a prefixing tag, which may be in |
---|
| 64 | uppercase. Do not use the mixed-case convention (e.g. |
---|
| 65 | SetVariableToValue ()) and *especially* do not use Microsoft |
---|
| 66 | Hungarian notation (char **rgszRedundantTag). |
---|
| 67 | -- preprocessor and enum constants should be all uppercase, and should |
---|
| 68 | be prefixed with a tag that groups related constants together. |
---|
| 69 | |
---|
| 70 | |
---|
[3238] | 71 | |
---|
| 72 | |
---|
| 73 | 3.Example |
---|
| 74 | ========= |
---|
| 75 | 3.1.Header |
---|
| 76 | ---------- |
---|
| 77 | A standard header has the same name as the Class it handles. |
---|
| 78 | |
---|
| 79 | someclass.h |
---|
| 80 | ----------- |
---|
| 81 | /*! |
---|
| 82 | \file someclass.h |
---|
| 83 | \brief Some short description of the someclass |
---|
| 84 | \todo maybe there remains something to do in this entire file |
---|
| 85 | |
---|
| 86 | Here comes a longer description |
---|
| 87 | this can also be longer than one line |
---|
| 88 | */ |
---|
| 89 | |
---|
| 90 | #ifndef _SOMECLASS_H |
---|
| 91 | #define _SOMECLASS_H |
---|
| 92 | |
---|
| 93 | #include <many_headers> |
---|
| 94 | #include "stdincl.h" |
---|
| 95 | |
---|
| 96 | //! short description of the class |
---|
| 97 | /** |
---|
| 98 | \todo what remains to be done here |
---|
| 99 | |
---|
| 100 | longer description |
---|
| 101 | */ |
---|
| 102 | class SomeClass |
---|
| 103 | { |
---|
| 104 | private: |
---|
| 105 | int usefull_variable; |
---|
| 106 | int cool_variable; //!< this variable is cool, because... |
---|
| 107 | |
---|
| 108 | protected: |
---|
| 109 | char* someOtherMember; |
---|
| 110 | |
---|
| 111 | public: |
---|
| 112 | SomeClass (void); |
---|
| 113 | ~SomeClass (void); |
---|
| 114 | |
---|
| 115 | int mightyFunction (char* name, int value); |
---|
| 116 | //.... |
---|
| 117 | }; |
---|
| 118 | |
---|
| 119 | #endif /* _SOMECLASS_H */ |
---|
| 120 | |
---|
| 121 | |
---|
| 122 | |
---|
| 123 | 3.2.CC-Files |
---|
| 124 | ------------ |
---|
| 125 | CC-files must be named the same as the .h-files they belong to. |
---|
| 126 | |
---|
| 127 | someclass.cc |
---|
| 128 | ------------ |
---|
| 129 | |
---|
| 130 | /* |
---|
| 131 | orxonox - the future of 3D-vertical-scrollers |
---|
| 132 | |
---|
| 133 | Copyright (C) 2004 orx |
---|
| 134 | |
---|
| 135 | This program is free software; you can redistribute it and/or modify |
---|
| 136 | it under the terms of the GNU General Public License as published by |
---|
| 137 | the Free Software Foundation; either version 2, or (at your option) |
---|
| 138 | any later version. |
---|
| 139 | |
---|
| 140 | ### File Specific: |
---|
| 141 | main-programmer: The Name of Myself |
---|
| 142 | co-programmer: ... |
---|
| 143 | */ |
---|
| 144 | |
---|
| 145 | #include "someclass.h" |
---|
| 146 | |
---|
| 147 | |
---|
| 148 | /** |
---|
| 149 | \brief <a brief description> |
---|
| 150 | \todo I think you get it. |
---|
| 151 | |
---|
| 152 | <more description> |
---|
| 153 | */ |
---|
| 154 | SomeClass::SomeClass (void) |
---|
| 155 | { |
---|
| 156 | //constructor-stuff |
---|
| 157 | } |
---|
| 158 | |
---|
| 159 | /** |
---|
| 160 | \brief <a brief description> |
---|
| 161 | \todo if something is not destructed it will be here |
---|
| 162 | |
---|
| 163 | <more description> |
---|
| 164 | */ |
---|
| 165 | ~SomeClass::~SomeClass (void) |
---|
| 166 | { |
---|
| 167 | //destroy all te allocated space of the Class |
---|
| 168 | } |
---|
| 169 | |
---|
| 170 | /** |
---|
| 171 | \brief <a brief description> |
---|
| 172 | \param name <what is this parameter for> |
---|
| 173 | \param valuse <what for...> |
---|
| 174 | \returns <what it returns> |
---|
| 175 | |
---|
| 176 | <more description> |
---|
| 177 | */ |
---|
| 178 | int SomeClass::mightyFuncion (char* name, int value) |
---|
| 179 | { |
---|
| 180 | this->coolVariable = value; |
---|
| 181 | // and so on..... |
---|
| 182 | } |
---|
| 183 | |
---|
| 184 | // ... |
---|