1.命名规则
- 所有变量，方法名和类名都应该遵循CamelCase风格。唯一不同的是，类名的首字母要大写。
class UserService {
private String firstName;
public String getFirstName() {
return firstName;
}
}
- 常量全部使用大写字母并且以下划线分割
final int MAX_AGE = 20;
- 变量名或者方法名不应包含太多单词。越简单越好，同时要能表达出意图。
//This is a BAD practice
public void saveUserIntoMongoDBDatabase(User user);
//This is much better
public void saveUser(User user);
2.写你所想
定义一个变量时不要偷懒，给它一个全名，而不是名字里的一个字母。变量名应该能表达出他的值。即便你知道这个变量是干嘛的，并不意味其他开发者也知道。同时当你过一阵时间再来看这段代码时，也可能记不起这个变量的含义。
让我们看看下面的一段代码：
public boolean check() {
if (a < max) {
return true;
} else {
return false;
}
}
这段代码非常不清晰，其他开发当然也不明所以。我们重构一下，看看怎么更好地写这段代码。
public boolean isAgeAcceptable() {
if (userAge < maxAge) {
return true;
} else {
return false;
}
}
3. 变量/方法声明
类的变量应该在其顶部声明。如此一来，你总能知道去哪里看变量的声明而不是滚动整个文件去寻找。
如果一个变量只在一个方法中使用，那么将其声明为一个局部变量即可。
方法声明的顺序应该和它们使用的顺序一致(从上而下)。举个栗子，如果一个方法调用当前的方法，那么这个方法应该声明当前方法的下面。
同时，越重要的方法就应该声明在前面。
4. 单一职责
一个方法应该只负责一件事情。如果你的方法同时做啦两件以上不同的事情，你应该这个方法的一些功能移到其他方法。
这样的方法才容易理解，扩展和被其他方法复用。
举个栗子，你有一个方法saveUserAndChargeCard()，将用户保存到数据库并且刷用户的信用卡。这个方法不好，因为他名字包含And而且干啦两件事。在方法命中避免使用and和or等等。你可以将这个方法分成saveUser()和chargeCard().
5.短方法
对开发这而言，短方法的长度并没有一个标准的准则。可能有人说最多10行，有些人说最多五行，有些情况下20行也是可以的。所以究竟多少才合适呢？事实上，并木有一个标准答案。尽量使你的方法越短越好。通常5到10行为最佳。当然你的方法有12行或则15行也不会有人打你的，但是要保证你这15行代码的可理解性和高质量。
6.最小化你的代码
能用一行搞定的就别用三行。这意味着你写代码时考虑下最佳的方式。
看看下面的例子：
boolean hasCreditCard(User user) {
if (user.hasCard) {
return true;
} else {
return false;
}
}
乍看之下，好像并无不妥，但是这段其实可以简化一下。
boolean hasCreditCard(User user) {
return user.hasCard;
}
看到没，一行抵过五行，所以下笔前三思囖。
7.重复是原罪
在你的工程里避免代码冗余。如果你的方法不能在其他地方复用，那么想个更好的方式去实现它。你的方法应该尽可能地通用。这也将有助于你的小伙伴直接使用你的方法而不是再去写一个。
8. 注释
注释在以前一段时间内都是开发者的日常操作，现在有些不同啦，如果你给你的代码加啦一段注释，那么说明你的代码不能自解释，你应该选择一个更好的方法来实现它。
注释存在的另一个问题是，开发在经常去改代码去忘啦改注释。
当然如果你写啦一段很复杂的算法，你的注释将帮助其他程序员理解。
9. 代码检测工具
即便你采纳啦以上的所有建议，我也推荐你使用一些工具来检查你的代码质量。这个工具可以扫描你的代码，并指出你出错的地方。因为你人工检查时可能会遗漏。
其中一个工具是SonarQube.一款代码质量分析的优秀产品。
10.最佳实践
无论你在实现什么功能，我想有人在你之前就已经做过啦。所以在你实现这个需求之前可以先调研一下。
翻译自：10 Tips To Keep Your Code Clean