【转】Eslint团队终于妥协了

大家好,我卡颂。

配置过代码格式化的同学一定纠结过如下问题:EslintPrettier都能格式化代码风格,是单用Eslint,还是两个一起用呢?

从今以后,你再也不用纠结这个问题,因为Eslint团队已经妥协了 —— 根据官方博客所说,从v8.53.0起,Eslint代码风格相关规则将被弃用。

有意思的是,造成上述局面的原因并不是技术问题导致的,更多是市场行为。

本文让我们聊聊事情的来龙去脉。

欢迎围观朋友圈、加入人类高质量前端交流群,带飞

Eslint的崛起

在2013年之前,前端工程师通常使用JSLintJSHint作为代码检查器,用以检测:

  1. 代码质量问题

比如:应该避免使用 eval(),应该使用===而不是==

  1. 代码中的错误

比如:未定义的变量、类型转换的问题…

其中,JSLint基于内部实现的JS解析器,对生成的token流(词法单元流)进行分析,检查代码语法。

[图片上传失败…(image-6f6f19-1698933234955)]

JSHint是从JSLint派生出来的,他们工作原理类似,但JSHint更灵活 —— 他提供了.jshintrc配置文件方便开发者自定义规则。

上述两个工具都能检查代码,但由于实现原理的限制,没法进行复杂的规则检查。同时,他们对代码风格的检查也较少。

在这一时期,代码风格检查(比如:缩进、行长度、引号类型、是否在语句末尾使用分号…)主要交给JSCS

2013年,Eslint问世。他将代码解析为AST并分析:

  • 相比于JSHintJSLint的实现,AST保留了更多代码上下文信息

所以,Eslint不仅可以进行更复杂的规则校验,还能让开发者以插件的形式自己编写规则。

  • 相比于JSCSEslint支持代码自动修复

所以,Eslint不仅能对代码风格提出建议,还能自动修复不符合规范的风格

更先进的功能,再加上作者身份加持(作者是红宝书作者),使得Eslint逐渐淘汰了上述竞品。

[图片上传失败…(image-d6bd25-1698933234955)]

Eslint与Prettier之争

虽然Eslint提供了大量规则,但并不是所有开发者都想配置一套自己的规则集。

慢慢的,一些Eslint规则集的最佳实践被提出(比如Airbnb规则standard规则)。

开发者通常会在这些规则集的基础上再做些个性化修改,组成项目的lint规则集。

这些规则集中,通常包含三类规则:

  • 代码质量检查
  • 代码错误检查
  • 代码风格检查

其中代码风格检查通常是非常主观的。如果团队成员的代码风格检查规则配置不一样,很影响提交时git diff的可读性。

为了强制规范代码风格检查Prettier出现了。这是一款固执己见的代码风格格式化工具,他集成了一套代码风格,并且可配置程度不高。

可配置程度不高是一把双刃剑,一方面,他能强制规范团队成员的代码风格。

但另一方面,如果想对代码风格做些个性化设置,Prettier很有可能不支持。

举个例子(来自为什么我不使用 Prettier 中的例子),Prettier中通过printWidth属性配置一行可以显示的字符数,超过就会折行。

有时候我们并不需要超过某个字符数就折行,因为在Git Diff时,折行会破坏Diff信息的可读性:

然而遗憾的是,Prettier并没有提供配置关闭这一行为。

基于上述原因,出现了两种解决方案:

方案1 EslintPrettier配合使用

其中Eslint负责代码质量、错误检查,Prettier负责代码风格检查。优点是能够满足代码质量、风格检查。缺点是:

  1. EslintPrettier规则可能冲突,配置成本高
  2. 代码风格检查的可配置性低(Prettier配置性低)

方案2 只使用Eslint

使用代码风格相关规则的集合,比如@stylistic/eslint-plugin-js管理代码风格。再使用其他规则管理代码质量。

这种方式优点明显 —— 可配置性高,且配置简单(只需要配置Eslint)。

显然,方案2是优于方案1的。既然如此,Eslint团队为什么要弃用所有代码风格相关规则呢?

Eslint团队的妥协

设想一下,每当出现新的语言特性,与该特性相关的规则包括:

  1. 少量的代码质量相关规则
  2. 少量的代码错误相关规则
  3. 各种奇怪的代码风格规则

显然前两者的优先级、重要性都高于第三者。如果说,在Eslint成长初期,为了收割JSCS的用户,Eslint必须实现所有JSCS支持的代码风格规则,此时实现各种代码风格规则是必要的。

但今时今日,Eslint早已成为JS领域代码检查器的老大,不需要再为了市场份额努力满足社区的一切需要。况且,有些时候,考虑规则冲突以及一致性,有些需求甚至无法满足。

规则冲突

最理想的情况,所有核心规则都能很好地相互配合,这意味着没有两个规则应该标记同一个问题,也不会有任何两个核心规则给出相互冲突的建议。

当核心规则少于30条时,这很容易。但对于越来越多的规则,这很难做到。

一致性问题

ESLint规则之间是无法互相访问的。这意味着我们会遇到无法正确修复错误的问题,因为信息可能位于另一个规则中。

举个例子,如果自动修复需要添加新的代码行,就需要知道文件是如何缩进的,以便应用正确的修复。但是,规则indent控制ESLint的缩进,这意味着其他规则需要在不缩进的情况下应用修复,然后相信indent规则将在后续传递中修复缩进。

总结

ESLintv8.53.0起,将弃用代码风格相关规则。这么做主要是因为继续维护代码风格相关规则对核心团队来说,投入产出比太低。

试想一下,核心团队花费大力气解决问题(规则冲突、一致性问题),推出新的代码风格规则,开发者会感谢Eslint核心团队的付出么?

不会的,这些代码风格规则会被集成到规则集中,并被冠以某种开发理念兜售给开发者(比如Airbnb规范)。

实际收获名利的是站在台前的宣传开发理念的团队,而背后辛苦干活的Eslint核心团队往往被忽略了,换你你乐意么?

原文:https://segmentfault.com/a/1190000044351212