写写这阵子接手别人代码遇到的坑

原本打算写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去管理沉浸式状态栏,导致在各个安卓版的页面样式都不一样,如果要修改的一百多个页面都可能受影响,我想了想,还是算了吧



第三坑:工具类名混乱


重写了各种123结尾的类

        各种类都是以123结尾,并且里面也没有备注用于什么地方。

        我仔细看了一下MyDialog、MyDialog2、MyDialog3中有啥区别,发现根本没区别!那你写这个的意义是什么!



第四坑:乱定义静态变量


乱定义静态变量,并重写了get、set方法去调用。。

        看到这个我人是崩溃的。


        这都写的啥啊,为何要定义一个静态的成员变量List,然后去重写getset方法?为何set后面跟了两个大写的DD?为何都是静态的了还要再写get方法?

        仔细阅读后发现不止写了这一个,一共写了4个这样的List,其目的为了在一个图片选择器PickImageAdapter中去先去get他们判断是否为空,然后去修改List里的图片,在设置到各自的页面里。

        而这个PickImageAdapter本身就是一个封装好的,通用的图片选择器,类里本身就有自己的List去控制图片的展示,无奈之下我只好重写了4个页面。。


第五坑:代码排版乱缩进



        从图上就能看到,各种奇怪的缩进,这还只是一部分,有接近100多个页面和xml文件,几乎全是这样的乱七八糟缩进。

        大哥你这样看不别扭吗?不难受吗?不奇怪吗?按一下自动整理代码的缩进快捷键不会吗?快捷键不知道不会百度吗?不会百度你还写啥代码啊



第六坑:方法名混乱


        依然是在上传图片的ImageLoader方法中,胡乱添加了一堆123、134、14的方法。怎么有的23都没了,这命名方法我也真醉了

        一怒之下我也是全删了,只剩2个,一个显示默认头像,一个显示默认图片。


第七坑:无脑复制代码


        在管理接口的Api类中,每个接口我都能看到这2行。这明显是复制上一方法留下无用代码,却也毫无保留的每个接口都留下来了,写代码敢不敢带点脑子啊??我就不说在其他页面动不动就一两百行的注释掉的无用代码了,真是让人崩溃。



第八坑:严重的多层嵌套代码


         看到没,在diglog的点击事件,已经整整嵌套了9层,9层啊!

        难道把这个调用接口的Api方法写出来很难吗?


第十坑:命名混乱


     tv_pinpai 恩恩 是“品牌”;tv_xinghao 恩恩 是“型号”;tv_choosetype 恩?恩?恩? 是“选择类型”?;tv_remind 恩?是 “提示”?

     不会英语不要紧,用有道翻译啊,嫌麻烦那就都用拼音来命名啊,不要动不动来2个英语2个拼音,甚至来个中英混搭的getshangpinList,半洋半土还不带驼峰的命名,这压根就是随意而来的吧?



        以上这些坑,大大小小全都分布在我同事给我的代码之中,并且基本没有注解,一些接口的字段参数,我都要去问一遍后台或者ios的人才能知道是干什么的。

        每天看着这*一样的代码,人都要变得暴躁起来,因此让我在自己的项目中加入了大量的注解,好让后面接手的人不那么痛苦,即使不会排版,也要经常按按自动缩进的快捷键。

        然而这个项目的bug还没改完,我还要继续看下去,希望下一个接手别人项目不要再有这么多坑了,阿门


最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 215,634评论 6 497
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 91,951评论 3 391
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 161,427评论 0 351
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 57,770评论 1 290
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 66,835评论 6 388
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 50,799评论 1 294
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,768评论 3 416
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 38,544评论 0 271
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,979评论 1 308
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 37,271评论 2 331
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 39,427评论 1 345
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 35,121评论 5 340
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,756评论 3 324
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 31,375评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,579评论 1 268
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 47,410评论 2 368
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 44,315评论 2 352

推荐阅读更多精彩内容

  • 一、编程规约 (一)命名规约 【强制】 代码中的命名均不能以下划线或美元符号开始,也不能以下划线或美元符号结束。反...
    喝咖啡的蚂蚁阅读 1,502评论 0 2
  • 在培养孩子的时候,越来越多的家长开始重视情商的培养。而情商是可以被培养出来的,尤其是孩子,一个人情商的培养,在孩童...
    Amyhe1997阅读 134评论 0 0
  • 落 叶 文/中流击水 空中起舞 地上打转 风儿说 你是秋天的一张张名片 原野 山恋 路边 你成了虫草的新伙伴 也曾...
    楚山汉水阅读 823评论 17 26
  • 也许每个人的心里,都有一个人,不管曾经有多么熟悉,最后都不得不散落天涯,各自安好。 只是路过他在的地方,会不由自主...
    仙子下凡尘阅读 131评论 0 0