Difference between revisions of "C Coding Standard"
Line 1: | Line 1: | ||
[[Category:Coding Standards]] | [[Category:Coding Standards]] | ||
{{UnderConstruction}} | {{UnderConstruction}} | ||
+ | ==Language consideration== | ||
+ | * Avoid using assignment inside an expression. | ||
+ | * Never use '''goto''' | ||
+ | * Never use '''return''' if this is not the last instructions of a routine | ||
+ | * Use '''break''' only to separate branches of a switch statement | ||
+ | ==Style== | ||
* Always use curly braces, even when there is only one instruction. Instead of | * Always use curly braces, even when there is only one instruction. Instead of | ||
<c>if (a) do_something(); | <c>if (a) do_something(); | ||
Line 33: | Line 39: | ||
} | } | ||
</c> | </c> | ||
− | |||
− |
Revision as of 08:01, 22 July 2009
Language consideration
- Avoid using assignment inside an expression.
- Never use goto
- Never use return if this is not the last instructions of a routine
- Use break only to separate branches of a switch statement
Style
- Always use curly braces, even when there is only one instruction. Instead of
if (a) do_something();
write
if (a) { do_something(); }
- Place opening brace at the same line as the condition as in
while (expr) { ... } for (;;) { ... }
unless the expression is very long as in
if (very_long_expression) { ... }
- Surround else with braces as in
if (expr) { ... } else { ... }