非常实用的代码阅读策略!

谁都喜欢可读性强的代码,希望接手的代码容易阅读,容易理解,从而减少交接的工作量,但并不是所有的代码都有好的易读性,接手前辈的“屎山”通常是一件令开发者非常痛苦的事情。
关于代码有一种流行说法:代码被阅读的次数是它被书写次数的十倍,而且产品的寿命越长,这个比例就越高。考虑到这点,我们似乎对“理解代码”的投资明显不足。开发者通常更侧重于编码的能力,而不是阅读和解释已有代码的能力,即便这种场景在日常工作中会频繁出现。
开发任务的前80-95%时间应该用来阅读代码以及文档。在研究现有代码的过程中,你可能会学到很多东西,只有读完代码之后才能说:“这个功能已经存在了,或者是加入这个功能弊大于利”。
本文将为你介绍一些实用的代码阅读策略,你可以根据实际情况使用它们。

1、重构局部变量和方法

有时候,一段代码非常模糊会误导读者甚至让人难以推理出其含义。一个几乎没有风险的方法是重新命名局部变量和私有方法,以更准确地描述它们的作用。这些类型的修改不会影响到当前工作文件之外的功能,只要注意避免命名冲突,就不会导致逻辑错误。如果可能的话,使用IDE的重构工具(而不是文本查找和替换),这样就可以一键重命名所有被使用的东西了。
例如,考虑下面这段JavaScript代码

function ib(a, fn) 
{return (a || []).reduce((o, i) => {
o[fn(i)] = i;
return o;
}, {});}

它阅读起来非常困难,方法名ib对理解函数功能毫无用处。不过,这并不妨碍你对它做出推断:

  • 由于reduce是在a上被调用的(并且它返回到一个空数组),a应该是一个数组类型。

  • 回调参数i将是该数组的一个元素。

  • reduce的第二个参数,一个空的对象{},告诉我们回调参数o是一个字典(对象)。

所以,通过重命名,我们可以得到如下结果:

function ib(array, fn) {return (array || []).reduce((dict, element) => {dict[fn(element)] = element;return dict; }, {});}

通过上面的调整,可以看到fn是把数组元素变成字典的键。这就揭示了函数ib的目的:将数组转化为字典,用一个自定义的回调来确定索引每个元素的键。你可以把fn改名为getKey,而ib应该被命名为indexBy。重新命名一些标识符有助于我们理解代码,而不需要改变它的逻辑,也不需要一下子考虑所有的部分。如果可以的话,强烈推荐修改。毕竟这样可以提高代码的可读性,将使整个团队受益,同时它并没有增加或改变程序的功能。

2、搞清楚代码是如何被调用的

大多数代码都会被其他代码调用。如果你在纠结一段代码,那么搞清楚它的调用情况对于了解它的功能有非常大的帮助。可以将方法重命名为ThisBreaksOnPurpose。然后进行编译,尽管在通过反射访问的情况下,你在运行时才会看到错误,但编译的错误提示会告诉这个方法在哪里被使用。
如果以上方法不可行,你可以通过文本搜索方法名。如果你很幸运,这个方法的名字在代码库中是唯一的。如果不是这样,你可能会得到一个更大的结果集,并且不得不翻阅大量不相关的代码。

3、搜索类似的代码

有时,即使所有的标识符都被很好地命名,用例也很清晰,但是代码还是很难理解。不是所有的代码都符合编码习惯。有时某个特定的操作并没有遵循编码习惯。在最坏的情况下,有问题的代码出现在工作的代码库中,同时也没有使用明显的惯用语。
然而真正独特的代码在长期存在的代码库中是很少见的,特别是在单个表达式或代码行上。如果花几分钟时间在项目中搜索类似的代码,你可能会发现一些蛛丝马迹来解开整个谜题。
全文搜索是其中最简单的方法。你可以选择一个突出的代码片段进行搜索,搜索工具通常包括一个 "全词 "搜索选项,这意味着搜索care.exe不会返回scare.exertion这样的结果。如果你想进一步缩小范围,可以用正则表达式而不是文本短语进行搜索。
当然,偶尔即使是正则表达式也不足以缩小范围,没有人愿意花几个小时在搜索结果中寻找可能没有帮助的东西。学习一些高级搜索技术也是值得的。许多程序员喜欢使用Unix的命令行工具,如grep和awk,或者在Windows上使用手写的PowerShell脚本。我的首选是JS Powered Search,这是一个VS Code扩展,可以让你在JavaScript中定义一个逻辑搜索查询。

