mirror of
https://github.com/EGroupware/egroupware.git
synced 2024-11-07 16:44:07 +01:00
Some spel checkng and formating
This commit is contained in:
parent
76e773eb1d
commit
46780af615
@ -1,23 +1,27 @@
|
||||
1) Format your code so that we can read it. Please!
|
||||
2) Use tabs for formating. NOT SPACES. Tabs creates smaller files and editors allow
|
||||
developers to view a tab as however many spaces as they prefer. Spaces do not allow this.
|
||||
3) Use ' instead of " for strings. This is a performance issue, and prevents alot of inconsistant
|
||||
coding styles.
|
||||
4) Comments go on the line ABOVE the code. NOT to the right of the code!
|
||||
5) For each section of code put a section divider with basic explaination of the following
|
||||
code/functions.
|
||||
1) Format your code so that we can read it, please!
|
||||
|
||||
2) Use tabs for formatting, NOT SPACES. Tabs create smaller files and editors allow
|
||||
developers to view a tab as however many spaces as they prefer. Spaces do not allow this.
|
||||
|
||||
3) Use ' instead of " for strings. This is a performance issue, and prevents
|
||||
a lot of inconsistent coding styles.
|
||||
|
||||
4) Comments go on the line ABOVE the code, NOT to the right of the code!
|
||||
|
||||
5) For each section of code put a section divider with basic explanation of the following
|
||||
code/functions. It should look like this:
|
||||
|
||||
It should look like this:
|
||||
/****************************************************************************\
|
||||
* These functions are used to pick my nose *
|
||||
\****************************************************************************/
|
||||
|
||||
6) Do not document every bit of code in comments. PHP is an interpreted language and it will be
|
||||
nasty on performance
|
||||
6) Do not document every bit of code in comments. PHP is an interpreted language and it will be
|
||||
nasty on performance.
|
||||
|
||||
7) Use switch statements where many elseif's are going to be used. Switch is faster and I like it
|
||||
7) Use switch statements where many elseif's are going to be used. Switch is faster and I like it
|
||||
better!
|
||||
8) If statement need to follow the following format
|
||||
|
||||
8) 'If' statements need to use the following format:
|
||||
|
||||
if ($var == 'example')
|
||||
{
|
||||
@ -25,25 +29,29 @@ It should look like this:
|
||||
}
|
||||
else
|
||||
{
|
||||
echo 'This is not a test. This is the real thing';
|
||||
echo 'This is not a test. This is the real thing';
|
||||
}
|
||||
|
||||
Do NOT make if statements like this
|
||||
Do NOT make if statements like this:
|
||||
|
||||
if ($var == 'example'){ echo 'An example'; }
|
||||
|
||||
All other styles are not to be used. This is it. Use it or I will personally come and nag you to
|
||||
death.
|
||||
All other styles are not to be used. This is it. Use it or I will personally come and nag you to
|
||||
death.
|
||||
|
||||
9) ALL if statements HAVE to have match { }
|
||||
Do NOT create if statements like this:
|
||||
if ($a == b)
|
||||
dosomething();
|
||||
or:
|
||||
if ($a == b) dosomething();
|
||||
9) ALL 'if' statements MUST have matching { } (brackets). Do NOT create 'if' statements like this:
|
||||
|
||||
They make the code more difficault to read and follow
|
||||
if ($a == b)
|
||||
dosomething();
|
||||
|
||||
or:
|
||||
|
||||
if ($a == b) dosomething();
|
||||
|
||||
They make the code more difficult to read and follow.
|
||||
|
||||
10) class/function format:
|
||||
|
||||
10) class/function format
|
||||
class testing
|
||||
{
|
||||
function print_to_screen()
|
||||
@ -56,23 +64,23 @@ death.
|
||||
}
|
||||
else
|
||||
{
|
||||
echo 'This is not a test. This is the real thing';
|
||||
echo 'This is not a test. This is the real thing';
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Notice that when defining a class we break from the norm (putting } on same line as statement) and move the { to the next line. Dont ask me why, but this is what I/we do.
|
||||
11) Associative arrays must be written in the following manner:
|
||||
|
||||
11) Arrays must be written in the following way
|
||||
$array = array(
|
||||
'var' => 'value',
|
||||
'var2' => 'value2'
|
||||
);
|
||||
|
||||
Spaces are permitied between the => 's
|
||||
Note that spaces are permitted between the => 's
|
||||
|
||||
12) Use the long format for <?php do NOT use <?
|
||||
13) All code should start with 1 tab example:
|
||||
12) Use the long format for <?php. Do NOT use <?.
|
||||
|
||||
13) All code should start with 1 tab. Example:
|
||||
|
||||
<?php
|
||||
dosomething();
|
||||
@ -81,7 +89,7 @@ Notice that when defining a class we break from the norm (putting } on same line
|
||||
dosomemorestuff();
|
||||
}
|
||||
|
||||
NOT
|
||||
NOT:
|
||||
|
||||
<?php
|
||||
dosomething();
|
||||
@ -90,6 +98,5 @@ if ($a)
|
||||
dosomemorestuff();
|
||||
}
|
||||
|
||||
14) Use lower case for variable and function names. No stubly case code
|
||||
|
||||
14) Use lower case for variable and function names. No stubbly-case (mixed-case) code.
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user