改的是正则里的一个量词,从 * 改成 +,本地测试过了,上线之后部分输入匹配失败。
问题在于测试用例没覆盖空字符串的情况。
复盘:
- Agent 改动范围很小,我 review 时扫了一眼就放过了。小改动最容易出问题,因为没人仔细看。
- 缺少边界测试。
改进:
- 正则、日期处理、编码转换这三类改动,必须补边界测试
- review 时对「一个字符的改动」格外警惕
现在的习惯:让它改这类代码时,先要求它列出所有边界情况,再动手。
主楼- 暂无回复
继续逛逛
改的是正则里的一个量词,从 * 改成 +,本地测试过了,上线之后部分输入匹配失败。
问题在于测试用例没覆盖空字符串的情况。
复盘:
改进:
现在的习惯:让它改这类代码时,先要求它列出所有边界情况,再动手。