代码整洁之道——初写代码者易犯的错误

1、不必要的重复代码

      很多初学者喜欢复制粘贴，是个非常不好的习惯，最后导致系统充斥着大量重复的代码，对后期的维护相当不便。这个是初学者最易犯的低级错误。

2、逻辑混乱、思路不清晰

     

     我曾见过这样的代码

   boolean flag;

  flag = str.indexOf(",")>-1;
                        if(flag==false)

大家注意这句

if(flag==false)

当然这个不能算错误，但是给人的感觉很不好。

还有三四层的if  else 、循环里面套三四层的循环，绝对看到你头晕脑胀。

例如以下代码：

if(userObj!=null){
	if(!"3".equals(userObj.getType())){
		if("1".equals(userObj.getStatus()))
			throw new DefineException("");
		if("2".equals(userObj.getStatus()))
			throw new DefineException("");
	}else{
		throw new DefineException("");
	}
}else{
  throw new DefineException("");
}

既然已经抛出异常了，咋还写这么多if else，还有这么多 1 2 3谁又知道是什么意思，若是以后1 2 3 含义有变化那就要到处修改了。 修改如下

  if(userObj==null){
        throw new DefineException("");
    }
         
    if(UserObj.TYPE_WEB.equals(userObj.getType())){
        throw new DefineException("");
    }
    if(UserObj.STATUS_LOCKED.equals(userObj.getStatus())){
        throw new DefineException("");
    }
    if(UserObj.STATUS_INVALID.equals(userObj.getStatus())){
        throw new DefineException("");
    }
    

将状态以静态常量的方式写到VO类里面，这样以后维护起来也方便一些。

3、类的职责过多

      这个也是初学者经常犯的错误，违反单一职责原则，将本来不属于A类的功能写到A类里来。造成耦合过高。

能力有限暂时写这么多，抛砖引玉。欢迎大家补充集思广益，后续我将大家的意见整理出来，持续更新。