静态扫描提前卡在提交前，坏味道才进不了代码库
静态扫描把坏味道在进库前拦一道，拦住了进库再清贵十倍。贵在要翻历史提交、要协调所有人重改，重改时大家手头都有活，活挤着改扫描意见，意见越积越没人理。一、卡在提交前
贵在要翻历史提交、要协调所有人重改，重改时大家手头都有活，活挤着改扫描意见，意见越积越没人理。历史欠账列清单，列清了还债有次序，次序乱了一笔笔压死人。覆盖率数字别当目标，冲数字凑出来的测试盖了行没盖心。
卡在提交前还能给人即时反馈，反馈即时改起来快，快了养成习惯。习惯养成了，代码质量在写的时候就好，好过写完一堆再被扫描打回，打回伤的是节奏感。扫描卡在提交前，卡住了坏味道就进不了库，进库再清贵十倍。大改动拆小再评，拆小了评审者看得细，细了漏掉的问题少。覆盖率报告进看板，进看板了团队天天见，见多了才上心。
二、规则别太严
摆设的规则，新人以为没要求，没要求就放飞，放飞后老代码更乱，乱到真有问题时也埋在噪点里。规则随语言版本更新，更新晚了新写法扫不出来等于漏防。关键路径覆盖要比数字重要，重要处稳了整体才敢动。
严宽要分档，分档后基础的卡死，风格的提醒。提醒不挡路，挡路的只有真问题，真问题少而精，精了团队才信这套规则，信了才配合改。规则别太严，太严满屏报错没人看，没人看规则就成了摆设。评审结论要留痕，留痕了日后扯起责任来有凭有据。
三、结果分优先级
不分的话，安全风险和缩进混在一起，缩进淹了风险，风险被忽略，忽略到出事才被翻出来。扫描报告给负责人，给到具体人头上问题才有人认领去改。没覆盖的地方标出来，标出来的人下次改它心里有底。
优先级还要跟着语言特性走，某些语言误报多，误报多的规则降权。降权后真问题更显眼，显眼了改的性价比高，性价比高团队愿意投入，投入形成正向循环。扫描结果要能分优先级，优先的先修，修不完的也知道轻重。新人提交老手先过，过一遍新人学的比看文档快。
四、新规则先试运行
不试就开，开了一堆误报，误报逼人加白名单，白名单越加越长的同时规则越发虚。评审看意图而非格式，格式交给工具，工具比人盯得稳当。覆盖率掉了要报警，报警了才知道哪次提交把防护拆了。
试运行要有人看报告，看报告的把误报清掉，清掉的规则才可信。可信的规则推广，推广时团队阻力小，阻力小执行顺，顺了代码质量稳步抬，抬起来不费劲。新规则先试运行，试运行期间不卡人，看误报多不多再决定。争议点当面聊，聊比来回评论省时间，省下的是双方耐心。
五、接入流水线
不接的话靠人手动，手动就忘，忘了某次没扫，没扫的那次正好带病合并，合并完病散开难收。评审要趁热，趁热改起来快，放凉了作者已切到别的活难回头。分支覆盖比行覆盖更难，难处正是容易藏逻辑缺陷的地方。
流水线挂了要拦合并，拦了逼人修，逼比劝有用。劝的时候大家说稍后，稍后成了永远，永远的债务堆高，堆高了某天集中爆，爆出来全在线上。误报要有人清理，清理了的规则团队才信它弹出的每条提醒。评审别拖过夜，拖久了合并排队，排队久了分支越积越乱。
六、报告给到人
扫描报告给到人头上，给到具体人头上问题才有人认领去改。给到团队笼统一份，笼统的没人认，没人认就晾着，晾着的报告下次也没人看，没人看就停更。一条评论只说一件事，说清了对方改起来不懵，不懵才改得准。历史低覆盖模块排计划，排了才一步步把坑填平，不平总绊脚。
报告还要附修复建议，附了新人才改得动，改得动才有成就感。成就感来自搞定具体问题，具体问题配上解法，解法清楚动手快，快了新人也愿意碰老代码。扫描接入流水线，接入了每次合并自动跑，自动跑才持续。好实践在评审里传，传开了团队水平整体往上走，不靠个人。
静态扫描是低成本的质量闸门，闸门立在提交口，坏味道就进不来。卡点、分档、优先级、试运行、流水线、到人，六件事做对，扫描才真帮忙，不是添堵的唠叨。 不管是内容还是排版都很用心，看得出来楼主花了不少时间，必须支持。 逻辑严谨，条理分明，一看就是经过认真思考和整理的内容。
页:
[1]