wxya（个人网站）CR
态度
双方互相学习，你好我也好
23年腾讯有个文章，说cr就是群里@一下，5秒后恢复done就结束，讲的是业务复杂性带来的代码不可维护
大家也可简简单单蒙混过关，我见过的大多数也都是这样，但是利于成长么？遇到认真CR的人，对于大家来说都是一件好事情
开源社群争吵也不过是正常现象，争吵不必太在意，也不必不在意，心平气和互相探讨一些即可
格式规范
前端eslint
后端java 阿里巴巴插件
加上提交前检查，和提交代码后github自动代码扫描，通过工具解决，这里要减少人为处理
甚至可以找到一些AI插件，提前CR代码，提出问题和修改意见
减小改动范围
减小改动范围（依赖、构建环境，非必要不更换）
有时候为了修复问题，一次升级好多不相关依赖，实际上没有必要
环境也是
没有明确要解决什么问题，盲目变更会带来位置风向
语意合理
单词、方法名称语义要合理，直观符合常识
扩展性
这个就要看你业务知识丰富不丰富，如果你除了代码，业务的事情一点都不了解，怎么来判断这个问题？
功能影响范围
cr的人，基本是长期对整体代码有过了解的，新同事不熟悉的同事，写的时候难免有局限性，要给予提示
测试了吗
偶尔问一句，也许会避免一些问题
小聪明
还是要遵遵循非必要不改动
我见过一个日志输出原本可以用，可她却要修改日志输出为滚动打印，导致Filebeat占用磁盘不释放，修正因为没有ci环境权限代码，最后过了一周才发现日志没有收集，原来不仅改了代码的日志输出配置，还修改Filebeat的收集文件
个人也经常犯这个错，修改的时候要忍住
坏味道
不合理的语义
绕弯弯的代码
Loading...