败犬日报 2025-09-25

1. 群友对他们组的一些要求

以下是我在 code review 中不会特意关注的方面。

这些方面我认为应该在 code review 之前就已经妥善处理，如果在 code review 环节处理纯属浪费 reviewer 时间。

如果这些方面出现问题，那说明 coding 极度不负责任，或者基本常识缺失，或者两者皆有。

如果我在 code review 中发现了以下问题，我会在第一次和第二次发现的时候指出。从第三次发现起，我会直接 code review -2 并且不会告诉你具体哪里犯了低级错误。如果因此造成了项目延期，自行承担后果。

你的头文件导入出现问题，包括但不限于：
- include 了 .h，.hpp 以外的文件；
- include 了没有被使用的文件；
- 为了省事 include 了万能文件头（即，你可以只 include 某个具体的 h 文件 A，但你没有这么做，而是 include 了文件 B，而文件 B include 了 A）；
- 总之只要你遵从 include-what-you-use 原则就不会有这方面的问题。
你的函数 / 变量命名出现问题。
- 命名标准见 7. 命名约定 — Google 开源项目风格指南；
- 如果我在 struct 中看到以下划线结尾的成员变量，我会直接 -2。
你的 change 开发了新的代码，这很好，但是你没把新产生的 dead code 删掉。dead code 包括但不限于：
- 因为此 change 导致的 dead 文件；
- 因为此 change 导致的多余的 include；
- 因为此 change 导致的逻辑上不再需要的变量；
- 因为此 change 导致的逻辑上不再需要的计算。
出现泛滥的不知所云的 log。
- log 中使用了除英文之外的语言（包括但不限于汉语拼音，日语罗马音）。
- INFO << "AAAAAAA"; 这种也是不知所云
将 dead code 注释掉而不是删除。
- 某些条件下的 log 可以注释掉。

来源，图上的每一项可点击。