原本打算寫glide的一些圖片使用方法,但在工作上卻遇到了大坑。
我本以為我看別人的代碼,總能學習到一些新的東西,但這次遇到的代碼卻只有無盡的坑。。
第一坑:變量名不一
? ? ? ?在XML文件中的名字各式各樣,使用一般的默認id如textview1 2 3,imageview1 2 3就算了,還有像上圖的xxx01的id。
更奇葩的是在代碼中卻取了xxx00的名字,這是啥邏輯?

第二坑:BaseActivity類混亂
? ? ? ?在項目中,我們通常會重寫一個BaseActivity類,用于管理大部分的Activity,這樣方便寫一些常用的方法,或者控制頁面進出的動畫。
? ? ? ?一般會用到Activity、FragmentActivity、Fragment這三個常用的基類,而我的項目中使用了仿蘋果的從左往右側滑關閉頁面Swipe,導致這位同事把NoSwipe全部重寫了一遍,還多了Base2Activity和BaseNormlActivity,讓我在修改進出動畫和一些finish事件時異常艱難。
? ? ? ? 更慘的是他還使用了SystemStatusManager去管理沉浸式狀態(tài)欄,導致在各個安卓版的頁面樣式都不一樣,如果要修改的一百多個頁面都可能受影響,我想了想,還是算了吧

第三坑:工具類名混亂
? ? ? ? 各種類都是以123結尾,并且里面也沒有備注用于什么地方。
? ? ? ? 我仔細看了一下MyDialog、MyDialog2、MyDialog3中有啥區(qū)別,發(fā)現根本沒區(qū)別!那你寫這個的意義是什么!

第四坑:亂定義靜態(tài)變量
? ? ? ? 看到這個我人是崩潰的。

? ? ? ? 這都寫的啥啊,為何要定義一個靜態(tài)的成員變量List,然后去重寫getset方法?為何set后面跟了兩個大寫的DD?為何都是靜態(tài)的了還要再寫get方法?
? ? ? ? 仔細閱讀后發(fā)現不止寫了這一個,一共寫了4個這樣的List,其目的為了在一個圖片選擇器PickImageAdapter中去先去get他們判斷是否為空,然后去修改List里的圖片,在設置到各自的頁面里。
? ? ? ? 而這個PickImageAdapter本身就是一個封裝好的,通用的圖片選擇器,類里本身就有自己的List去控制圖片的展示,無奈之下我只好重寫了4個頁面。。
第五坑:代碼排版亂縮進
? ? ? ? 從圖上就能看到,各種奇怪的縮進,這還只是一部分,有接近100多個頁面和xml文件,幾乎全是這樣的亂七八糟縮進。
? ? ? ? 大哥你這樣看不別扭嗎?不難受嗎?不奇怪嗎?按一下自動整理代碼的縮進快捷鍵不會嗎?快捷鍵不知道不會百度嗎?不會百度你還寫啥代碼啊

第六坑:方法名混亂
? ? ? ? 依然是在上傳圖片的ImageLoader方法中,胡亂添加了一堆123、134、14的方法。怎么有的23都沒了,這命名方法我也真醉了
? ? ? ? 一怒之下我也是全刪了,只剩2個,一個顯示默認頭像,一個顯示默認圖片。
第七坑:無腦復制代碼
? ? ? ? 在管理接口的Api類中,每個接口我都能看到這2行。這明顯是復制上一方法留下無用代碼,卻也毫無保留的每個接口都留下來了,寫代碼敢不敢?guī)c腦子?。??我就不說在其他頁面動不動就一兩百行的注釋掉的無用代碼了,真是讓人崩潰。
第八坑:嚴重的多層嵌套代碼
? ? ? ? ?看到沒,在diglog的點擊事件,已經整整嵌套了9層,9層啊!
? ? ? ? 難道把這個調用接口的Api方法寫出來很難嗎?
第十坑:命名混亂
? ? ?tv_pinpai 恩恩 是“品牌”;tv_xinghao 恩恩 是“型號”;tv_choosetype 恩?恩?恩? 是“選擇類型”?;tv_remind 恩?是 “提示”?
? ? ?不會英語不要緊,用有道翻譯啊,嫌麻煩那就都用拼音來命名啊,不要動不動來2個英語2個拼音,甚至來個中英混搭的getshangpinList,半洋半土還不帶駝峰的命名,這壓根就是隨意而來的吧?

? ? ? ? 以上這些坑,大大小小全都分布在我同事給我的代碼之中,并且基本沒有注解,一些接口的字段參數,我都要去問一遍后臺或者ios的人才能知道是干什么的。
? ? ? ? 每天看著這*一樣的代碼,人都要變得暴躁起來,因此讓我在自己的項目中加入了大量的注解,好讓后面接手的人不那么痛苦,即使不會排版,也要經常按按自動縮進的快捷鍵。
? ? ? ? 然而這個項目的bug還沒改完,我還要繼續(xù)看下去,希望下一個接手別人項目不要再有這么多坑了,阿門
