开发-代码走查

近几周一直在加班,主要的工作内容就是修改已离职人员代码中的bug。

每天白天都有想骂街和砸电脑的冲动,然而看着一个个bug修复,跳过前人埋下的坑,将整体代码架构重新梳理优化,又有一种玩游戏时闯过一个个关卡的快感。

造成这种bug横飞的局面主要原因就是我在带开发小组的时候,只关注了项目的进度和软件的主要功能流程,忽略了代码具体实现细节,因为开发人员也是有多年开发经验,项目起初的时候我看他们代码写都还可以,也给他们交待了一些代码规范的要求,之后因为忙于整理需求和参与其它项目的工作,就在没看过他们的代码。

他们的工作完成后,我会看主要功能点,看功能点都实现了就没做详细检查,就将工作标记为“已完成”然后转给开发人员进行测试。

因为我带了两个开发人员都是外包公司新入职的员工,入职不久到我们公司就参与到该项目,在过了三个月试用期之后,可能他们以为公司的工作比较累,就在转正的时候提出了加薪,外包公司人事主管 没有答应,他们就没有提交转正资料,另找了其它起薪比现在高的工作。公司也没有去留他们,他们就直接走了,留下了我认为写的还不错的代码。

他们在的时候,项目测试工作还没有开始,所以问题没有暴漏。他们走后项目正好也开始了公司的内测和客户安排的第三方软件测试,bug不断的涌现了出来,因为没有多余的开发人员,我只能自己去修改bug,此时才开始去看他们的代码,不看还好,一看立马懵B了。

主要的问题有下面几个:

代码命名比较任性,有的控件名称和变量直接用aa、bb、cc没有实际含义的字符命名。

函数的名称和实际功能不符,比如有个返回值为void的InsertEntity的函数,我以为只执行了插入实体,其实里面写了好多查询逻辑将结果赋值给了公共变量。

函数从其它类模块拷贝另一个类中,只进行了逻辑的修改,函数的命名没有修改,更逗的是代码注释都没有修改,比如一个班级的类你一看里面方法都是写的和缴费类相关的方法注释。

对面向对象是用的不好,函数没有进行很好的封装,写的时候固然省事,造成的结果就是代码量过多,可读性查。

好在这周的时间将他们的代码模块整理完毕了,但是在此过程中我也看了下项目中其它同事的代码,那也是有些乱,我总结的上述问题,在他们的代码中都有出现,因为是几年的同事了,我了解到他们的水平不致于这样,我就在吃饭的时候,随口问了问,你代码中的这些问题你知道吗?他说“知道呀,现在不是忙吗?哪有时间去做的那么细”,我又找部门经理聊了聊“他说,他也很无奈,时间比较紧,就先不去计较那些细节了”。

做项目我们都知道“范围、时间、成本、质量”这写点是互相制约的,我觉得不能为了图快、图敢进度而去忽略了必要的质量把控过程,比如代码的走查,在项目上线的时候修复质量问题同样会花费时间、成本。

我认为项目开发阶段就忽略质量把控环节,无异于饮鸩止渴,这次我也尝到了苦果,还是那句话”步子迈得大了容易扯着蛋“,还是一步步来,该做的环节不能省。

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

推荐阅读更多精彩内容

  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 171,462评论 25 707
  • 先说项目开发过程中团队人员的分工协作。 一 人员安排 毕业至今的大部分项目都是独立完成,虽然也有和其他同事协作的时...
    SnowflakeCloud阅读 10,744评论 3 59
  • 我们现在是什么关系呢?朋友?或者只是两个偶然相逢的个体?如果是这样,那为什么那时要遇见,做两个不曾相识的陌生人多好...
    漂泊归所阅读 309评论 0 0
  • 我知道你看不到,我也希望你别看到,那个黑黑的,帅帅的,戴网球帽的公交车司机,西安自古出帝王,英雄与美男子也同...
    叫我漫天就好阅读 187评论 0 0
  • 我已将我的心藏起 我将独自面对回忆 梦,梦中的场景 是如此的希冀 温情的岁月, 温柔的少女 温热的心 一切尽在未来...
    古风长歌阅读 212评论 4 6