El Psy Congroo

Code Review 约定

统一的代码格式 使用统一的配置文件，提交前进行格式化（不属于review内容） 编程风格，可读性和可维护性 变量命名是否做到简洁、清晰无歧义（clear，precise） 方法是否进行了合理的抽象，避免为了复用而复用（方法vs代码块） 方法抽象层次是否合理（同级方法都在同一个抽象层次上） 方法是否只做了一件事，是否和命名相符，是否有副作用 是否遵循OOP原则（SOLID），类是否内聚，是否使

单元测试规范

基本原则 测试任何可能失败的事物。测试主执行路径很好，而且很需要做；但测试异常处理可能更重要。 测试先行。在写任何代码之前，必须先写一个失败的测试。 为还没有实现的测试代码抛出一个异常。这就避免了该测试通过，而且会提醒你必须实现其代码。 不应该依赖于任何不可控的外部服务，如xingng，使用mock替代 不应该依赖于外部的数据（即不要依赖数据库中现有数据等，以保证数据库变更等不会影响到测试结果），