Apache Log4j Code Style GuidelinesIntroductionThis document serves as the complete definition of the Log4j project's coding standards for source code in the Java™ Programming Language. It originated from the Google coding standards but incorporates modifications that reflect the desires of the Log4j community. 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). Terminology notesIn this document, unless otherwise clarified:
Other "terminology notes" will appear occasionally throughout the document. Guide notesExample code in this document is non-normative. That is, while the examples are in Log4j 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. File nameThe source file name consists of the case-sensitive name of the top-level class it contains, plus the .java extension. Special charactersWhitespace charactersAside 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:
Special escape sequencesFor 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. Non-ASCII charactersFor 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:
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. Source file structureA source file consists of, in order:
Exactly one blank line separates each section that is present. Apache LicenseThe Apache license belongs here. No other license should appear. Other licenses that apply should be referenced in a NOTICE file Package statementThe package statement is not line-wrapped. The column limit (Column limit: 120) does not apply to package statements. Import statementsStatic wildcard imports in the test treeWildcard static imports are encouraged for test imports like JUnit, EasyMock, and Hamcrest. No line-wrappingImport statements are not line-wrapped. The column limit (Column limit: 120) does not apply to import statements. Ordering and spacingImport statements are divided into the following groups, in this order, with each group separated by a single blank line:
Within 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.) IDE settings for ordering imports automatically can be found in the source distributions under src/ide. For example:
Class declarationClass member orderingClass members should be grouped in the following order>.
FormattingTerminology Note: block-like construct refers to the body of a class, method or constructor. Note that, by array initializers, any array initializer may optionally be treated as if it were a block-like construct. BracesBraces are used where optionalBraces are used with if, else, for, do and while statements, even when the body is empty or contains only a single statement. Nonempty blocks: K & R styleBraces follow the Kernighan and Ritchie style ("Egyptian brackets") for nonempty blocks and block-like constructs:
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. Empty blocks: may be conciseAn 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() {} Block indentation: +4 spacesEach time a new block or block-like construct is opened, the indent increases by four 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.) Column limit: 120The column limit for Log4j is 120 characters. Except as noted below, any line that would exceed this limit must be line-wrapped, as explained in Line-wrapping. Exceptions:
Line-wrappingTerminology 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. Where to breakThe prime directive of line-wrapping is: prefer to break at a higher syntactic level. Also:
Indent continuation lines at least +8 spacesWhen line-wrapping, each line after the first (each continuation line) is indented at least +8 from the original line. When there are multiple continuation lines, indentation may be varied beyond +8 as desired. In general, two continuation lines use the same indentation level if and only if they begin with syntactically parallel elements. The section on Horizontal alignment addresses the discouraged practice of using a variable number of spaces to align certain tokens with previous lines. WhitespaceVertical WhitespaceA single blank line appears:
Multiple consecutive blank lines are permitted, but never required (or encouraged). Horizontal whitespaceBeyond 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.
Note: This rule never requires or forbids additional space at the start or end of a line, only interior space. Horizontal alignment: never requiredTerminology 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. Grouping parentheses: recommendedOptional 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. Specific constructsEnum classesAfter 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 (see array initializers). private enum Suit { CLUBS, HEARTS, SPADES, DIAMONDS } Since enum classes are classes, all other rules for formatting classes apply. Variable declarationsOne variable per declarationEvery variable declaration (field or local) declares only one variable: declarations such as int a, b; are not used. Declared when needed, initialized as soon as possibleLocal 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. ArraysSwitch statementsTerminology 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. IndentationAs 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. Fall-through: commentedWithin 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); } AnnotationsAnnotations 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. CommentsBlock comment styleBlock 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. Rules common to all identifiersIdentifiers 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. Rules by identifier typePackage namesPackage names are all lowercase, with consecutive words simply concatenated together (no underscores). For example, com.example.deepspace, not com.example.deepSpace or com.example.deep_space. Class namesClass names are written in UpperCamelCase. Class names are typically nouns or noun phrases. For example, Character or ImmutableList. Interface names may also be nouns or noun phrases (for example, List), but may sometimes be adjectives or adjective phrases instead (for example, Readable). 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 namesMethod names are written in lowerCamelCase. Method names are typically verbs or verb phrases. For example, sendMessage or stop. 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 namesConstant 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 namesNon-constant field names (static or otherwise) are written in lowerCamelCase. These names are typically nouns or noun phrases. For example, computedValues or index. Parameter namesParameter names are written in lowerCamelCase. One-character parameter names should be avoided. Local variable namesLocal 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. Type variable namesEach type variable is named in one of two styles:
Camel case: definedSometimes 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:
*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. @Override: always usedA method is marked with the @Override annotation whenever it is legal. This includes a class method overriding a superclass method, a class method implementing an interface method, and an interface method respecifying a superinterface method. Exception:@Override may be omitted when the parent method is @Deprecated. Caught exceptions: not ignoredExcept 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) { } Static members: qualified using classWhen 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 Finalizers: not usedIt 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. FormattingGeneral formThe basic formatting of Javadoc blocks is as seen in this 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 basic 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. ParagraphsOne 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. At-clausesAny 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 summary fragmentThe 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. */. Where Javadoc is usedAt 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. Other classes and members 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.) Exception: self-explanatory methodsJavadoc 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". 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! |