CS 246 Style Guide
Prof. Richard Eisenberg
Spring 2017
All organizations and companies have specific conventions for formatting code. While these formatting rules may vary from place to place, they are essential for making your code readable and for enabling others to understand, use, and modify your code in the future.
While I do not expect anyone (including myself) to follow all the rules, here is a very detailed guide used as an inspiration for the notes below.
radius
, not r
and not rad
.num_students
.#define
are written in ALL_CAPS._t
suffix.The most-readable programs are written with prudent use of whitespace (including both blank lines and spaces).
{
, indent by at least 2 spaces until the matching }
.Every source code file should contain a header comment that describes the contents of the file and other pertinent information. It must include the following information:
For example:
/*****************************************
* Assignment 4
* Name: Barbara Smith
* E-mail: bsmith22@brynmawr.edu
*
* The main driver program for project 4.
*
* This program reads the file specified as the first command line
* argument, counts the number of words, spaces, and characters and
* displays the results in the format specified in the project description.
*
*****************************************/
All global variables must be commented. Most local variables should be commented, too.
All members of a struct
definition must be commented.
All functions should be commented. The comments should explain what the function does, what its parameters are (not just their types!), and what it returns. Another way to consider function comments is to imagine a contract: “If the caller of the function does xyz, then the function will do abc.” The “xyz” there are called pre-conditions, and the “abc” there are called the post-conditions.
You should strive for your code to be self-explanatory. However, it is inevitable that some lines of code are more intricate. In these cases, a comment describing the code is well-advised. The comment should not simply translate the code to English, but should explain what’s really going on. For example:
// Unhelpful comment:
star_sides = 5; // set star_sides to 5
// Helpful comment:
star_sides = 5; // reset star_sides to original value
Well-structured code will be broken into logical sections that perform a simple task. Each of these sections of code (typically starting with an if
statement or a loop) should be documented.
An in-line comment too long to appear to the right of your code appears above the code to which it applies and is indented to the same level as the code. For example:
// increment all the odd values in the array
for (i = 0; i < n; i++) {
// add 1 only to the odd values
if (array[i] % 2 == 1) {
array[i] = array[i] + 1;
}
}
Choose one of the two styles and use it consistently (note how the braces are placed):
if (condition) { if (condition)
... {
} else if (condition) { ...
... }
} else { else if (condition)
... {
} ...
}
else
{
...
}
for (loop control expressions) { for (loop control expressions)
... {
} ...
}
while (condition) { while (condition)
... {
} ...
}
do { do
... {
} while (condition); ...
} while (condition);
switch (variable) { switch (variable)
case constant1: ... {
break; case constant1: ...
case constant2: ... break;
break; case constant2: ...
case default: ... break;
} case default: ...
}