代码质量与技术债系列分享之一 - 如何做好 Code Review

这个验证函数，严重违反了单一职责，首先包含了多种校验逻辑，还承担了 submit 数据预处理、submit、error处理；看到下面这段代码，大概能够想象 newValidate 出现的原因，为了文章阅读体验， 删除部分代码。✓ 方法学: 了解Devops，极限编程，Scrum，精益，看板，瀑布，结构化分析，结构化设计。✓ 实践: 实践测试驱动开发，面向对象设计，结构化编程，函数式编程，持续集成，结对编程。可以看到第一段代码，没有规律，阅读成本高，第1行, 第5行出现了重复引用。