diff --git a/javaguide.css b/javaguide.css new file mode 100644 index 0000000..4571058 --- /dev/null +++ b/javaguide.css @@ -0,0 +1,507 @@ +table { + border-collapse: collapse; +} + +td, th { + border: 1px solid #ccc; + padding: 2px 12px; + font-size: 10pt; +} + +code, samp, var { + color: #060; +} + +pre { + font-size: 10pt; + display: block; + color: #060; + background-color: #e8fff6; + border-color: #f0fff0; + border-style: solid; + border-top-width: 1px; + border-bottom-width: 1px; + border-right-width: 1px; + border-left-width: 5px; + padding-left: 12px; + padding-right: 12px; + padding-top: 4px; + padding-bottom: 4px; +} + +pre.badcode { + color: #c00; + background-color: #ffe6d8; + border-color: #fff0f0; +} + +hr { + margin-top: 3.5em; + border-width: 1px; + color: #fff; +} + +html { + margin-top:2em; + margin-left:10%; + margin-right:10%; + padding:0; +} + +.bp-reset-element, +body, +h1, +h2, +h3, +h4, +h5, +h6, +article, +aside, +details, +figcaption, +figure, +footer, +header, +hgroup, +menu, +nav, +section, +summary, +blockquote, +q, +th, +td, +caption, +table, +div, +span, +object, +iframe, +p, +pre, +a, +abbr, +acronym, +address, +code, +del, +dfn, +em, +img, +dl, +dt, +dd, +ol, +ul, +li, +fieldset, +form, +label, +legend, +caption, +tbody, +tfoot, +thead, +tr { + margin:0; + padding:0; + border:0; + font-weight:inherit; + font-style:inherit; + font-size:100%; + font-family:inherit; + vertical-align:baseline; +} + +body { + font-family:'Arial', sans-serif; + font-size:81.25%; + color:#222; + background-color:#fff; + line-height:1.67; + overflow: auto; +} + +.change { + text-align: right; + margin-bottom:1em; +} + +h1, +h2, +h3, +h4, +h5, +h6 { + font-weight:bold; +} + +h1 { + margin-bottom:.50em; + text-align: center +} + +h2, +h3, +h4, +h5, +h6 { + margin-top:1.5em; + margin-bottom:.75em; +} + +h1 {font-size:200%;} +h2 {font-size:167%;} +h3 {font-size:133%;} +h4 {font-size:120%;} +h5 {font-size:110%;} + +p { + margin:0 0 1.5em; +} + +a[href=''] { + cursor:default; +} + +h1 img, +h2 img, +h3 img, +h4 img, +h5 img, +h6 img { + margin:0; +} + +a img { + border:none; +} + +pre { + margin:1.5em 0; + white-space:pre; +} + +pre, +code, +kbd, +tt { + font:1em 'Droid Sans Mono', monospace; + line-height:1.5; +} + +dl { + margin:0 0 1.5em 0; +} + +dl dt { + font-weight:bold; +} + +dd { + margin-left:1.5em; +} + +hr { + height:0; + border:0; + border-top:1px solid #ccc; + background-color:#ccc; +} + +table { + border:1px solid #bbb; + border-spacing:0; + border-collapse:collapse; + margin:0 0 1.5em; + vertical-align:middle; + width:100%; +} + +table.unlined, +table.unlined th, +table.unlined tr, +table.unlined td { + border:0; +} + +th, +td, +caption { + float:none !important; + text-align:left; + font-weight:normal; + vertical-align:middle; + padding:4px; +} + +caption { + padding:0; +} + +td { + border:1px solid #bbb; + vertical-align:top; +} + +th { + border:0; + border-bottom:1px solid black; + font-weight:bold; + background:rgb(229, 236, 249); +} + +table th code { + background-color:inherit; + color:inherit; +} + +table tfoot th { + border:1px solid #bbb; +} + +tfoot { + font-style:italic; +} + +caption { + background:#eee; +} + +table[border='0'] { + border:none; +} + +table[border='0']>tbody>tr>td, +table[border='0']>tr>td { + border:none; +} + +tr.alt td, +td.alt { + background-color:#efefef; +} + +table.striped tr:nth-child(even) td, +table tr.even td { + background:#efefef; +} + +table.columns { + border:none; +} + +table.columns>tbody>tr>td, +table.columns>tr>td { + border:none; + padding:0 3em 0 0; +} + +table.columns>tbody>tr>td:last-child, +table.columns>tr>td:last-child { + border:none; + padding:0; +} + +ul, +ol { + margin:0 1.5em 1.5em 0; + padding-left:2em; +} + +li ul, +li ol { + margin:0; +} + +ul { + list-style-type:disc; +} + +ol { + list-style-type:decimal; +} + +ul { + list-style-type:disc; +} + +ul ul { + list-style-type:circle; +} + +ul ul ul { + list-style-type:square; +} + +ul.disc { + list-style-type:disc; +} + +ul.circle { + list-style-type:circle; +} + +ul.square { + list-style-type:square; +} + +ol { + list-style-type:decimal; +} + +ol ol { + list-style-type:lower-alpha; +} + +ol ol ol { + list-style-type:lower-roman; +} + +ol ul { + list-style-type:circle; +} + +ol.decimal { + list-style-type:decimal; +} + +ol.upper-alpha { + list-style-type:upper-alpha; +} + +ol.lower-alpha { + list-style-type:lower-alpha; +} + +ol.upper-roman { + list-style-type:upper-roman; +} + +ol.lower-roman { + list-style-type:lower-roman; +} + +ol.nolist, +ul.nolist { + padding-left:0; + list-style-image:none; + list-style-type:none; + margin-left:0; +} + +.center { + text-align:center; +} + +code, +kbd, +pre { + color:#007000; +} + +kbd { + font-weight: bold; +} + +table.striped code { + background-color:inherit; +} + +pre { + padding:6px 10px; + background-color:#FAFAFA; + border:1px solid #bbb; + overflow:auto; +} + +pre.prettyprint { + padding:6px 10px !important; + border:1px solid #bbb !important; +} + +code.bad, code.badcode { + background-color:#fcc; +} +pre.bad, pre.badcode { + background-color:#fcc; + border-top:1px inset #a03; + border-left:1px inset #a03; +} + +.tip { + background-color:#fffbd9; + padding:6px 8px 6px 10px; + border-left:6px solid #ffef70; +} + +.note { + background-color:#e5ecf9; + padding:6px 8px 6px 10px; + border-left:6px solid #36c; +} + +@media print { + + .str { + color:#060; + } + + .kwd { + color:#006; + font-weight:bold; + } + + .com { + color:#600; + font-style:italic; + } + + .typ { + color:#404; + font-weight:bold; + } + + .lit { + color:#044; + } + + .pun, + .opn, + .clo { + color:#440; + } + + .pln { + color:#000; + } + + .tag { + color:#006; + font-weight:bold; + } + + .atn { + color:#404; + } + + .atv { + color:#060; + } + + h1 { + font-style:italic; + } +} + +ol.linenums { + margin-top:0; + margin-bottom:0; +} + +code { + background-color:#FAFAFA; + padding: 0.25em 0.5em; + white-space: nowrap +} diff --git a/javaguide.html b/javaguide.html new file mode 100644 index 0000000..d09f49e --- /dev/null +++ b/javaguide.html @@ -0,0 +1,665 @@ + +
+ + + + +This document serves as the complete definition of Google's coding standards for +source code in the Java™ Programming Language. A Java source file is described as being in +Google Style if and only if it adheres to the rules herein.
Like other programming style guides, the issues covered span not only aesthetic issues of +formatting, but other types of conventions or coding standards as well. However, this document +focuses primarily on the hard-and-fast rules that we follow universally, and +avoids giving advice that isn't clearly enforceable (whether by human or tool). +
+In this document, unless otherwise clarified:
@interface
).Other "terminology notes" will appear occasionally throughout the document.
+Example code in this document is non-normative. That is, while the examples +are in Google Style, they may not illustrate the only stylish way to represent the +code. Optional formatting choices made in examples should not be enforced as rules.
+The source file name consists of the case-sensitive name of the top-level class it contains,
+plus the .java
extension (aside from package-info.java
files).
Source files are encoded in UTF-8.
+Aside from the line terminator sequence, the ASCII horizontal space +character (0x20) is the only whitespace character that appears +anywhere in a source file. This implies that:
For any character that has a special escape sequence
+(\b
,
+\t
,
+\n
,
+\f
,
+\r
,
+\"
,
+\'
and
+\\
), that sequence
+is used rather than the corresponding octal
+(e.g. \012
) or Unicode
+(e.g. \u000a
) escape.
For the remaining non-ASCII characters, either the actual Unicode character
+(e.g. ∞
) or the equivalent Unicode escape
+(e.g. \u221e
) is used, depending only on which
+makes the code easier to read and understand.
Tip: in the Unicode escape case, and occasionally even when actual Unicode characters +are used, an explanatory comment can be very helpful.
Examples:
Example | Discussion |
---|---|
String unitAbbrev = "μs"; | Best: perfectly clear even without a comment. |
String unitAbbrev = "\u03bcs"; // "μs" | Allowed, but there's no reason to do this. |
String unitAbbrev = "\u03bcs";
+ // Greek letter mu, "s" | Allowed, but awkward and prone to mistakes. |
String unitAbbrev = "\u03bcs"; | Poor: the reader has no idea what this is. |
return '\ufeff' + content;
+ // byte order mark | Good: use escapes for non-printable characters, and comment if necessary. |
Tip: Never make your code less readable simply out of fear that some programs might +not handle non-ASCII characters properly. If that should happen, those programs +are broken and they must be fixed.
+A source file consists of, in order:
Exactly one blank line separates each section that is present.
+If license or copyright information belongs in a file, it belongs here.
+The package statement is not line-wrapped. The column limit (Section 4.4, +Column limit: 80 or 100) does not apply to package statements.
+Wildcard imports, static or otherwise, are not used.
+Import statements are not line-wrapped. The column limit (Section 4.4, +Column limit: 80 or 100) does not apply to import +statements.
+Import statements are divided into the following groups, in this order, with each group +separated by a single blank line:
com.google
imports
+ (only if this source file is in the com.google
package
+ space)android
, com
, junit
, org
,
+ sun
java
importsjavax
importsWithin a group there are no blank lines, and the imported names appear in ASCII sort +order. (Note: this is not the same as the import statements being in +ASCII sort order; the presence of semicolons warps the result.)
+Each top-level class resides in a source file of its own.
Exception: of course, no such class appears in package-info.java
+files.
The ordering of the members of a class can have a great effect on learnability, but there is +no single correct recipe for how to do it. Different classes may order their members +differently.
What is important is that each class order its members in some logical +order, which its maintainer could explain if asked. For example, new methods are not +just habitually added to the end of the class, as that would yield "chronological by date +added" ordering, which is not a logical ordering.
+When a class has multiple constructors, or multiple methods with the same name, these appear +sequentially, with no intervening members.
+Terminology Note: block-like construct refers to the body of a class, method or +constructor. Note that, by Section 4.8.3.1 on +array initializers, any array initializer +may optionally be treated as if it were a block-like construct.
+Braces are used with
+if
,
+else
,
+for
,
+do
and
+while
statements, even when the
+body is empty or contains only a single statement.
Braces follow the Kernighan and Ritchie style +("Egyptian brackets") +for nonempty blocks and block-like constructs:
else
or a
+ comma.Example:
+return new MyClass() { + @Override public void method() { + if (condition()) { + try { + something(); + } catch (ProblemException e) { + recover(); + } + } + } +}; +
A few exceptions for enum classes are given in Section 4.8.1, +Enum classes.
+An empty block or block-like construct may be closed immediately after it is
+opened, with no characters or line break in between
+({}
), unless it is part of a
+multi-block statement (one that directly contains multiple blocks:
+if/else-if/else
or
+try/catch/finally
).
Example:
+ void doNothing() {} ++
Each time a new block or block-like construct is opened, the indent increases by two +spaces. When the block ends, the indent returns to the previous indent level. The indent level +applies to both code and comments throughout the block. (See the example in Section 4.1.2, +Nonempty blocks: K & R Style.)
+Each statement is followed by a line-break.
++ Projects are free to choose a column limit of either 80 or 100 characters. + +Except as noted below, any line that would exceed this limit must be line-wrapped, as explained in +Section 4.5, Line-wrapping. +
Exceptions:
package
and
+ import
statements (see Sections
+ 3.2 Package statement and
+ 3.3 Import statements).Terminology Note: When code that might otherwise legally occupy a single line is divided into +multiple lines, typically to avoid overflowing the column limit, this activity is called +line-wrapping.
There is no comprehensive, deterministic formula showing exactly how to line-wrap in +every situation. Very often there are several valid ways to line-wrap the same piece of code.
Tip: extracting a method or local variable may solve the problem without the need to +line-wrap.
+The prime directive of line-wrapping is: prefer to break at a +higher syntactic level. Also:
.
), the ampersand in type bounds
+ (<T extends Foo & Bar>
), and the pipe in
+ catch blocks
+ (catch (FooException | BarException e)
).for
("foreach") statement.(
) that follows it.,
) stays attached to the token that
+ precedes it.When line-wrapping, each line after the first (each continuation line) is indented +at least +4 from the original line.
When there are multiple continuation lines, indentation may be varied beyond +4 as +desired. In general, two continuation lines use the same indentation level if and only if they +begin with syntactically parallel elements.
Section 4.6.3 on Horizontal alignment addresses +the discouraged practice of using a variable number of spaces to align certain tokens with +previous lines.
+A single blank line appears:
Multiple consecutive blank lines are permitted, but never required (or encouraged).
+Beyond where required by the language or other style rules, and apart from literals, comments and +Javadoc, a single ASCII space also appears in the following places only.
if
,
+ for
or
+ catch
, from an open parenthesis
+ ((
)
+ that follows it on that lineelse
or
+ catch
, from a closing curly brace
+ (}
) that precedes it on that line{
), with two exceptions:
+ @SomeAnnotation({a, b})
(no space is used)String[][] x = {{"foo"}};
(no space is required
+ between {{
, by item 8 below)<T extends Foo & Bar>
catch (FooException | BarException e)
:
) in an enhanced
+ for
("foreach") statement,:;
or the closing parenthesis
+ ()
) of a cast//
) that
+ begins an end-of-line comment. Here, multiple spaces are allowed, but not required.List<String> list
new int[] {5, 6}
and
+ new int[] { 5, 6 }
are both validNote: this rule never requires or forbids additional space at the start or end of a +line, only interior space.
+Terminology Note: Horizontal alignment is the practice of adding a variable number of +additional spaces in your code with the goal of making certain tokens appear directly below certain +other tokens on previous lines.
This practice is permitted, but is never required by Google Style. It is not +even required to maintain horizontal alignment in places where it was already used.
Here is an example without alignment, then using alignment:
+private int x; // this is fine +private Color color; // this too + +private int x; // permitted, but future edits +private Color color; // may leave it unaligned +
Tip: Alignment can aid readability, but it creates problems for future maintenance. +Consider a future change that needs to touch just one line. This change may leave the +formerly-pleasing formatting mangled, and that is allowed. More often it prompts +the coder (perhaps you) to adjust whitespace on nearby lines as well, possibly triggering a +cascading series of reformattings. That one-line change now has a "blast radius." This can at worst +result in pointless busywork, but at best it still corrupts version history information, slows down +reviewers and exacerbates merge conflicts.
+Optional grouping parentheses are omitted only when author and reviewer agree that there is no +reasonable chance the code will be misinterpreted without them, nor would they have made the code +easier to read. It is not reasonable to assume that every reader has the entire Java +operator precedence table memorized.
+After each comma that follows an enum constant, a line-break is optional.
An enum class with no methods and no documentation on its constants may optionally be formatted +as if it were an array initializer:
+private enum Suit { CLUBS, HEARTS, SPADES, DIAMONDS } +
Since enum classes are classes, all other rules for formatting classes apply.
+Combined declarations such as int a, b;
are
+not used.
Local variables are not habitually declared at the start of their containing +block or block-like construct. Instead, local variables are declared close to the point they are +first used (within reason), to minimize their scope. Local variable declarations typically have +initializers, or are initialized immediately after declaration.
+Any array initializer may optionally be formatted as if it were a "block-like +construct." For example, the following are all valid (not an exhaustive +list):
+new int[] { new int[] { + 0, 1, 2, 3 0, +} 1, + 2, +new int[] { 3, + 0, 1, } + 2, 3 +} new int[] + {0, 1, 2, 3} ++
The square brackets form a part of the type, not the variable:
+String[] args
, not
+String args[]
.
Terminology Note: Inside the braces of a switch block are one or more
+statement groups. Each statement group consists of one or more switch labels
+(either case FOO:
or
+default:
), followed by one or more statements.
As with any other block, the contents of a switch block are indented +2.
After a switch label, a newline appears, and the indentation level is increased +2, exactly as +if a block were being opened. The following switch label returns to the previous indentation +level, as if a block had been closed.
+Within a switch block, each statement group either terminates abruptly (with a
+break
,
+continue
,
+return
or thrown exception), or is marked with a comment
+to indicate that execution will or might continue into the next statement group. Any
+comment that communicates the idea of fall-through is sufficient (typically
+// fall through
). This special comment is not required in
+the last statement group of the switch block. Example:
+switch (input) { + case 1: + case 2: + prepareOneOrTwo(); + // fall through + case 3: + handleOneTwoOrThree(); + break; + default: + handleLargeNumber(input); +} ++
Each switch statement includes a default
statement
+group, even if it contains no code.
Annotations applying to a class, method or constructor appear immediately after the +documentation block, and each annotation is listed on a line of its own (that is, one annotation +per line). These line breaks do not constitute line-wrapping (Section +4.5, Line-wrapping), so the indentation level is not +increased. Example:
+@Override +@Nullable +public String getNameIfPresent() { ... } +
Exception: a single parameterless annotation may instead appear +together with the first line of the signature, for example:
+@Override public int hashCode() { ... } +
Annotations applying to a field also appear immediately after the documentation block, but in +this case, multiple annotations (possibly parameterized) may be listed on the same line; +for example:
+@Partial @Mock DataLoader loader; +
There are no specific rules for formatting parameter and local variable annotations.
+Block comments are indented at the same level as the surrounding code. They may be in
+/* ... */
style or
+// ...
style. For multi-line
+/* ... */
comments, subsequent lines must start with
+*
aligned with the *
on the previous line.
+/* + * This is // And so /* Or you can + * okay. // is this. * even do this. */ + */ +
Comments are not enclosed in boxes drawn with asterisks or other characters.
Tip: When writing multi-line comments, use the
+/* ... */
style if you want automatic code formatters to
+re-wrap the lines when necessary (paragraph-style). Most formatters don't re-wrap lines in
+// ...
style comment blocks.
Class and member modifiers, when present, appear in the order +recommended by the Java Language Specification: +
+public protected private abstract static final transient volatile synchronized native strictfp ++
Identifiers use only ASCII letters and digits, and in two cases noted below, underscores. Thus
+each valid identifier name is matched by the regular expression \w+
.
In Google Style special prefixes or
+suffixes, like those seen in the examples name_
,
+mName
,
+s_name
and
+kName
, are not used.
Package names are all lowercase, with consecutive words simply concatenated together (no +underscores).
+Class names are written in UpperCamelCase.
Class names are typically nouns or noun phrases. Interface names may sometimes be adjectives or +adjective phrases instead. There are no specific rules or even well-established conventions for +naming annotation types.
Test classes are named starting with the name of the class they are testing, and ending
+with Test
. For example,
+HashTest
or
+HashIntegrationTest
.
Method names are written in lowerCamelCase.
Method names are typically verbs or verb phrases.
Underscores may appear in JUnit test method names to separate logical components of the
+name. One typical pattern is test<MethodUnderTest>_<state>
,
+for example testPop_emptyStack
. There is no One Correct
+Way to name test methods.
Constant names use CONSTANT_CASE
: all uppercase
+letters, with words separated by underscores. But what is a constant, exactly?
Every constant is a static final field, but not all static final fields are constants. Before +choosing constant case, consider whether the field really feels like a constant. For +example, if any of that instance's observable state can change, it is almost certainly not a +constant. Merely intending to never mutate the object is generally not +enough. Examples:
+// Constants +static final int NUMBER = 5; +static final ImmutableList<String> NAMES = ImmutableList.of("Ed", "Ann"); +static final Joiner COMMA_JOINER = Joiner.on(','); // because Joiner is immutable +static final SomeMutableType[] EMPTY_ARRAY = {}; +enum SomeEnum { ENUM_CONSTANT } + +// Not constants +static String nonFinal = "non-final"; +final String nonStatic = "non-static"; +static final Set<String> mutableCollection = new HashSet<String>(); +static final ImmutableSet<SomeMutableType> mutableElements = ImmutableSet.of(mutable); +static final Logger logger = Logger.getLogger(MyClass.getName()); +static final String[] nonEmptyArray = {"these", "can", "change"}; +
These names are typically nouns or noun phrases.
+Non-constant field names (static or otherwise) are written +in lowerCamelCase.
These names are typically nouns or noun phrases.
+Parameter names are written in lowerCamelCase.
One-character parameter names should be avoided.
+Local variable names are written in lowerCamelCase, and can be +abbreviated more liberally than other types of names.
However, one-character names should be avoided, except for temporary and looping variables.
Even when final and immutable, local variables are not considered to be constants, and should not +be styled as constants.
+Each type variable is named in one of two styles:
E
, T
,
+ X
, T2
)
+ T
(examples:
+ RequestT
,
+ FooBarT
).Sometimes there is more than one reasonable way to convert an English phrase into camel case, +such as when acronyms or unusual constructs like "IPv6" or "iOS" are present. To improve +predictability, Google Style specifies the following (nearly) deterministic scheme.
Beginning with the prose form of the name:
Note that the casing of the original words is almost entirely disregarded. Examples:
Prose form | Correct | Incorrect |
---|---|---|
"XML HTTP request" | XmlHttpRequest | XMLHTTPRequest |
"new customer ID" | newCustomerId | newCustomerID |
"inner stopwatch" | innerStopwatch | innerStopWatch |
"supports IPv6 on iOS?" | supportsIpv6OnIos | supportsIPv6OnIOS |
"YouTube importer" | YouTubeImporter YoutubeImporter * |
*Acceptable, but not recommended.
Note: Some words are ambiguously hyphenated in the English language: for example
+"nonempty" and "non-empty" are both correct, so the method names
+checkNonempty
and
+checkNonEmpty
are likewise both correct.
The @Override
annotation is used in any context in
+which it is legal.
Except as noted below, it is very rarely correct to do nothing in response to a caught
+exception. (Typical responses are to log it, or if it is considered "impossible", rethrow it as an
+AssertionError
.)
When it truly is appropriate to take no action whatsoever in a catch block, the reason this is +justified is explained in a comment.
+try { + int i = Integer.parseInt(response); + return handleNumericResponse(i); +} catch (NumberFormatException ok) { + // it's not numeric; that's fine, just continue +} +return handleTextResponse(response); +
Exception: in tests, a caught exception may be ignored without comment if it is
+named expected
. The following is a very common idiom
+for ensuring that the method under test does throw an exception of the expected type, so
+a comment is unnecessary here.
+try { + emptyStack.pop(); + fail(); +} catch (NoSuchElementException expected) { +} ++
When a reference to a static class member must be qualified, it is qualified with that class's +name, not with a reference or expression of that class's type.
+Foo aFoo = ...; +Foo.aStaticMethod(); // good +aFoo.aStaticMethod(); // bad +somethingThatYieldsAFoo().aStaticMethod(); // very bad ++
It is extremely rare to override Object.finalize
.
Tip: Don't do it. If you absolutely must, first read and understand +Effective Java +Item 7, "Avoid Finalizers," very carefully, and then don't do it.
+The basic formatting of Javadoc blocks is as seen in this general example:
+/** + * Multiple lines of Javadoc text are written here, + * wrapped normally... + */ +public int method(String p1) { ... } +
... or in this single-line example:
+/** An especially short bit of Javadoc. */ +
The general form is always acceptable. The single-line form may be substituted when there are no +at-clauses present, and the entirety of the Javadoc block (including comment markers) can fit on a +single line.
+One blank line—that is, a line containing only the aligned leading asterisk
+(*
)—appears between paragraphs, and before the group of "at-clauses" if
+present. Each paragraph but the first has <p>
immediately before the first word,
+with no space after.
Any of the standard "at-clauses" that are used appear in the order @param
,
+@return
, @throws
, @deprecated
, and these four types never
+appear with an empty description. When an at-clause doesn't fit on a single line, continuation lines
+are indented four (or more) spaces from the position of the @
.
+
The Javadoc for each class and member begins with a brief summary fragment. This +fragment is very important: it is the only part of the text that appears in certain contexts such as +class and method indexes.
This is a fragment—a noun phrase or verb phrase, not a complete sentence. It does
+not begin with A {@code Foo} is a...
, or
+This method returns...
, nor does it form a complete imperative sentence
+like Save the record.
. However, the fragment is capitalized and
+punctuated as if it were a complete sentence.
Tip: A common mistake is to write simple Javadoc in the form
+/** @return the customer ID */
. This is
+incorrect, and should be changed to
+/** Returns the customer ID. */
.
At the minimum, Javadoc is present for every
+public
class, and every
+public
or
+protected
member of such a class, with a few exceptions
+noted below.
Javadoc is optional for "simple, obvious" methods like
+getFoo
, in cases where there really and truly is
+nothing else worthwhile to say but "Returns the foo".
The test methods of a unit test class are perhaps the most common example of this exemption. +These methods can usually be named descriptively enough that no additional documentation is +needed.
Tip: Important: it is not appropriate to cite this exception to justify
+omitting relevant information that a typical reader might need to know. For example, for a method
+named getCanonicalName
, don't omit its documentation
+(with the rationale that it would say only /** Returns
+the canonical name. */
) if a typical reader may have no idea what the term "canonical name"
+means!
Javadoc is not always present on a method that overrides a supertype method. +
+Classes and members that are not visible outside their package still have Javadoc as +needed. Whenever an implementation comment would be used to define the overall purpose or +behavior of a class, method or field, that comment is written as Javadoc instead. (It's more +uniform, and more tool-friendly.)