按照敏捷开发，需要代码评审，没有采用Pull Request全部评审，挑选有代表性的代码评审。
空值这破玩意就是一个糟糕的存在。处处都应该校验空值，所有的错都是因为认为不会发生，实际却发生造成的，所有的约定似乎都是鬼扯淡的。比如只用一次的私有函数中不校验参数是否为空，因为写的人知道他自己传入的参数没有空值，但是这样的代码有潜在风险，有一天这个方法被改成公有方法，传入的参数为空值，那就可能出错了。有人说，在修改的时候改的人应该处理新的情况，或许吧，只能说你我的认知不同，做出来不同的决定。
字符串是常用的类型，字符串的解析也是较常见的。需要处理的特殊类型比较多，稍有不慎就会出错。
参数的类型、长度、范围、是否可空等。这是一个坑爹的地方，因为很多时候根本就没有定义具体的范围，这种情况很容易引起不必要的争论。