This style guide is different from other you may see, because the focus is centered on readability for print and the web. We created this style guide to keep the code in our tutorials consistent.
Our overarching goals are conciseness, readability and simplicity.
This style-guide is somewhat of a mash-up between the existing Java language style guides, and a tutorial-readability focused Swift style-guide. The language guidance is drawn from the Android contributors style guide and the Google Java Style Guide. Alterations to support additional readability in tutorials were inspired by the raywenderlich.com Swift style guide.
- Nomenclature
- Declarations
- Spacing
- Getters & Setters
- Brace Style
- Switch Statements
- Annotations
- XML Guidance
- Language
- Credit
On the whole, naming should follow Java standards.
Package names are all lower-case, multiple words concatenated together, without hypens or underscores:
BAD:
com.RayWenderlich.funky_widgetGOOD:
com.raywenderlich.funkywidgetWritten in UpperCamelCase. For example RadialSlider.
Written in lowerCamelCase. For example setValue.
Written in lowerCamelCase.
Static fields should be written in uppercase, with an underscore separating words:
public static final int THE_ANSWER = 42;As distasteful as it is, field naming should follow the Android source code naming conventions:
- Non-public, non-static field names start with an
m. - Static field names start with an
s.
For example:
public class MyClass {
public static final int SOME_CONSTANT = 42;
public int publicField;
private static MyClass sSingleton;
int mPackagePrivate;
private int mPrivate;
protected int mProtected;
}Note: You can set Android Studio to follow this convention. See this SO link for details http://stackoverflow.com/questions/22732722/intellij-android-studio-member-variable-prefix
Written in lowerCamelCase.
Single character values to be avoided except for temporary looping variables.
In code, acronyms should be treated as words. For example:
BAD:
XMLHTTPRequest
String URL
findPostByIDGOOD:
XmlHttpRequest
String url
findPostByIdAccess level modifiers should be explicitly defined for classes, methods and member variables.
Prefer single declaration per line.
BAD:
String username, twitterHandle;GOOD:
String username;
String twitterHandle;Exactly one class per source file, although inner classes are encouraged where scoping appropriate.
Enum classes should be avoided where possible, due to a large memory overhead. Static constants are preferred. See http://developer.android.com/training/articles/memory.html#Overhead for further details.
Enum classes without methods may be formatted without line-breaks, as follows:
private enum CompassDirection { EAST, NORTH, WEST, SOUTH }Spacing is especially important in raywenderlich.com code, as code needs to be easily readable as part of the tutorial. Java does not lend itself well to this.
Indentation is using spaces - never tabs.
Indentation for blocks uses 2 spaces (not the default 4):
BAD:
for (int i = 0; i < 10; i++) {
Log.i(TAG, "index=" + i);
}GOOD:
for (int i = 0; i < 10; i++) {
Log.i(TAG, "index=" + i);
}Indentation for line wraps should use 4 spaces (not the default 8):
BAD:
CoolUiWidget widget =
someIncrediblyLongExpression(that, reallyWouldNotFit, on, aSingle, line);GOOD:
CoolUiWidget widget =
someIncrediblyLongExpression(that, reallyWouldNotFit, on, aSingle, line);Lines should be no longer than 100 characters long.
There should be exactly one blank line between methods to aid in visual clarity and organization. Whitespace within methods should separate functionality, but having too many sections in a method often means you should refactor into several methods.
For external access to fields in classes, getters and setters are preferred to
direct access of the fields. Fields should rarely be public.
However, it is encouraged to use the field directly when accessing internally (i.e. from inside the class). This is a performance optimization recommended by Google: http://developer.android.com/training/articles/perf-tips.html#GettersSetters
Only trailing closing-braces are awarded their own line. All others appear the same line as preceding code:
BAD:
class MyClass
{
void doSomething()
{
if (someTest)
{
// ...
}
else
{
// ...
}
}
}GOOD:
class MyClass {
void doSomething() {
if (someTest) {
// ...
} else {
// ...
}
}
}Conditional statements are always required to be enclosed with braces, irrespective of the number of lines required.
BAD:
if (someTest)
doSomething();
if (someTest) doSomethingElse();GOOD:
if (someTest) {
doSomething();
}
if (someTest) { doSomethingElse(); }Switch statements fall-through by default, but this can be unintuitive. If you require this behavior, comment it.
Alway include the default case.
BAD:
switch (anInput) {
case 1:
doSomethingForCaseOne();
case 2:
doSomethingForCaseOneOrTwo();
break;
case 3:
doSomethingForCaseOneOrThree();
break;
}GOOD:
switch (anInput) {
case 1:
doSomethingForCaseOne();
// fall through
case 2:
doSomethingForCaseOneOrTwo();
break;
case 3:
doSomethingForCaseOneOrThree();
break;
default:
break;
}Standard annotations should be used - in particular @Override. This should
appear the line before the function declaration.
BAD:
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
}GOOD:
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
}Since Android uses XML extensively in addition to Java, we have some rules specific to XML.
View-based XML files should be prefixed with the type of view that they represent.
BAD:
login.xmlmain_screen.xmlrounded_edges_button.xml
GOOD:
activity_login.xmlfragment_main_screen.xmlbutton_rounded_edges.xml
Similarly to Java, indentation should be two characters.
Wherever possible XML resource files should be used:
- Strings =>
res/values/strings.xml - Styles =>
res/values/styles.xml - Colors =>
res/color/colors.xml - Animations =>
res/anim/ - Drawable =>
res/drawable
Where appropriate, XML attributes should appear in the following order:
idattributelayout_*attributes- style attributes such as
gravityortextColor - value attributes such as
textorsrc
Within each of these groups, the attributes should be ordered alphabetically.
Use US English spelling.
BAD:
String colour = "red";GOOD:
String color = "red";This style guide is a collaborative effort from the most stylish raywenderlich.com team members: