Jump to content

Policies/Frameworks Coding Style: Difference between revisions

From KDE Community Wiki
Aseigo (talk | contribs)
--scrollbars
Aseigo (talk | contribs)
as agreed on in trysil (and just re-confirmed w/dfaure) we decided not to include this one exception as it is error prone, particularly for inexperienced developers
Line 7: Line 7:
* 4 Spaces instead of one tab
* 4 Spaces instead of one tab


== Variable delclaration ==
== Variable declaration ==
* Each variable declaration on a new line
* Each variable declaration on a new line
* Take useful names, no short names, except:
* Take useful names, no short names, except:
Line 76: Line 76:
</code>
</code>


Use curly braces when the body of a conditional statement contains more than one line, and also if a single line statement is somewhat complex.
Use curly braces even when the body of a conditional statement contains only one line.


Example:
Example:
<code cppqt>
<code cppqt>
// wrong
// wrong
if (true) {
    return true;
}
for (int i = 0; i < 10; ++i) {
    qDebug("%i", i);
}
// correct
if (true)
if (true)
     return true;
     return true;
Line 95: Line 86:
for (int i = 0; i < 10; ++i)
for (int i = 0; i < 10; ++i)
     qDebug("%i", i);
     qDebug("%i", i);
</code>


Exception 1: Use braces also if the parent statement covers several lines or wraps.
// correct
 
if (true) {
Example:
<code cppqt>
if (address.isEmpty() || !isValid()
    || !codec) {
    return false;
}
</code>
 
Exception 2: Use braces also in if-then-else blocks where either the if-code or the else-code covers several lines.
 
Example:
<code cppqt>
// wrong
if (true)
     return true;
     return true;
else {
    ++it;
    return false;
}
}


// correct
for (int i = 0; i < 10; ++i) {
if (true) {
     qDebug("%i", i);
    return true;
} else {
    ++it;
     return false;
}
}
</code>
</code>

Revision as of 22:23, 15 February 2007

This document describes the recommended coding style for kdelibs. Nobody is forced to use this style, but to have consistent formating of the source code files it is recommended to take use of it.

In short: Kdelibs coding style follows the Qt 4 coding style.

Indentation

  • No tabs
  • 4 Spaces instead of one tab

Variable declaration

  • Each variable declaration on a new line
  • Take useful names, no short names, except:
  • Single character variable names can be used for counters and temporary variables, where the purpose is obvious
  • Variables and functions start with a small letter
  • Each new word in a variable name starts with a capital letter
  • Avoid abbreviations

Example: // wrong KProgressBar *prbar; QString prtxt, errstr;

// correct KProgressBar *downloadProgressBar; QString progressText; QString errorString;

Whitespace

  • Use blank lines to group statements
  • Use only one empty line
  • Use one space after each keyword
  • For pointers or references, use a single space before '*' or '&', but not after
  • No space after a cast

Example: // wrong QString* myString; if(true){ }

// correct QString *myString; if (true) { }

Braces

As a base rule, the left curly brace goes on the same line as the start of the statement.

Example: // wrong if (true) { }

// correct if (true) { }

Exception: Function implementations and class declarations always have the left brace on the start of a line.

Example: void debug(int i) {

   qDebug("foo: %i", i);

}

class Debug { };

Use curly braces even when the body of a conditional statement contains only one line.

Example: // wrong if (true)

   return true;

for (int i = 0; i < 10; ++i)

   qDebug("%i", i);

// correct if (true) {

   return true;

}

for (int i = 0; i < 10; ++i) {

   qDebug("%i", i);

}

Switch statements

Case labels are on the same column as the switch

Example: switch (myEnum) { case Value1:

   doSomething();
   break;

case Value2:

   doSomethingElse();
   // fall through

default:

   defaultHandling();
   break;

}

Artistic Style (astyle) automatic code formatting

You can use astyle (<=1.19) to format code or to test if you have followed this document. Run the following command: astyle --indent=spaces=4 --brackets=linux \

      --indent-labels --pad=oper --unpad=paren \
      --one-line=keep-statements --convert-tabs \
      --indent-preprocessor \
      `find -type f -name '*.cpp'` `find -type f -name '*.h'`

Vim script

You can find a vim script in kdesdk/scripts/kde-devel-vim.vim that helps you to keep the coding style correct. In addition to defaulting to the kdelibs coding style it will automatically use the correct style for Solid and kdepim code. If you want to add rules for other projects feel free to add them in the SetCodingStyle function.

To use the script, include it in your ~/.vimrc like this: source /path/to/kde/sources/kdesdk/scripts/kde-devel-vim.vim


Document started by Urs Wolfer. Some parts of this document have been adopted from the Qt Coding Style document posted by Zack Rusin on kde-core-devel.