Java中常见的违规编码

欢迎进入Java社区论坛,与200万技术人员互动交流 >>进入

三、简化if-else

我写了几个实用的方法作为参考,检查语句条件并且基于该条件返回值。比如,考虑到isEligible方法,正如你之前所看到的:

1.private boolean isEligible(int age){

2. boolean result;

3. if(age > 18){

4. result = true;

5. }else{

6. result = false;

7. }

8. return result;

9.}

整个方法以一个单一的return语句重新编写:

1.private boolean isEligible(int age){

2.3.return age > 18;

4.5.}

四、不要给Boolean, Integer或者String创建新的实例

避免给Boolean,Integer,String创建新的实例。比如,使用new Boolean(true),Boolean,valueOf(true)。修改后的语句与之前的效果基本相同,除了在性能上有所提升。

五、使用大括号模块语句

永远别忘了使用大括号模块语句比如if、for、while.这样做的好处是当你在修改模块级语句时减少了模糊代码并且避免引进bug的机会。

不建议:

1.if(age > 18)

2. result = true;

3.else

4. result = false;

建议:

1.if(age > 18){

2. result = true;

3.}else{

4. result = false;

5.}

六、以final类型标记方法参数,任何时候都适用

请记住,以final类型标记方法参数,任何时候都适用。这样做的好处在于当你不小心修改参数值时,编译器会给你警告,同时它还能以更好的方式优化编译器代码字节。

建议:

private boolean isEligible(final int age){ … }

七、在UPPERCASE中命名public static final字段

在UPPERCASE中命名public static final字段(通常也被称之为常量)。这个可以让你轻松区分常量字段和局部变量之间的不同。

不建议:

public static final String testAccountNo = “12345678”;

建议:

public static final String TEST_ACCOUNT_NO = “12345678”;,

八、组合成单一的if语句

在尽可能多的情况下,把多个if语句组合成单一的if语句,比如下面的代码:

1.if(age > 18){

2. if( voted == false){

3. // eligible to vote.

4. }

5.}

合并成单一的if语句:

1.if(age > 18 && !voted){

2. // eligible to vote

3.}

九、Switch应该有default

始终给Switch语句添加default.

十、使用常量来避免重复定义相同的字符串值

如果你在多个地方必须使用字符串,那么使用常量来避免重复定义拥有相同值的字符串。

比如,看下面的代码:

1.private void someMethod(){

2. logger.log(”My Application” + e);

3. …

4. …

5. logger.log(”My Application” + f);

6.}

string literal”我的应用”可以作为常量并且能在代码中使用。

1.public static final String MY_APP = “My Application”;

2.

3.private void someMethod(){

4. logger.log(MY_APP + e);

5. …

6. …

7. logger.log(MY_APP + f);

8.}

[1][2]

人生就是要感受美丽的、善良的,丑恶的、病态的。

Java中常见的违规编码

相关文章:

你感兴趣的文章:

标签云: