1、不必要的重復代碼

      很多初學者喜歡復制粘貼，是個非常不好的習慣，最后導致系統充斥著大量重復的代碼，對后期的維護相當不便。這個是初學者最易犯的低級錯誤。

2、邏輯混亂、思路不清晰

     我曾見過這樣的代碼

大家注意這句

當然這個不能算錯誤，但是給人的感覺很不好。

還有三四層的if  else 、循環里面套三四層的循環，絕對看到你頭暈腦脹。

例如以下代碼：

既然已經拋出異常了，咋還寫這么多if else，還有這么多 1 2 3誰又知道是什么意思，若是以后1 2 3 含義有變化那就要到處修改了。 修改如下

將狀態以靜態常量的方式寫到VO類里面，這樣以后維護起來也方便一些。

3、類的職責過多

      這個也是初學者經常犯的錯誤，違反單一職責原則，將本來不屬于A類的功能寫到A類里來。造成耦合過高。

能力有限暫時寫這么多，拋磚引玉。歡迎大家補充集思廣益，后續我將大家的意見整理出來，持續更新。