Appendix B: Writing Clear Code
The overarching goal when writing code is to make it
easy to read and to understand. Well-written programs
are easier to debug, easier to maintain, and have
fewer errors.
Writing a program is a lot like writing an essay.
When writing an essay, your message is more convincing when it is
accompanied by proper grammar and punctuation. When writing computer programs,
you should follow the same principle. It is even more important when programming
since someone may be assigned to maintain and support your code for long
periods of time. You will appreciate the importance of good style when it is
your task to understand and maintain someone else's code!
Coding.
- Keep programs and methods short and manageable.
- Use language-specific idioms.
- Use straightforward logic and flow-of-control.
- Avoid magic numbers (numbers other than −1, 0, 1, and 2); instead, give them meaningful symbolic names.
Naming conventions.
Here are some general principles when choosing names for your variables, methods, and classes.- Use meaningful names that convey the purpose of the variable. Choose names that are easy to pronounce, and avoid cryptic abbreviations. For example, use wagePerHour or hourlyWage instead of wph. Use polygon instead of p or pgon.
- Be consistent.
- Name boolean variables and methods so that their meaning is unambiguous, e.g., isPrime() or isEmpty() or contains().
- Use shorter names (e.g., i) for short-lived variables and loop-index variables. Use more descriptive names for variables that serve an important purpose.
- Avoid generic names like foo or tmp and meaningless names like fred. Use terminology from the application domain when possible.
- Name a constant by its meaning, not its value, e.g., name your variable DAYS_PER_WEEK instead of SEVEN.
IDENTIFIER NAMING RULES EXAMPLE Variables A short, but meaningful, name that communicates to the casual observer what the variable represents, rather than how it is used. Begin with a lowercase letter and use camel case (mixed case, starting with lower case). mass
hourlyWage
isPrimeConstant Use all capital letters, separating internal words with the underscore character. BOLTZMANN
MAX_HEIGHTClass A noun that communicates what the class represents. Begin with an uppercase letter and use camel case for internal words. Complex
Charge
PhoneNumberMethod A verb that communicates what the method does. Begin with a lowercase letter and use camelCase for internal words. move()
draw()
enqueue()
Commenting.
Programmers use comments to annotate a program and help the reader (or grader) understand how and why your program works. As a general rule, the code explains to the computer and programmer what is being done; the comments explain to the programmer why it is being done. Comments can appear anywhere within a program where whitespace is allowed. The Java compiler ignores comments.- Line comments. An end-of-line comment begins with // (two forward slashes) and ends at the end of the line on which the forward slashes appear. Any text from the // to the end of the line is ignored.
- Block comments. A block comment begins with /* (a forward slash and asterisk) and ends with */ (asterisk and a forward slash). Any text between these delimiters (even if it spans multiple lines) is ignored.
- Bold comments.
A bold comment is a special case of a block comment designed to draw
attention.
/*--------------------------------------------------------- * Here is a block comment that draws attention * to itself. *---------------------------------------------------------*/
- Javadoc comments. A Javadoc comment is a special case of a block comment that begins with /** (a forward slash and two asterisks). They are typically used to automatically generate the API for a class. Here are guidelines for writing Javadoc comments.
There is no widely agreed upon set of rules. Good programmers write code that documents itself.
- Make sure that comments agree with the code. Be careful to update the comments when you update the code.
- Do not write comments that merely restate the code.
Generally, comments should describe what
or why you are doing something, rather than how.
i++; // increment i by one
- Comment any potentially confusing code, or better yet, rewrite the code so that it isn't confusing.
- Comment every important variable name (including
all instance variables).
private double rx, ry; // position private double q; // charge
- Comment each method with a description of what it does.
Include what it takes as input, what it returns as output, and any
side effects. Use the parameter variable names in your description.
/* * Rearranges the elements of the array a[] in uniformly random order. * Throws an IllegalArgumentException if a is null. */ public static void shuffle(String[] a)
Whitespace.
Programmers use whitespace in their code to make it easier to read.- Don't put more than one statement on a line.
- Use blank lines to separate your code into logical sections.
- Put a space between all binary operators
(e.g., <=, =, +) and their operands.
One possible exception is to emphasize precedence.
a*x + b
- Include a space between a keyword (e.g., while, for, if) and its opening parenthesis.
- Put a space after each statement in a for loop.
for(int i=0;i<n;i++) vs. for (int i = 0; i < n; i++)
- Put a space after each comma in an argument list.
- Put space after each comment delimiter.
//This comment has no space // This comment has two //after the delimiter and is // spaces after the delimiter //difficult to read. // and is easier to read.
- Do not put spaces before a semicolon.
- Do not put spaces between an object name, the . separator, and a method name.
- Do not put spaces between a method name and its left parenthesis.
- Include blank lines to improve readability by grouping blocks of related code.
- Use spaces to align parallel code whenever it enhances readability.
int n = Integer.parseInt(args[0]); // size of population int trials = Integer.parseInt(args[1]); // number of trials
Indenting.
Programmers format and indent their code to reveal structure, much like an outline.- Avoid lines longer than 80 characters.
- Indent a fixed number of spaces. We recommend 4.
- Always use spaces instead of tabs. Modern IDEs (including IntelliJ) insert spaces when you type the tab key—these are known as soft tabs. Hard tabs are obsolete: in ancient times, they were used for data compression.
- Use a new indentation level for every level of nesting in your program.
- Follow either the K&R or BSD/Allman
indentation styles for
curly braces, and use it consistently. We consistently
use the former for the booksite and the latter in the textbook.
// K&R style indenting public static void main(String[] args) { System.out.println("Hello, World"); } // BSD-Allman style indenting public static void main(String[] args) { System.out.println("Hello, World"); }
Q + A
Q. Are there any official coding standards?
A. Here are Sun's Code Conventions for the Java Programming Language. However, this document was written in 1997 and is no longer being maintained.
Q. Any good references on programming style?
A. The Practice of Programming by Brian W. Kernighan and Rob Pike is a classic.
Q. Do Java comments nest?
A. No. So you cannot eliminate a block of code by simply surrounding it with the block comment delimiters (since the block itself may contain a */ delimiter).
Q. How can I autoindent my code?
A. Use an editor designed for writing code. For example, if you used our Java installer, IntelliJ will automatically indent and reformat your code when you save it.
Q. Are there tools for enforcing coding style?
A. Yes, we recommend Checkstyle. If you followed our Windows, Mac OS X, or Linux instructions, IntelliJ is configured to run Checkstyle automatically while you are editing.
Q. How can I write unmaintainable code?
A. Here's one guide to designing unmaintainable code and here's another.
Exercises
- Fun with comments.
What is the value of a after the following code fragment is executed?
//*/ a = 17; /*/ a = -17; //*/
- More fun with comments.
What does the following print?
public static void main(String[] args) { boolean nesting = true; /* /* */ nesting = false; // */ System.out.println(nesting); }
Answer: this code prints true if /* comments can nest and false if they can't. Since Java comments don't nest, it prints false.