Difference between revisions of "Team Excellence - oop344 20113 Code Standards"

From CDOT Wiki
Jump to: navigation, search
(Comments)
(File Header)
Line 83: Line 83:
 
== File Header ==
 
== File Header ==
 
At the start of your files put a comment that defines the file and and declares when it was last updated and by whom
 
At the start of your files put a comment that defines the file and and declares when it was last updated and by whom
 +
 
<syntaxhighlight lang="cpp">
 
<syntaxhighlight lang="cpp">
// One Line File Name / Description
+
/**
// filename.h
+
*          File: test.cpp
//
+
*    Designed By: Sandip Patel
// Chad Pilkey
+
*          Date: 14th October, 2011
// Oct 13 2011
+
*    Description: To test xxx Module
// Version 1.0
+
*
//
+
*/
 
</syntaxhighlight>
 
</syntaxhighlight>

Revision as of 17:00, 14 October 2011


OOP344 | Weekly Schedule | Student List | Teams | Project | Student Resources

Code Standards

Fardad suggested in class today that we make a document that defines the format that our code is presented in so I've made this page to outline them.

The sections that follow are just ones that I could think of. If you know of other things feel free to add them

If Statement

if (condition){
  stuff;
}
else if (condition){
  stuff;
}
else {
  stuff;
}

Notice the space between the 'if' and '(' and the lack of a space between the ')' and '{'. Also the 'else' starts a line under the '}'.

If the "stuff" is only one line the curly braces can be omitted.

Dzmitry: I have a proposal. How about doing it like this:

if (condition){	
    stuff;
}else if (condition){
    stuff;
}else{
    stuff;
}

That will save a few lines, also space between "stuff" and beginning of the line should be default Visual Studio Tab (Mine is 4 spaces).

For Loop

for (i=0; condition; i++){
  stuff;
}

Notice the space after the 'for' and the lack of space between the ')' and '{'

Similar deal as with the if statement if "stuff" is only one like the curly braces can be omitted.

While Loop

while (true){
  stuff;
}

Notice the space after the 'while' and the lack of space after the ')' and '{'.

Also the curly braces can be omitted if "stuff" is only one line.

Function Declarations

void foo(int a){
  stuff;
}

The function header isn't indented at all and the first line follows immediately after with one indent.

Also there's no space between the ')' and '{'.

Indents

I think each indent should be the equivalent of two spaces. You can change the settings of Visual Studio to put in 2 spaces whenever you press tab. This will keep the code properly aligned and if someone isn't used to using tab for indent they can easily just use two spaces.

Dzmitry: I think at least 3-4 spaces will be better. First of all, it makes code easier to read (less characters on a page) and original console.cpp (at least for me) is written with 4 spaces spacing, so we won't need to change professor's code.

Class Member Variables

I think we should follow Fardad's lead and start all of our class' member variable names with an underscore '_' to differentiate them from other variables.

Comments

Dzmitry: We have two choices:

// this comment is an example
int foo(int x);

OR

int foo(int x);   // this comment is an example

I prefer the second one, because I got used to it, but I think that the second one is more affective, and it is the best choice, to put comments on the top line.

File Header

At the start of your files put a comment that defines the file and and declares when it was last updated and by whom

/**
*           File: test.cpp
*    Designed By: Sandip Patel
*           Date: 14th October, 2011
*    Description: To test xxx Module
*
*/