年度代码滚动更新站点的前端代码审查问题摘要



阿里梅指南。

* 回顾在技术团队的工程师文化建设是很有意义的，这是形成统一团队作风的最有效方法，笔者自己团队在一年的CR中常见的小问题做了一些梳理，希望能对大家有所帮助。 一、引言。

团队已经开展了一年多的线下周评，作为主评委，我原本以为给出**建议会花费大量的时间和精力，一直以为对被评审者改进**设计问题会有很大的帮助，但实际上评审发现大部分学生还存在一些基本的**质量问题， 而改进设计或发现业务bug以达到目标的审查很少出现，所以我觉得团队对CR还是应该有足够的耐心，把基本问题当成一个bug来解释，在通往优秀工程的路上积累起来。同时，我也能深深感受到，每个学生对**的态度都不一样，这就产生了两个差异，有的人进步很快，有的人会不断重复复习过的问题。我希望它对每个人都能帮助不大。二、翻车场景（CR中的常见问题）。

2.1 **规范类

2.1.1 使用法力值展期指数：描述：Mana 表示一个未在 ** 中声明并直接使用的值，这在我们的 ** 复习问题中很普遍。

bad case

const now = date.now();const lastvisittime = localstorage.getitem('last_visit_time');

if (lastvisittime &&parseint(lastvisittime, 10) >24 * 60 * 60 * 1000)

localstorage.setitem('last_visit_time', now.tostring())

const last_visit_time_cache_key = 'last_visit_time';const day_in_ms = 24 * 60 * 60 * 1000;

const now = date.now();const lastvisittime = localstorage.getitem(last_visit_time_cache_key);

if (lastvisittime &&parseint(lastvisittime, 10) >day_in_ms)

localstorage.setitem(last_visit_time_cache_key, now.tostring())

2.1.3 使用 Ghost 依赖项展期指数：描述：幽灵依赖项（也称为隐式依赖项或隐式依赖项）是不在项目模块中的包，但不在该模块中直接在 JSON 文件中声明的依赖项。

2.1.4 不遵循“不归还”原则展期指数：描述：no-else-return 是一个质量规则，强调简洁性和可读性，用于指示当 if 块已包含 return 语句时无需使用 else 块。 因为 if 块中的 return 语句会在执行 if 块中的 return 语句后立即退出当前函数，所以 else 是多余的。 ❌ you can skip the else block

function hello(condition: boolean) else

function hello(condition: boolean)

return '