改的是正则里的一个量词,从 * 改成 +,本地测试过了,上线之后部分输入匹配失败。

问题在于测试用例没覆盖空字符串的情况。

复盘:

  1. Agent 改动范围很小,我 review 时扫了一眼就放过了。小改动最容易出问题,因为没人仔细看。
  2. 缺少边界测试。

改进:

  • 正则、日期处理、编码转换这三类改动,必须补边界测试
  • review 时对「一个字符的改动」格外警惕

现在的习惯:让它改这类代码时,先要求它列出所有边界情况,再动手。

主楼