CodingStandards
(New page: {{MainMenu}} Original source modified for our use http://framework.zend.com/manual/en/coding-standard.html ==Overview== ===Scope=== This document provides the guidelines and resources fo...) |
|||
Line 1: | Line 1: | ||
+ | {{Disclaimer}} | ||
{{MainMenu}} | {{MainMenu}} | ||
Line 442: | Line 443: | ||
* All of the arguments | * All of the arguments | ||
* All of the possible return values | * All of the possible return values | ||
− | |||
− | |||
− |
Revision as of 04:51, 12 May 2007
Important note: When you edit this page, you agree to release your contribution into the public domain. |
Contents |
Original source modified for our use http://framework.zend.com/manual/en/coding-standard.html
Overview
Scope
This document provides the guidelines and resources for developers and teams developing or developing on WoWRoster and it's AddOns.
The subjects covered are:
- PHP File Formatting
- Naming Conventions
- Coding Style
- Inline Documentation
Goals
Good coding standards are important in any development project, but particularly when multiple developers are working on the same project.
Having coding standards helps ensure that the code is of high quality, has fewer bugs, and is easily maintained.
PHP File Formatting
General
For files that contain only PHP code, the closing tag ("?>") is never permitted.
It is not required by PHP.
Not including it prevents trailing whitespace from being accidentally injected into the output.
Indentation
Use tabbed indents, not spaces.
Maximum Line Length
The target line length is 80-120 characters, i.e. developers should aim keep code as close to the 80-column boundary as is practical.
This just keeps the code more easily readable.
However, longer lines are acceptable and will happen, and we understand.
Line Termination
Line termination is the standard way for Unix text files. Lines must end only with a linefeed (LF).
Linefeeds are represented as ordinal 10, or hexadecimal 0x0A.
Do not use carriage returns (CR) like Macintosh computers (0x0D).
Do not use the carriage return/linefeed combination (CRLF) as Windows computers (0x0D, 0x0A).
Naming Conventions
Classes
WoWRoster employs a class naming convention whereby the names of the classes directly map to the directories in which they are stored.
The root level directory of WoWRoster is the "roster/" directory, under which all files are stored hierarchically.
Class names may only contain alphanumeric characters.
Numbers are permitted in class names but are discouraged.
Underscores are only permitted in place of the path separator -- the filename "roster/lib/table.php" should map to the class name "Roster_Lib_Table".
If a class name is comprised of more than one word, the first letter of each new word must be capitalized.
Successive capitalized letters are not allowed, e.g. a class "Roster_PDF" is not allowed while "Roster_Pdf" is acceptable.
These are examples of acceptable names for classes:
Roster_Db Roster_View Roster_View_Helper
IMPORTANT: Code that operates within Roster but is not part of Roster, e.g. code written by an AddOn Developer and not the WoWRoster Dev Team, should not start with "Roster_".
Interfaces
Interface classes must follow the same conventions as other classes (see above), however must end with the word "Interface", such as in these examples:
Zend_Log_Adapter_Interface Zend_Controller_Dispatcher_Interface
Filenames
For all other files, only alphanumeric characters, underscores, and the dash character ("-") are permitted.
Spaces are prohibited.
Filenames must always have all letters lower case.
Any file that contains any PHP code should end with the extension ".php".
These examples show the acceptable filenames for containing the class names from the examples in the section above:
roster/lib/wowdb.php roster/index.php roster/lib/functions.lib.php
File names should follow the mapping to class names described above.
Functions and Methods
Function names may only contain alphanumeric characters.
Underscores are discouraged.
Numbers are permitted in function names but are discouraged.
Function names must always start with a lowercase letter.
When a function name consists of more than one word, the first letter of each new word must be capitalized.
This is commonly called the "camelCaps" method.
Verbosity is encouraged.
Function names should be as verbose as is practical to enhance the understandability of code.
These are examples of acceptable names for functions:
filterInput() getElementById() widgetFactory()
For object-oriented programming, accessors for objects should always be prefixed with either "get" or "set". When using design patterns, such as the singleton or factory patterns, the name of the method should contain the pattern name where practical to make the pattern more readily recognizable.
Functions in the global scope ("floating functions") are permitted but discouraged. It is recommended that these functions should be wrapped in a static class.
Variables
Variable names may only contain alphanumeric characters.
Underscores are discouraged.
Numbers are permitted in variable names but are discouraged.
For class member variables that are declared with the "private" or "protected" construct, the first character of the variable name must be a single underscore.
This is the only acceptable usage of an underscore in a function name.
Member variables declared "public" may never start with an underscore.
Like function names (see section 3.3, above) variable names must always start with a lowercase letter and follow the "camelCaps" capitalization convention.
Verbosity is encouraged.
Variables should always be as verbose as practical.
Terse variable names such as "$i" and "$n" are discouraged for anything other than the smallest loop contexts.
If a loop contains more than 20 lines of code, the variables for the indices need to have more descriptive names.
Constants
Constants may contain both alphanumeric characters and the underscore.
Numbers are permitted in constant names.
Constants must always have all letters capitalized.
To enhance readability, words in constant names must be separated by underscore characters.
For example, EMBED_SUPPRESS_EMBED_EXCEPTION
is permitted but EMBED_SUPPRESSEMBEDEXCEPTION
is not.
Constants must be defined as class members by using the "const" construct.
Defining constants in the global scope with "define" is permitted but discouraged.
Coding Style
PHP Code Demarcation
PHP code must always be delimited by the full-form, standard PHP tags:
<?php ?>
Short tags are never allowed. For files containing only PHP code, the closing tag must always be omitted (See Section 2.1, “General”).
Strings
String Literals
When a string is literal (contains no variable substitutions), the apostrophe or "single quote" must always used to demarcate the string:
$a = 'Example String';
String Literals Containing Apostrophes
When a literal string itself contains apostrophes, it is permitted to demarcate the string with quotation marks or "double quotes". This is especially encouraged for SQL statements:
$sql = "SELECT `id`, `name` from `people` WHERE `name`='Fred' OR `name`='Susan'";
The above syntax is preferred over escaping apostrophes.
Variable Substitution
Variable substitution is permitted using either of these two forms:
$greeting = "Hello $name, welcome back!"; $greeting = "Hello {$name}, welcome back!";
For consistency, this form is not permitted:
$greeting = "Hello ${name}, welcome back!";
String Concatenation
Strings may be concatenated using the "." operator.
A space must always be added before and after the "." operator to improve readability:
$company = 'Zend' . 'Technologies';
When concatenating strings with the "." operator, it is permitted to break the statement into multiple lines to improve readability. In these cases, each successive line should be padded with whitespace such that the "."; operator is aligned under the "=" operator:
$sql = "SELECT `id`, `name` FROM `people` " . "WHERE `name` = 'Susan' " . "ORDER BY `name` ASC ";
Arrays
Numerically Indexed Arrays
Negative numbers are not permitted as indices.
An indexed array may be started with any non-negative number, however this is discouraged and it is recommended that all arrays have a base index of 0.
When declaring indexed arrays with the array construct, a trailing space must be added after each comma delimiter to improve readability:
$sampleArray = array(1, 2, 3, 'Zend', 'Studio');
It is also permitted to declare multi line indexed arrays using the "array" construct. In this case, each successive line must be padded with whitespace such that beginning of each line aligns as shown below:
$sampleArray = array( 1, 2, 3, 'Zend', 'Studio', $a, $b, $c, 56.44, $d, 500 );
Associative Arrays
When declaring associative arrays with the array construct, it is encouraged to break the statement into multiple lines.
In this case, each successive line must be padded with whitespace such that both the keys and the values are aligned:
$sampleArray = array( 'firstKey' => 'firstValue', 'secondKey' => 'secondValue' );
Classes
Class Declaration
Classes must be named by following the naming conventions.
The brace is always written on the line underneath the class name ("one true brace" form).
Every class must have a documentation block that conforms to the PHPDocumentor standard.
Any code within a class must be indented one tab space.
Only one class is permitted per PHP file.
Placing additional code in a class file is permitted but discouraged.
In these files, two blank lines must separate the class from any additional PHP code in the file.
This is an example of an acceptable class declaration:
/** * Documentation Block Here */ class SampleClass { // entire content of class // must be indented four spaces }
Class Member Variables
Member variables must be named by following the variable naming conventions.
Any variables declared in a class must be listed at the top of the class, prior to declaring any functions.
Accessing member variables directly by making them public is permitted but discouraged in favor of accessor variables (set/get).
Functions and Methods
Function and Method Declaration
Functions must be named by following the naming conventions.
Like classes, the brace is always written on the line underneath the function name ("one true brace" form).
There is no space between the function name and the opening parenthesis for the arguments.
Functions in the global scope are strongly discouraged.
This is an example of an acceptable function declaration in a class:
/** * Documentation Block Here */ class Foo { /** * Documentation Block Here */ function bar() { // entire content of function // must be indented using tabs } }
NOTE: Passing by-reference is permitted in the function declaration only:
/** * Documentation Block Here */ class Foo { /** * Documentation Block Here */ function bar(&$baz) { // entire content of function // must be indented using tabs } }
Call-time pass by-reference is prohibited.
The return value must not be enclosed in parentheses. This can hinder readability and can also break code if a method is later changed to return by reference.
/** * Documentation Block Here */ class Foo { /** * WRONG */ function bar() { return($this->bar); } /** * RIGHT */ function bar() { return $this->bar; } }
Function and Method Usage
Function arguments are separated by a single trailing space after the comma delimiter.
This is an example of an acceptable function call for a function that takes three arguments:
threeArguments(1, 2, 3);
Call-time pass by-reference is prohibited.
See the function declarations section for the proper way to pass function arguments by-reference.
For functions whose arguments permitted arrays, the function call may include the "array" construct and can be split into multiple lines to improve readability.
In these cases, the standards for writing arrays still apply:
threeArguments(array(1, 2, 3), 2, 3); threeArguments( array( 1, 2, 3, 'Zend', 'Studio', $a, $b, $c, 56.44, $d, 500 ), 2, 3 );
Control Statements
If / Else / Elseif
Control statements based on the if and elseif constructs must have a single space after the opening parenthesis of the conditional, and a single space before the closing parenthesis.
Within the conditional statements between the parentheses, operators must be separated by spaces for readability.
Inner parentheses are encouraged to improve logical grouping of larger conditionals.
The opening brace is always written on the line underneath the control structure name ("one true brace" form).
The closing brace is always written on its own line.
Any content within the braces must be indented one tab space.
if( $a != 2 ) { $a = 2; }
For "if" statements that include "elseif" or "else", the formatting must be as in these examples:
if( $a != 2 ) { $a = 2; } else { $a = 7; }
if( $a != 2 ) { $a = 2; } elseif( $a == 3 ) { $a = 4; } else { $a = 7; }
PHP allows for these statements to be written without braces in some circumstances.
The coding standard makes no differentiation and all "if", "elseif" or "else" statements must use braces.
Use of the "else if" construct is permitted but highly discouraged in favor of "elseif".
Switch
Control statements written with the "switch" construct must have a single space after the opening parenthesis of the conditional statement, and also a single space before the closing parenthesis.
All content within the "switch" statement must be indented one tab space.
Content under each "case" statement must be indented with additional tab spaces.
switch( $numPeople ) { case 1: break; case 2: break; default: break; }
The construct default may never be omitted from a switch statement.
NOTE: It is sometimes useful to write a case statement which falls through to the next case by not including a break or return in that case.
To distinguish these cases from bugs, any case statement where break or return are omitted must contain the comment "// break intentionally omitted".
Inline Documentation
Documentation Format
All documentation blocks ("docblocks") must be compatible with the phpDocumentor format.
Describing the phpDocumentor format is beyond the scope of this document.
For more information, visit: http://phpdoc.org
All source code file written for WoWRoster or that operates within the WoWRoster must contain a "file-level" docblock at the top of each file and a "class-level" docblock immediately above each class.
Below are examples of such docblocks.
Files
Every file that contains PHP code must have a header block at the top of the file that contains these phpDocumentor tags at a minimum:
/** * Short description for file * * Long description for file (if any)... * * LICENSE: Some license information * * @copyright 2007 WoWRoster.net * @license http://creativecommons.org/licenses/by-nc-sa/2.5 Attribution-NonCommercial-ShareAlike 2.5 * @version SVN: $Id: $ * @link http://www.wowroster.net * @since File available since Release 1.7.0 */
Classes
Every class must have a docblock that contains these phpDocumentor tags at a minimum:
/** * Short description for class * * Long description for class (if any)... * * @copyright 2007 WoWRoster.net * @license http://creativecommons.org/licenses/by-nc-sa/2.5 Attribution-NonCommercial-ShareAlike 2.5 * @version Release: package_version * @link http://www.wowroster.net * @since Class available since Release 1.7.0 * @deprecated Class deprecated in Release 2.0.0 */
Functions
Every function, including object methods, must have a docblock that contains at a minimum:
- A description of the function
- All of the arguments
- All of the possible return values