Title: Mavibot Coding Standards
Notice: Licensed to the Apache Software Foundation (ASF) under one
    or more contributor license agreements.  See the NOTICE file
    distributed with this work for additional information
    regarding copyright ownership.  The ASF licenses this file
    to you under the Apache License, Version 2.0 (the
    "License"); you may not use this file except in compliance
    with the License.  You may obtain a copy of the License at
    .
    http://www.apache.org/licenses/LICENSE-2.0
    .
    Unless required by applicable law or agreed to in writing,
    software distributed under the License is distributed on an
    "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    KIND, either express or implied.  See the License for the
    specific language governing permissions and limitations
    under the License.

# Mavibot Coding Standards

Welcome to you fellow developer.  You've maybe been elected committer on the project, or want to contribute some code / patch? This is great news! However, in order to be able to share your 'vision' with your code, some rules must be followed.

Hey, remember that those rules are not the best nor the worst, they are pretty much what they are for historical reasons, or for technical reasons, however, please, accept them as they are, and avoid religious war (please, oh please, no mail to say "WTF ? You are using spaces instead of tab ??? How stupid is this rule etc etc.) Rules are **alway*s* stupid, but smart people follow them ;)

**eclipse IDE**

Eclipse users can import those two files to enforce the code formating : [formatting.xml](http://svn.apache.org/repos/asf/directory/project/trunk/resources/formatting.xml) and [codetemplates.xml](http://svn.apache.org/repos/asf/directory/project/trunk/resources/codetemplates.xml)

**IDEA IDE**

IDEA users can import [this file](settings.jar) to enforce the code formating.


## Headers

First, you **must** (and this rule accept no exception) use this header in top of all source file, or each file in which you can have comments :


	:::java
	/*
	 *  Licensed to the Apache Software Foundation (ASF) under one
	 *  or more contributor license agreements.  See the NOTICE file
	 *  distributed with this work for additional information
	 *  regarding copyright ownership.  The ASF licenses this file
	 *  to you under the Apache License, Version 2.0 (the
	 *  "License"); you may not use this file except in compliance
	 *  with the License.  You may obtain a copy of the License at
	 *
	 *    http://www.apache.org/licenses/LICENSE-2.0
	 *
	 *  Unless required by applicable law or agreed to in writing,
	 *  software distributed under the License is distributed on an
	 *  "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
	 *  KIND, either express or implied.  See the License for the
	 *  specific language governing permissions and limitations
	 *  under the License.
	 *
	 */

### Class/Interface headers

Each **Class** or *Interface* should have an header which must contains :

* A description of this class/interface
* an *author* tag which should be :

		:::java
		@author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>

<DIV class="note" markdown="1">
Thanks to avoid to put your name. The code is not yours, and much more important, but putting your name and e-mail, you will intimidate other developer ("Oh, no, I won't mess with this code, it has been developed by XXXX \!") and second, you will receive mail in three years even if you have stopped all commitment on the project (and those who have sent you an e-mail will think that the project's member are not responsive...)
</DIV>

If you use **html** tags, remember to escape '<' and '>' characters...

### Static members and other members

Just add a single line javadoc comment like : _/** blah ... */_ before each member

### Methods

Follow the standard **javadoc** rules : Description, **@param**, **@exception** and **@return**. It should be enough. Avoid **@tags**, **@todo** tags, etc...

Escape **html** characters

## Comments

No special rules, except that you should avoid :

* Useless comments like : i++; /\* Increment i \*/
* Overusing comments : if you have to heavily comment a piece of code, then this piece of code might be too complex ...
* Spreading little comments all over a method : if possible, write blocs of comments. The method header generally contains a full description of the code, and if it's not the case, just consider your method might be too long !
* Dead code commented. If it's dead, then put it in a coffin. We use SVN, the Ressuscitator !

Basically, use your common sense :-)

## Naming

Naming ! Sounds like Blaming :-). Ok. We use **Sun(tm/c/r)** style :

* Constants are in UPPER CASE with accepted '_'
* Class starts with an uppercase and each starting word is upper cased. No '_', please !
* Methods starts with lower case and then follow the same rule than classes. No '_', please !
* Interfaces should not start with an 'I'
* Classes which implements an Interface must be followed by the postfix 'impl'
* Variables follow the method naming convention. No '_', please !
* Use meaningful names.
* No double letter variables like ii, jj etc...

If you browse the code, you will see that many classes does not respect those rules. That's life ! Don't fix it if you don't touch a class. If you are fixing a method in a class, then you can change the code to respect the rules. Little by little, we may reach a stable state where all the code respect the rules ;)

Naming is really important for **APIs**. Be smart. If you are not sure, ask.

## Spaces vs tabs

<DIV class="warning" markdown="1">
**FOUR SPACES, NO TAB. Final.**
</DIV>

No discussion. Using tabs break diffs. Modify your **IDE** to insert spaces when you use tabs, before it saves the file.

## Formatting

Use the **formatting.xml** file which can be found in the **resources** directory in the root of the project. This is for *Eclipse*. If you don't use eclipse, then translate the formating to your favorite **IDE**.

Use the **codetemplates.xml** file if you are using *Eclipse* too. You will find it at the same location. It brings you some standard headers for new classes, nex methods, etc.

Use **UTF-8** as a default for your files (except for properties, thanks to **java**, which should be in **ISO-8859-1**). Forget about exotic encoding...

<DIV class="warning" markdown="1">
**DO NOT USE AN AUTOMATIC FORMATER FOR COMMENTS!!!**
</DIV>

People spend a lot of time making their comment looks like pretty, so if you just format them, you will have to recover the previous comments...

Some general rules :

* Always use '{' and '}' even for a single instruction, or if you have an empty block (don't use ';' for empty blocks)
* No more than one instruction on a single line, the only exception is the '?' ':' operation
* Use *this* to address the class variable if there is a risk of confusion (for instance if you have a parameter with the same name.
* Don't add a 'a_', or 'the_' before a parameter's name to distinguish it from the class variable which has the same name. Use **this** instead.
* Don't add **final** everywhere. Even if **final** is a substitute for **const**, it's semantic is not clear enough that you use it everywhere.
* Add spaces in method calls after '(' and before ')'
* '{' and '}' must be on the same column

This is a code example :

	:::java    
	...
    int result = myMethod( param1, param2 )
    
    if ( result > 0 )
    {
        // do something
    }
    ...


## Imports

Always declare all the classes you import, do not use **x.y.\***

## What else ?

Well, this was a very short introduction about coding rules. Use common sense, look at what you see around you when adding some code, ask people about format, if you have a question.

That's it ! (I wait your comments, guys :-)