4、运行单元测试

在一个完美的代码库中,你可以通过使用单元测试了解代码运行的状态。但是大多数代码库并不完美;由于效率的原因,单元测试工作往往显得可有可无,有时单元测试所描述的是过时的行为。尽管如此,检查并执行代码测试仍旧是一个好主意。至少,他们会描述代码的输入和输出。
如果没有单元测试或者单元测试不够全面,你还有第二次挽救的机会。可以编写一两个测试来证明代码是否存在的问题。如果发现问题并修复它然后提交修改,增加代码库的稳定性,让这段代码具有自解释的能力。你永远不必担心增加自动化测试会破坏现有的功能。
测试需要花费时间来编写,但此举可以大大提升代码执行效率。测试是代码正常工作的实际证据,有单元测试在你就会相信代码功能不会被破坏。

5、使用Debugger工具

一旦有了单元测试,就有了很好的机制帮助你进行逐步的调试。设置一个断点或在这段代码的顶部添加一个断点/调试器语句。然后运行测试。一旦碰到了断点,执行就会暂停,你可以每次前进一行,进入和退出函数,并检查范围内所有变量的值。
如果你知道哪些用户行为触发了相关的代码,你就可以设置断点并正常运行程序,与程序的界面进行交互。如果你这样做,反馈回路会更长,但它也会使用更真实的数据,这可能有助于你发现空引用和边缘案例。
从上到下的逐行调试可能对运行几十或几百次的代码不太有用,比如嵌套的循环。对于这样的代码,可以在每个循环中添加汇总的变量,方便在循环结束的时查看总量。许多集成开发环境还允许你设置条件性断点,可以通过设置条件在循环中暂停并进入断点从而查看对应变量的值。

6、搜索知识库

如果你的团队把编写文档作为开发过程的一部分,你可以快速跳过这一步。文档不应该是唯一的真理来源,你应该依靠代码来了解程序的行为方式。
文档虽然可以解释代码的 "How",但它往往更擅长解释 "Why"。有时你明白一段代码在做什么,但从另一个角度看貌似有些不对。所以在改变它之前,你应该尽一切努力去了解原来的程序员是根据什么信息或约束来编码的。
一篇好的内部文档也能为你指出知道真相的队友。如果你已经走到了这一步,做了足够多的工作,那么可以向外寻求帮助。确保让对方知道你在做什么工作,你想解决什么问题,他们很有可能会注意到你的视野盲区。

7、使用版本控制注释

看到这里,你已经了解了几种有效的代码阅读策略。但即使如此,也可能会有无法解决的问题:一个奇怪的设计决定,一个打破代码库编码模式的方法,一个没有明显理由的代码特质。
版本控制系统可以显示代码库中任何一行代码的作者和提交。在Git中,就是git blame命令。大多数系统称它为"blue"或"annotate"。你可以在命令行或IDE中运行这个命令。出现的将是一个逐行的提交列表:一个提交哈希值,一个提交信息,以及一个作者。
如果该行代码的最近一次提交没有意义——例如它是一个格式化或空白的变化,就需要通过文件的变更历史来找到引入该行代码的提交。同样,版本控制系统有一些工具可以帮助你做到这一点。
一旦你拿到了PR和Ticket,不仅拥有了代码的背景,还可以找到与之相关的工作人员:代码的作者、PR审核者、任何评论或更新Ticket的人、签署QA的人。如果前几种方法都不奏效,那么是时候该和前辈们聊聊了。

8、先理解,再编码

通过对以上步骤的学习,或许对你有所帮助,特别是对代码背景的理解以及功能的实现方面。在你继续前进之前,还需要考虑重构代码以使其清晰,创建新的文档,在这里投入的任何时间都会让你和你的团队在代码的互动中获得回报。
有效阅读代码的能力是一种秘密武器,它可以使你快速通过技术面试,并使你成为任何团队的重要成员。擅长写代码的程序员是有价值的,擅长读代码的程序员就更具价值了。当生产中出现错误或急需开发新功能时,第一步也是最重要的一步就是理解,阅读代码是能让你顺利到达彼岸。

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

推荐阅读更多精彩内容