评审反馈与风险分级

共 18 题
#

1. Review 反馈的"优先级"(priority)中 nit、suggestion、issue、blocking 的分级,如何让作者优先处理高价值意见?

A 所有意见都同权,作者自行取舍
B 分级会降低作者主动性,应取消
C nit/suggestion/issue/blocking 分级让作者优先处理高价值意见,blocking 必须修复 ✓ 正确答案
D 只有 blocking 需要记录,其余忽略
#

2. Review 反馈的"具体性"(specificity)原则中指向行号、给出示例、解释原因

A 反馈只需指出"这里有问题",定位由作者自己找
B 具体性会让反馈过于冗长,应尽量简短
C 具体反馈应指向行号、给出示例并解释原因,让作者直接可行动 ✓ 正确答案
D 原因说明是多余的,作者照着改即可
#

3. Review 反馈的"可操作性"(actionable)中给出修复建议而非单纯指出问题,如何提升意见的可执行性?

A 可操作反馈给出修复方向与理由,让作者知道如何改,提升采纳率 ✓ 正确答案
B 指出问题就够了,怎么改是作者的事
C 可操作反馈会限制作者思路,应只指出问题
D 可操作性只适用于新手评审
#

4. Review 反馈的"客观证据"(objective evidence)中文档、规范、测试、metrics 引用

A 反馈应只用个人观点,不需要证据
B 客观证据只在大型项目需要
C 引用规范、文档、测试与 metrics 等客观证据能减少主观争执、提升说服力 ✓ 正确答案
D 引用证据会让反馈过于冗长
#

5. Review 反馈的"建设性"(constructive)中聚焦代码而非个人

A 反馈应直接指出"你写错了",最直接有效
B 建设性意味着只说好话,不指出问题
C 建设性反馈聚焦代码而非个人,用客观语言描述问题,保护心理安全 ✓ 正确答案
D 措辞只影响感受,不影响采纳率
#

6. 代码变更的"风险分级"(risk classification)中 critical、high、medium、low 的客观标准

A 风险分级只由评审者主观感觉决定
B 分级只影响发布,不影响评审
C 所有变更风险相同,无需分级
D 依据影响面、波及范围、数据/资金敏感性等客观维度分级,并驱动评审强度与门禁 ✓ 正确答案
#

7. 风险分级的"变更影响"维度中单服务 vs 跨服务、单库 vs 跨库

A 单服务与跨服务变更风险相同
B 跨服务/跨库变更涉及契约、发布顺序与一致性,风险显著更高,需更严格评审 ✓ 正确答案
C 单库变更一定比跨库更危险
D 影响维度不影响风险分级
#

8. 风险分级的"变更类型"维度中 feature、refactor、bugfix、perf、security

A 不同类型(重构/安全/性能等)风险特征不同,评审关注点应相应调整 ✓ 正确答案
B 所有变更类型风险相同,评审方式一致
C 安全修复风险最低,可快速合并
D 重构无任何风险
#

9. Review 反馈中 emoji 表情的使用规范(:bulb:、:warning:、:hammer:)

A emoji 越多越好,能表达所有情绪
B 统一约定 emoji 语义(如 :warning: 表示高风险)可辅助快速分类,但应适度使用 ✓ 正确答案
C emoji 会显得不专业,应完全禁用
D emoji 只用于个人消息,不用于评审
#

10. Review 反馈的"thread"(讨论串)中保持讨论集中避免分散

A thread 把同一问题的讨论收敛在特定位置,便于迭代与回溯 ✓ 正确答案
B 每个问题都应单独开多条评论,越分散越好
C thread 会阻碍讨论,应避免使用
D thread 只对远程评审有意义
#

11. critical 级变更的多重评审(multi-reviewer)要求

A critical 变更只需一个人看,避免拖延
B 多重评审只用于大型团队
C 评审者越多质量一定越高
D 多重评审提供独立视角互补,降低漏检与权威偏差,适合高风险变更 ✓ 正确答案
#

12. 评审中"我不同意但作者有道理"时如何收敛,避免无限辩论?

A 无客观依据的分歧应尊重作者选择,避免无限辩论,必要时升级仲裁 ✓ 正确答案
B 评审者必须坚持自己的意见直到作者同意
C 分歧越多人讨论越好,无限辩论也应继续
D 作者永远无权决定,评审者说了算
#

13. 如何避免评审变成"找茬竞赛",建立以学习为导向的评审文化?

A 评审员应尽量多报问题,报得越多越优秀
B 强调肯定性反馈、提问式表达与知识沉淀,让评审成为共同学习而非对抗 ✓ 正确答案
C 找茬竞赛能提升评审强度,应鼓励
D 文化建设与评审质量无关
#

14. 评审意见的"拒绝与辩解"流程中作者不同意意见时的记录、讨论与仲裁机制,如何既避免无效争论又保留合理异议?

A 作者必须无条件接受所有评审意见
B 作者可以无视评审意见,无需说明
C 异议应记录、限定轮次讨论,实现细节尊重作者、不可逆决策升级仲裁 ✓ 正确答案
D 任何异议都必须争论到一方服软
#

15. 风险分级的上下文维度中新增代码、修改核心路径与删除代码的评审关注点与风险差异如何判断?

A 新增、修改、删除代码的风险完全相同
B 新增关注设计契合、修改核心路径关注回归、删除关注引用与契约,评审重点应不同 ✓ 正确答案
C 删除代码没有任何风险
D 上下文不影响评审关注点
#

16. 评审的责任边界中评审者负责发现、作者负责修复与验证,如何界定职责避免推诿?

A 评审者应对所有缺陷负责,即使没发现
B 评审者应替作者写好所有修复
C 作者无需回应评审意见
D 评审者负责发现并说明,作者负责修复验证并对变更负责,二者互补 ✓ 正确答案
#

17. low 级变更的快速评审(quick review)或快速合并

A low 级变更也必须完整评审,不能省
B low 级变更无需任何检查即可合并
C low 级变更可走快速通道(快速评审或自动合并),释放评审精力给高价值变更 ✓ 正确答案
D 所有变更都应快速合并
#

18. 评审意见与修复提交的追溯中如何把每条意见关联到对应提交,便于审计与复盘?

A 通过 thread resolved、提交引用等把意见与修复关联,形成闭环便于审计与复盘 ✓ 正确答案
B 评审意见与修复提交无关,无需关联
C 追溯只用于合规审计
D 提交信息不应引用评审意见