一一中文网

手机浏览器扫描二维码访问

第45章 代码审查（第1页）

周一上午九点，林晨坐在工位上，盯着Github页面上那个刚刚创建的pullRequest（代码合并请求），心跳有些加速。

这是他在新公司的第一个pR。

过去一周，他几乎每天加班到晚上十点，周末也泡在代码里。

任务不算复杂：为一个deFi（去中心化金融）项目的前端添加一个质押收益计算器组件。

他用了自己最熟悉的React方式完成了开发，自测了几遍，功能都正常。

鼠标在“createpullrequest”

按钮上悬停了十几秒，林晨深吸一口气，点了下去。

页面跳转，pR编号#347出现在屏幕上。

他顺手在Slack的技术频道里@了techLead陈峰：“陈哥，质押计算器组件的pR已提，麻烦有空review一下”

。

消息刚发出去，陈峰几乎秒回：“收到，上午处理”

。

林晨松了口气，起身去接水。

路过陈峰的工位时，看到对方已经点开了pR页面，眉头微皱，手指在键盘上快速敲击。

回到座位，林晨强迫自己不去刷新页面，转而开始看项目里另一个模块的代码。

但注意力总是不集中，每隔几分钟就忍不住瞥一眼屏幕右下角的Slack图标。

这种等待代码审查的感觉，比他预想的要焦虑。

在上一家跨境电商公司，他的代码审查通常很顺利——十年老员工，对业务系统和代码库了如指掌，提交的pR往往只有些格式小调整。

但在这里，一切都是新的：新技术栈、新编码规范、新的最佳实践，甚至新的思维方式。

上午十点半，Slack图标终于闪动起来。

陈峰：“@林晨看完你的pR了，问题比较多，我们快速过一下？现在方便的话来小会议室”

。

林晨心里一沉，回复：“好的，马上来”

。

拿起笔记本和笔，他走向走廊尽头的小玻璃会议室。

推门进去时，陈峰已经坐在里面，面前的macbook屏幕上正是那个pR的页面，右侧评论栏里密密麻麻的红色标记。

“坐”

。

陈峰指了指对面的椅子，语气平静但严肃，“我们先整体说，然后你回去逐条修改”

。

林晨坐下，打开笔记本准备记录。

“首先，功能实现没问题，计算逻辑是对的”

。

陈峰开门见山，“但问题出在代码质量、安全性和可维护性上。

我一条条说，你记一下”

。