code_review
时间：2021-03-08 14:04:45
收藏：0
阅读：0
|
类别
|
检查项
|
检查结果
|
|
代码
|
Readability
|
通过
|
|
Duplicated Code
|
通过
|
|
|
Hard Coding
|
通过
|
|
|
Long Method（100行）
|
通过
|
|
|
Long Parameter List（最多4个）
|
通过
|
|
|
Large Class
|
通过
|
|
|
Switch Statement
|
通过
|
|
|
Lazy Class
|
通过
|
|
|
多线程
|
必要性：总时间，vs 单线程
|
不涉及
|
|
是否可能创建大量线程吃光CPU
|
不涉及
|
|
|
线程间是否负载均衡
|
不涉及
|
|
|
线程池：并行度，队列是否可能把堆打满，是否合理指定饱和策略，是否正确关闭
|
不涉及
|
|
|
数据库
|
是否涉及三表以上的关联查询
|
不涉及
|
|
是否涉及通过非主键更新或删除记录
|
不涉及
|
|
|
确保查询是走索引的，观注读选比
|
不涉及
|
|
|
确保不会一次查询大量结果，将堆打满
|
不涉及
|
|
|
深度分页确保使用索引覆盖 + 批量ID查询的方式实现
|
不涉及
|
|
|
其它
|
是否可能产生死循环
|
不涉及
|
|
是否可能产生死锁
|
不涉及
|
|
|
是否一次性加载大量对象或超大对象，将堆打满
|
不涉及
|
|
|
是否存在重调用
|
不涉及
|
评论（0）