寫寫這陣子接手別人代碼遇到的坑

原本打算寫glide的一些圖片使用方法,但在工作上卻遇到了大坑。

我本以為我看別人的代碼,總能學習到一些新的東西,但這次遇到的代碼卻只有無盡的坑。。


第一坑:變量名不一

變量名不一


? ? ? ?在XML文件中的名字各式各樣,使用一般的默認id如textview1 2 3,imageview1 2 3就算了,還有像上圖的xxx01的id。

更奇葩的是在代碼中卻取了xxx00的名字,這是啥邏輯?



第二坑:BaseActivity類混亂


BaseActivity類混亂

? ? ? ?在項目中,我們通常會重寫一個BaseActivity類,用于管理大部分的Activity,這樣方便寫一些常用的方法,或者控制頁面進出的動畫。

? ? ? ?一般會用到Activity、FragmentActivity、Fragment這三個常用的基類,而我的項目中使用了仿蘋果的從左往右側滑關閉頁面Swipe,導致這位同事把NoSwipe全部重寫了一遍,還多了Base2Activity和BaseNormlActivity,讓我在修改進出動畫和一些finish事件時異常艱難。

? ? ? ? 更慘的是他還使用了SystemStatusManager去管理沉浸式狀態(tài)欄,導致在各個安卓版的頁面樣式都不一樣,如果要修改的一百多個頁面都可能受影響,我想了想,還是算了吧



第三坑:工具類名混亂


重寫了各種123結尾的類

? ? ? ? 各種類都是以123結尾,并且里面也沒有備注用于什么地方。

? ? ? ? 我仔細看了一下MyDialog、MyDialog2、MyDialog3中有啥區(qū)別,發(fā)現根本沒區(qū)別!那你寫這個的意義是什么!



第四坑:亂定義靜態(tài)變量


亂定義靜態(tài)變量,并重寫了get、set方法去調用。。

? ? ? ? 看到這個我人是崩潰的。


? ? ? ? 這都寫的啥啊,為何要定義一個靜態(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ù)看下去,希望下一個接手別人項目不要再有這么多坑了,阿門


最后編輯于
?著作權歸作者所有,轉載或內容合作請聯系作者
【社區(qū)內容提示】社區(qū)部分內容疑似由AI輔助生成,瀏覽時請結合常識與多方信息審慎甄別。
平臺聲明:文章內容(如有圖片或視頻亦包括在內)由作者上傳并發(fā)布,文章內容僅代表作者本人觀點,簡書系信息發(fā)布平臺,僅提供信息存儲服務。

相關閱讀更多精彩內容

  • 一、編程規(guī)約 (一)命名規(guī)約 【強制】 代碼中的命名均不能以下劃線或美元符號開始,也不能以下劃線或美元符號結束。反...
    喝咖啡的螞蟻閱讀 1,610評論 0 2
  • 在培養(yǎng)孩子的時候,越來越多的家長開始重視情商的培養(yǎng)。而情商是可以被培養(yǎng)出來的,尤其是孩子,一個人情商的培養(yǎng),在孩童...
    Amyhe1997閱讀 160評論 0 0
  • 落 葉 文/中流擊水 空中起舞 地上打轉 風兒說 你是秋天的一張張名片 原野 山戀 路邊 你成了蟲草的新伙伴 也曾...
    楚山漢水閱讀 910評論 17 26
  • 也許每個人的心里,都有一個人,不管曾經有多么熟悉,最后都不得不散落天涯,各自安好。 只是路過他在的地方,會不由自主...
    仙子下凡塵閱讀 156評論 0 0

友情鏈接更多精彩內容