For investors
股价:
5.36 美元 %
认识达内从这里开始
认真做教育 专心促就业
我们在进行软件开发的时候，为了能够节约开发时间，有时候有些功能是可以进行合并执行的。今天，我们就一起来了解下在合并需求的时候，都有哪些比较常见的问题。
1.不佳思索的赞同
事情就是这么有诱惑力。这次的合并请求规模真的很大，提交者又是你信任的某个人。他们已经在这份代码上工作一段时间了，而且一直都处理的很好。不要说你还有自己的截止日期(deadline)要跟。
2.拖延
毕竟是一个这么大的合并请求，为什么非要现在审核?而且你现在的任务实在太重要了。你终会抽时间来处理它的，是吧?或者你可能只是在等其他人发表意见……
扪心自问，让原力贯通全身!在你内心抵制的背后可能会有一些真实的抱怨。
3.标准diff
你是否正在审查一些乱七八糟的代码?GitHub和企业版GitHub的默认diff工具是“标准diff”。这种模式下，为了把一系列的修改都表示在一个文件中，GitHub会关注增加和删除的行并很智能地将修改编成组块，而且这些都是行内表示。但是你知道吗，大多数情况下，“标准”diff是很难阅读的。所谓智能的组块选择其实一点也不智能。
4.风格盖过本质
只有了解上下文环境才能做到真正的代码审查。有些在审查表面结构/风格的时候，看起来合理的地方在你理解了终极目标之后可能就变得不可接受了。
5.没有捕获不完善的修改
diff工具确实能很清晰地告诉你哪些地方有修改。但这也是问题所在!显然它们没告诉你哪些地方没有改变。要当心有些本应该应用更广泛的修改，像find/replace这类的可能没有覆盖整个代码库。
.粉饰太平的测试代码
合并请求里只要有一些测试代码，很容易就让人产生这很安全的错觉。如果他们把测试考虑进去，那他们一定是高质量且综合全面的。是这样的吗?
7.低估前端的复杂度
如果CSS和HTML有修改，会倾向于把它看作和算法代码更改一样。看到格式良好的修改就会想象他们在浏览器里如何执行。“看上去很合理，”你说。
8.思维狭隘
还有一个地方可能会因为diff里格式良好的代码让你昏昏欲睡。但考虑大规模情况是很重要的。目前项目里的新代码，到底有什么变化?会发生什么?
9.短期思考
有些合并请求里有相当多的往返，可能是因为意见不一致或者是阐明的需要。这确实是一件好事情—它在建立共享上下文环境。但是当下一个开发人员来了并拿到这份代码会发生什么呢?他们可能就没那么容易理解这场讨论了。
10.审查修正的时候草草了事
终于，你集中注意力完成了这个合并请求，并将它返回给提交者。也给出了反馈，提交者回应正在修正。
译者：fzr
节选：博客园
【免责声明】：本内容转载于网络，转载目的在于传递信息。文章内容为作者个人意见，本平台对文中陈述、观点保持中立，不对所包含内容的准确性、可靠性与完整性提供形式地保证。请读者仅作参考。
< 上一篇：初级程序员培养成本过高原因分析
下一篇：运城seo优化网站改版需要做哪些准备工作 >
2024-03-29
免费试听课程
- 全部课程
- IT课程
- 设计课程
- 运营课程
4578
最新开班时间
- 北京
- 上海
- 广州
- 深圳
- 南京
- 成都
- 武汉
- 西安
- 青岛
- 天津
- 杭州
- 重庆
- 哈尔滨
- 济南
- 沈阳
- 合肥
- 郑州
- 长春
- 苏州
- 长沙
- 昆明
- 太原
- 无锡
- 石家庄
- 南宁
- 佛山
- 珠海
- 宁波
- 保定
- 呼和浩特
- 洛阳
- 烟台
- 运城
- 潍坊
- 开课名称
- 开班时间
- 抢座
- 咨询
- 开课名称
- 开班时间
- 抢座
- 咨询
-
- Java全链路开发
- 4月26日
- 火热抢座中
- 立即咨询
- 云计算全栈开发
- 4月26日
- 火热抢座中
- 立即咨询
-
- 网络安全工程师
- 4月26日
- 火热抢座中
- 立即咨询
- 人工智能工程师
- 4月26日
- 火热抢座中
- 立即咨询
-
- 数据分析与商业智能
- 4月26日
- 火热抢座中
- 立即咨询
- C++物联网工程师
- 4月26日
- 火热抢座中
- 立即咨询
-
- Web全栈开发
- 4月26日
- 火热抢座中
- 立即咨询
- 软件测试工程师
- 4月26日
- 火热抢座中
- 立即咨询
-
- AI大模型全栈工程师
- 4月26日
- 火热抢座中
- 立即咨询
- 鸿蒙原生应用开发
- 4月26日
- 火热抢座中
- 立即咨询
-
- 智能职场办公
- 4月26日
- 火热抢座中
- 立即咨询
- VFX商业视效设计
- 4月26日
- 火热抢座中
- 立即咨询
-
- AGI商业设计变现
- 4月26日
- 火热抢座中
- 立即咨询
- UID全链路设计
- 4月26日
- 火热抢座中
- 立即咨询
-
- 新媒体电商运营
- 4月26日
- 火热抢座中
- 立即咨询
-
- Java全链路开发
- 4月26日
- 火热抢座中
- 立即咨询
- 云计算全栈开发
- 4月26日
- 火热抢座中
- 立即咨询
-
- 网络安全工程师
- 4月26日
- 火热抢座中
- 立即咨询
- C++物联网工程师
- 4月26日
- 火热抢座中
- 立即咨询
-
- Web全栈开发
- 4月26日
- 火热抢座中
- 立即咨询
- 软件测试工程师
- 4月26日
- 火热抢座中
- 立即咨询
-
- AI大模型全栈工程师
- 4月26日
- 火热抢座中
- 立即咨询
- 鸿蒙原生应用开发
- 4月26日
- 火热抢座中
- 立即咨询
-
- 智能职场办公
- 4月26日
- 火热抢座中
- 立即咨询
- VFX商业视效设计
- 4月26日
- 火热抢座中
- 立即咨询
-
- AGI商业设计变现
- 4月26日
- 火热抢座中
- 立即咨询
- UID全链路设计
- 4月26日
- 火热抢座中
- 立即咨询
-
- 新媒体电商运营
- 4月26日
- 火热抢座中
- 立即咨询
-
- 云计算全栈开发
- 4月26日
- 火热抢座中
- 立即咨询
- 网络安全工程师
- 4月26日
- 火热抢座中
- 立即咨询
-
- C++物联网工程师
- 4月26日
- 火热抢座中
- 立即咨询
- Web全栈开发
- 4月26日
- 火热抢座中
- 立即咨询
-
- 软件测试工程师
- 4月26日
- 火热抢座中
- 立即咨询
- AI大模型全栈工程师
- 4月26日
- 火热抢座中
- 立即咨询
-
- 鸿蒙原生应用开发
- 4月26日
- 火热抢座中
- 立即咨询
- 智能职场办公
- 4月26日
- 火热抢座中
- 立即咨询
-
- VFX商业视效设计
- 4月26日
- 火热抢座中
- 立即咨询
- AGI商业设计变现
- 4月26日
- 火热抢座中
- 立即咨询
-
- UID全链路设计
- 4月26日
- 火热抢座中
- 立即咨询
- 新媒体电商运营
- 4月26日
- 火热抢座中
- 立即咨询
-
- Java全链路开发
- 4月26日
- 火热抢座中
- 立即咨询
- 网络安全工程师
- 4月26日
- 火热抢座中
- 立即咨询
-
- 数据分析与商业智能
- 4月26日
- 火热抢座中
- 立即咨询
- C++物联网工程师
- 4月26日
- 火热抢座中
- 立即咨询
-
- Web全栈开发
- 4月26日
- 火热抢座中
- 立即咨询
- 软件测试工程师
- 4月26日
- 火热抢座中
- 立即咨询
-
- AI大模型全栈工程师
- 4月26日
- 火热抢座中
- 立即咨询
- 鸿蒙原生应用开发
- 4月26日
- 火热抢座中
- 立即咨询
-
- 智能职场办公
- 4月26日
- 火热抢座中
- 立即咨询
- VFX商业视效设计
- 4月26日
- 火热抢座中
- 立即咨询
-
- AGI商业设计变现
- 4月26日
- 火热抢座中
- 立即咨询
- UID全链路设计
- 4月26日
- 火热抢座中
- 立即咨询
-
- 新媒体电商运营
- 4月26日
- 火热抢座中
- 立即咨询
-
- 云计算全栈开发
- 4月26日
- 火热抢座中
- 立即咨询
- 网络安全工程师
- 4月26日
- 火热抢座中
- 立即咨询
-
- C++物联网工程师
- 4月26日
- 火热抢座中
- 立即咨询
- Web全栈开发
- 4月26日
- 火热抢座中
- 立即咨询
-
- 软件测试工程师
- 4月26日
- 火热抢座中
- 立即咨询
- AI大模型全栈工程师
- 4月26日
- 火热抢座中
- 立即咨询
-
- 鸿蒙原生应用开发
- 4月26日
- 火热抢座中
- 立即咨询
- 智能职场办公
- 4月26日
- 火热抢座中
- 立即咨询
-
- VFX商业视效设计
- 4月26日
- 火热抢座中
- 立即咨询
- AGI商业设计变现
- 4月26日
- 火热抢座中
- 立即咨询
-
- UID全链路设计
- 4月26日
- 火热抢座中
- 立即咨询
- 新媒体电商运营
- 4月26日
- 火热抢座中
- 立即咨询
预约申请试听课
开设课程
开课城市
友情链接
Copyright © 2023 Tedu.cn All Rights Reserved 京ICP备08000853号-56 京公网安备 11010802029508号达内时代科技集团有限公司 版权所有
注：学员信息+企业信息均来自达内教育内部CRM系统真实统计，非宣传使用。
课程适用人群：成人
400-111-8989
lihm@tedu.cn
