Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
xindoo
eng-practices-cn
提交
077f0576
E
eng-practices-cn
项目概览
xindoo
/
eng-practices-cn
通知
1
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
E
eng-practices-cn
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
前往新版Gitcode,体验更适合开发者的 AI 搜索 >>
未验证
提交
077f0576
编写于
11月 05, 2019
作者:
H
Hanfei Sun
提交者:
GitHub
11月 05, 2019
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Update comments.md
修改翻译错误
上级
54ee2e09
变更
1
隐藏空白更改
内联
并排
Showing
1 changed file
with
1 addition
and
1 deletion
+1
-1
review/reviewer/comments.md
review/reviewer/comments.md
+1
-1
未找到文件。
review/reviewer/comments.md
浏览文件 @
077f0576
...
...
@@ -29,7 +29,7 @@
## 接受解释
当你要求开发人员解释一段你看不懂的代码的时候,通常的结果是
**
他们重写代码,让代码更清晰**
。当一段代码不是太过于复杂的时候通过加一些注释偶尔也是一种不错的做法。
与其要求让开发人员解释一段你看不懂的代码,其实更应该做的是
**让
他们重写代码,让代码更清晰**
。当一段代码不是太过于复杂的时候通过加一些注释偶尔也是一种不错的做法。
**解释仅仅是写在Code Review工具里的,不利于将来的代码阅读者。**
只有极少数情况是可行的,比如你对你评审的需求不太熟悉,但是开发人员解释的是大多数人都知道的。
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录