原本打算寫glide的一些圖片使用方法块蚌,但在工作上卻遇到了大坑机蔗。
我本以為我看別人的代碼蒲祈,總能學習到一些新的東西,但這次遇到的代碼卻只有無盡的坑萝嘁。梆掸。
第一坑:變量名不一
? ? ? ?在XML文件中的名字各式各樣,使用一般的默認id如textview1 2 3,imageview1 2 3就算了牙言,還有像上圖的xxx01的id酸钦。
更奇葩的是在代碼中卻取了xxx00的名字,這是啥邏輯咱枉?
第二坑:BaseActivity類混亂
? ? ? ?在項目中卑硫,我們通常會重寫一個BaseActivity類,用于管理大部分的Activity蚕断,這樣方便寫一些常用的方法欢伏,或者控制頁面進出的動畫。
? ? ? ?一般會用到Activity亿乳、FragmentActivity硝拧、Fragment這三個常用的基類,而我的項目中使用了仿蘋果的從左往右側(cè)滑關(guān)閉頁面Swipe葛假,導致這位同事把NoSwipe全部重寫了一遍障陶,還多了Base2Activity和BaseNormlActivity,讓我在修改進出動畫和一些finish事件時異常艱難聊训。
? ? ? ? 更慘的是他還使用了SystemStatusManager去管理沉浸式狀態(tài)欄抱究,導致在各個安卓版的頁面樣式都不一樣,如果要修改的一百多個頁面都可能受影響带斑,我想了想媳维,還是算了吧
第三坑:工具類名混亂
? ? ? ? 各種類都是以123結(jié)尾酿雪,并且里面也沒有備注用于什么地方遏暴。
? ? ? ? 我仔細看了一下MyDialog侄刽、MyDialog2、MyDialog3中有啥區(qū)別朋凉,發(fā)現(xiàn)根本沒區(qū)別州丹!那你寫這個的意義是什么!
第四坑:亂定義靜態(tài)變量
? ? ? ? 看到這個我人是崩潰的所计。
? ? ? ? 這都寫的啥啊,為何要定義一個靜態(tài)的成員變量List团秽,然后去重寫getset方法主胧?為何set后面跟了兩個大寫的DD?為何都是靜態(tài)的了還要再寫get方法习勤?
? ? ? ? 仔細閱讀后發(fā)現(xiàn)不止寫了這一個踪栋,一共寫了4個這樣的List,其目的為了在一個圖片選擇器PickImageAdapter中去先去get他們判斷是否為空图毕,然后去修改List里的圖片夷都,在設(shè)置到各自的頁面里。
? ? ? ? 而這個PickImageAdapter本身就是一個封裝好的予颤,通用的圖片選擇器囤官,類里本身就有自己的List去控制圖片的展示,無奈之下我只好重寫了4個頁面蛤虐。党饮。
第五坑:代碼排版亂縮進
? ? ? ? 從圖上就能看到,各種奇怪的縮進笆焰,這還只是一部分劫谅,有接近100多個頁面和xml文件,幾乎全是這樣的亂七八糟縮進嚷掠。
? ? ? ? 大哥你這樣看不別扭嗎捏检?不難受嗎?不奇怪嗎不皆?按一下自動整理代碼的縮進快捷鍵不會嗎贯城?快捷鍵不知道不會百度嗎?不會百度你還寫啥代碼啊
第六坑:方法名混亂
? ? ? ? 依然是在上傳圖片的ImageLoader方法中霹娄,胡亂添加了一堆123能犯、134鲫骗、14的方法。怎么有的23都沒了踩晶,這命名方法我也真醉了
? ? ? ? 一怒之下我也是全刪了执泰,只剩2個,一個顯示默認頭像渡蜻,一個顯示默認圖片术吝。
第七坑:無腦復制代碼
? ? ? ? 在管理接口的Api類中,每個接口我都能看到這2行茸苇。這明顯是復制上一方法留下無用代碼排苍,卻也毫無保留的每個接口都留下來了,寫代碼敢不敢?guī)c腦子把堋淘衙??我就不說在其他頁面動不動就一兩百行的注釋掉的無用代碼了腻暮,真是讓人崩潰彤守。
第八坑:嚴重的多層嵌套代碼
? ? ? ? ?看到?jīng)],在diglog的點擊事件西壮,已經(jīng)整整嵌套了9層遗增,9層啊款青!
? ? ? ? 難道把這個調(diào)用接口的Api方法寫出來很難嗎做修?
第十坑:命名混亂
? ? ?tv_pinpai 恩恩 是“品牌”;tv_xinghao 恩恩 是“型號”抡草;tv_choosetype 恩饰及?恩?恩康震? 是“選擇類型”燎含?;tv_remind 恩腿短?是 “提示”屏箍?
? ? ?不會英語不要緊,用有道翻譯啊橘忱,嫌麻煩那就都用拼音來命名啊赴魁,不要動不動來2個英語2個拼音,甚至來個中英混搭的getshangpinList,半洋半土還不帶駝峰的命名钝诚,這壓根就是隨意而來的吧颖御?
? ? ? ? 以上這些坑,大大小小全都分布在我同事給我的代碼之中凝颇,并且基本沒有注解潘拱,一些接口的字段參數(shù)疹鳄,我都要去問一遍后臺或者ios的人才能知道是干什么的。
? ? ? ? 每天看著這*一樣的代碼芦岂,人都要變得暴躁起來瘪弓,因此讓我在自己的項目中加入了大量的注解,好讓后面接手的人不那么痛苦盔腔,即使不會排版杠茬,也要經(jīng)常按按自動縮進的快捷鍵。
? ? ? ? 然而這個項目的bug還沒改完弛随,我還要繼續(xù)看下去,希望下一個接手別人項目不要再有這么多坑了宁赤,阿門