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

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

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


第一坑:變量名不一

變量名不一


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

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



第二坑:BaseActivity類混亂


BaseActivity類混亂

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

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

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



第三坑:工具類名混亂


重寫了各種123結(jié)尾的類

? ? ? ? 各種類都是以123結(jié)尾酿雪,并且里面也沒有備注用于什么地方遏暴。

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



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


亂定義靜態(tài)變量杂彭,并重寫了get墓毒、set方法去調(diào)用。亲怠。

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


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


最后編輯于
?著作權(quán)歸作者所有,轉(zhuǎn)載或內(nèi)容合作請聯(lián)系作者
  • 序言:七十年代末舀透,一起剝皮案震驚了整個濱河市,隨后出現(xiàn)的幾起案子决左,更是在濱河造成了極大的恐慌愕够,老刑警劉巖,帶你破解...
    沈念sama閱讀 211,743評論 6 492
  • 序言:濱河連續(xù)發(fā)生了三起死亡事件佛猛,死亡現(xiàn)場離奇詭異惑芭,居然都是意外死亡,警方通過查閱死者的電腦和手機继找,發(fā)現(xiàn)死者居然都...
    沈念sama閱讀 90,296評論 3 385
  • 文/潘曉璐 我一進店門遂跟,熙熙樓的掌柜王于貴愁眉苦臉地迎上來,“玉大人婴渡,你說我怎么就攤上這事幻锁。” “怎么了边臼?”我有些...
    開封第一講書人閱讀 157,285評論 0 348
  • 文/不壞的土叔 我叫張陵哄尔,是天一觀的道長。 經(jīng)常有香客問我柠并,道長岭接,這世上最難降的妖魔是什么? 我笑而不...
    開封第一講書人閱讀 56,485評論 1 283
  • 正文 為了忘掉前任臼予,我火速辦了婚禮鸣戴,結(jié)果婚禮上,老公的妹妹穿的比我還像新娘瘟栖。我一直安慰自己葵擎,他們只是感情好,可當我...
    茶點故事閱讀 65,581評論 6 386
  • 文/花漫 我一把揭開白布半哟。 她就那樣靜靜地躺著酬滤,像睡著了一般签餐。 火紅的嫁衣襯著肌膚如雪。 梳的紋絲不亂的頭發(fā)上盯串,一...
    開封第一講書人閱讀 49,821評論 1 290
  • 那天氯檐,我揣著相機與錄音,去河邊找鬼体捏。 笑死冠摄,一個胖子當著我的面吹牛,可吹牛的內(nèi)容都是我干的几缭。 我是一名探鬼主播河泳,決...
    沈念sama閱讀 38,960評論 3 408
  • 文/蒼蘭香墨 我猛地睜開眼,長吁一口氣:“原來是場噩夢啊……” “哼年栓!你這毒婦竟也來了拆挥?” 一聲冷哼從身側(cè)響起,我...
    開封第一講書人閱讀 37,719評論 0 266
  • 序言:老撾萬榮一對情侶失蹤某抓,失蹤者是張志新(化名)和其女友劉穎纸兔,沒想到半個月后,有當?shù)厝嗽跇淞掷锇l(fā)現(xiàn)了一具尸體否副,經(jīng)...
    沈念sama閱讀 44,186評論 1 303
  • 正文 獨居荒郊野嶺守林人離奇死亡汉矿,尸身上長有42處帶血的膿包…… 初始之章·張勛 以下內(nèi)容為張勛視角 年9月15日...
    茶點故事閱讀 36,516評論 2 327
  • 正文 我和宋清朗相戀三年,在試婚紗的時候發(fā)現(xiàn)自己被綠了备禀。 大學時的朋友給我發(fā)了我未婚夫和他白月光在一起吃飯的照片洲拇。...
    茶點故事閱讀 38,650評論 1 340
  • 序言:一個原本活蹦亂跳的男人離奇死亡,死狀恐怖痹届,靈堂內(nèi)的尸體忽然破棺而出呻待,到底是詐尸還是另有隱情,我是刑警寧澤队腐,帶...
    沈念sama閱讀 34,329評論 4 330
  • 正文 年R本政府宣布蚕捉,位于F島的核電站,受9級特大地震影響柴淘,放射性物質(zhì)發(fā)生泄漏迫淹。R本人自食惡果不足惜,卻給世界環(huán)境...
    茶點故事閱讀 39,936評論 3 313
  • 文/蒙蒙 一为严、第九天 我趴在偏房一處隱蔽的房頂上張望敛熬。 院中可真熱鬧,春花似錦第股、人聲如沸应民。這莊子的主人今日做“春日...
    開封第一講書人閱讀 30,757評論 0 21
  • 文/蒼蘭香墨 我抬頭看了看天上的太陽诲锹。三九已至繁仁,卻和暖如春,著一層夾襖步出監(jiān)牢的瞬間归园,已是汗流浹背黄虱。 一陣腳步聲響...
    開封第一講書人閱讀 31,991評論 1 266
  • 我被黑心中介騙來泰國打工, 沒想到剛下飛機就差點兒被人妖公主榨干…… 1. 我叫王不留庸诱,地道東北人捻浦。 一個月前我還...
    沈念sama閱讀 46,370評論 2 360
  • 正文 我出身青樓,卻偏偏與公主長得像桥爽,于是被迫代替她去往敵國和親朱灿。 傳聞我的和親對象是個殘疾皇子,可洞房花燭夜當晚...
    茶點故事閱讀 43,527評論 2 349

推薦閱讀更多精彩內(nèi)容

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