国内项目三月记

LgPersonalDev.jpg

意外的加入了国内项目,一待就是三个多月。之前常驻国外项目,一下子跳转到国内项目,对比还是很明显的。由于工作环境的限制,我们只得常驻客户现场 (暂称为A厂)。工作内容是在一个遗留Android App上继续开发,时间紧任务重,整个团队一起努力,项目最后还是获得了不错的成绩。过程中对于当前的工作也有了一点点心得体会:

关于烂代码

现状

项目前后经手过两三个团队,经历过两三年。看到代码的一刻,整个人都惊呆了,深度超过10层的逻辑嵌套,超过 1500 行的 Activity 文件,重复代码片段可以重复到10多处,莫名其妙的非空判断,随处可见的私吞Exception,哭笑不得的代码注释,从来不敢删除的注释代码(约30%)......

困境

对于烂代码,我们想到的第一个词是“重构”,但是“重构”的前提是测试。没有测试保障的重构是一件相当危险的事情。但是遗留混乱代码中怎么可能有测试。所以这是个死结。

当然你也许会说先加上测试,这就涉及到一个需求的问题。因为从代码中你很难推断出真实的需求。因为有些代码也许是和需求无关的,是开发人员的知识储备不够,添加的冗余代码,给冗余代码添加测试是一件很罪恶的事情。相当于在保证垃圾不被清楚掉。更关键的是,如果不知道具体的业务逻辑,那是没办法添加有效测试的。

从客户角度来说,重构烂代码不能带来立竿见影的价值,但却可能引起线上版本的各种未知问题。所以从某种程度上来看,如果大刀阔斧的重构一定得挨刀子。

解决

像这种情况,其实最好的解决方案是重写整个代码,当然前提是客户愿意为你的重写买单。这是一劳永逸的办法,但是对于“唯快不破”的国内客户而言,并且客户和我方信任没有完全建立起来的情况下,实现起来难度比较大。

所以我们只能使用一些迂回战术。比如让客户方接口人意识到原有代码质量比较烂的问题,然后对于需求明确的页面,小步的重构,并手动验证需求。当然需要做好版本控制,随时做好回退的准备。对于一些比较大的混乱的页面,可以尝试直接重写一个新的,同时保留老的页面。先直接用新页面验证,一旦出现紧急问题再切换回来。一两个迭代验证之后,再把之前的彻底删除。

所以,关键点在于把握好度,在资源允许的前提下,一步步的重写,并且尽量加上测试的保证,最后达到改善项目代码质量的目的。

原因

代码的复杂度反映了需求的复杂度。尤其是面对客户的各种需求,有些开发人员会不假思索的采取填鸭式的方式不断在原有代码上打补丁。我们需要理解需求,理顺需求,然后使用最优的方案实现,这样真的可以有效避免一些奇葩代码。同时对于不断变脏的模块或者方法,要不断的回顾,重构,优化。

烂代码的出现主要主要是软件开发人员自身的技术功底,这需要长期的培养与习惯养成。比如读一下《重构》或者《代码整洁之道》,可能会有效的改善代码质量。

关于“烂”需求

现状

在上述谈到烂代码的原因时,我们有谈到需求对代码质量的影响。需求本身并不是烂代码的罪魁祸首,但是却是烂代码的催化剂。我也遇到过项目中很多奇葩需求,有些需求人员本身提的需求不太符合现实。比如我遇到的,在一段代码里面出现了:

if(bookId == "1388") {
  doTrickyThing();
}

这就是一个典型的奇葩需求影响的结果。对于某些数据做某些特殊处理应该直接放到数据库或者后台API层面做,如果加到客户端,或者有 web/Android/iOS等多个客户端需要显示此数据,那么这种奇怪的逻辑就得在三个地方重复。

或者你觉得这种逻辑出现一两次也没啥关系,但是一旦项目里面充斥着这种逻辑,并且开发人员没有用代码很好的描述时,这种代码对于其他人而言就是魔法代码,后面接手项目的人也将很难理解其中的逻辑。

方案

这种问题的出现大多是人为因素,需求本身并没有任何问题。只是实现手段不佳,所以引起来代码上的奇怪逻辑。混乱的代码其实也是一个信号,预示着这个解决方案可能不是最佳的。

所以,“奇葩”需求产生时,尝试保持质疑精神。向客户了解如下内容:

  • 需求的业务价值;
  • 需求的实现方案;
  • 评估对比各方案的成本及优劣;
  • 结合项目实际情况向客户推荐方案;

在找技术方案时,最好是能够提前找资料做足功课,才能在面对客户质疑时更加从容。

原因

在国内外包项目中,大部分情况下客户都比较强势,直接给需求。但是至于需求的价值,需求的实现手段等,开发人员应该是可以提出自己的见解的。一旦双方能够统一战线,把产品做好作为自己的使命,那么很多需求也是可以讨论优化,然后到达一个和谐的状态。

因为我们是咨询师,我们不是代码的搬运工,必要的时候我们是可以和客户协商需求,并给客户提供各种建议的。

关于主动性

背景

两周一次的迭代,PM和BA也许需要和客户划定需求的范围,落实需求的细节,考虑项目成本以及交付风险等。当项目交付太紧时,技术负责人以及测试人员都不一定能够顾虑到项目中的所有事情。所以即使作为一个开发,也需要时刻关注手头开发工作之外的事情。

方案

其实作为开发人员在项目中承担的任务也可以很多,比如:

  • 迭代内故事卡的技术预研;
  • 知识总结及传递;
  • 关注上线流程;
  • 代码质量,单元测试,功能测试等;
  • 给故事卡估点;

不过在一个项目里面,你能否有机会承担这些工作和项目组本身也有关系。我经历的有些项目可能高级工程师比较多,这时候除非主动争取,要不然这些事情可能都有人做了。如果项目人员配置比较紧张,那你可能是被动的承担这部分工作。

不管是主动还是被动,开发人员是可以成为团队里面的多面手,协助BA(业务分析师)或者QA(质量保障工程师)开展相关工作的。

感想

每一个项目都是一段难忘的经历,有些项目现在回想起来觉得不容易。但是你也会惊讶自己在其中的成长,你会感谢当年那个努力奋斗的自己。

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

推荐阅读更多精彩内容

  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 171,370评论 25 707
  • 先说项目开发过程中团队人员的分工协作。 一 人员安排 毕业至今的大部分项目都是独立完成,虽然也有和其他同事协作的时...
    SnowflakeCloud阅读 10,732评论 3 59
  • 在这个互联网时代,微信、微博、QQ这些交友工具已经不再有私密的空间了。但是随着年龄的增长,很多心事不能再与父母说,...
    小可爱hh阅读 198评论 0 0
  • 本来老样子写操作建议,甚至连点评的个股都已经准备好。但在刚才仔细复盘几只雄安个股的时候发现一个有意思的情况,也是迟...
    3b33766cd083阅读 199评论 0 2
  • 小雪,北京的温度很配合节气,用俗不可耐的寒风凌冽来形容也没有过分夸张。终于鼓起勇气去跑步了,距离上次去健身房已经过...
    芒果悠悠阅读 328评论 2 4