抱着试试看的心态让它 review 自己的代码,本来没指望什么。

提了三条,都成立:

  1. 有个循环里做了数据库查询,数据量大了会慢
  2. 异常捕获之后只记日志没处理,等于吞掉了
  3. 有个函数的参数超过六个,调用处可读性差

第三条我认了但没改,前两条当场就修了。

新的用法:每次提交前让它扫一遍,只问「有没有性能或异常处理上的明显问题」。范围窄,误报少。

泛泛地问「帮我看看代码」,它会挑一堆风格问题,反而没价值。

主楼