如果我不打算返回任何东西,我应该使用像map和过滤这样的数组方法吗?

2022-03-29 00:00:00 arrays javascript ecmascript-6 eslint

在过去一年中,我更多地使用map和Filter等数组方法,而不是在数组上使用标准for loop。它感觉读写起来更简单,而且做了我最有可能做的所有事情,比如创建一个局部变量。

很多时候,我不会返回任何东西。不过,埃斯林特不是很喜欢我。根据他们的说法,他们说你总是需要退货,否则"可能是个错误" https://eslint.org/docs/rules/array-callback-return

为什么?只是良好的实践吗?无返回值的数组方法有什么缺点?

对此思考了一段时间。任何见解或想法都是伟大的。


解决方案

如果我不打算返回任何内容,是否应该使用map和Filter等数组方法?

不,您不应该这样做。

为什么?只是良好的实践吗?

是的。对于您正在进行的迭代类型,使用适当的迭代方法是一种很好的实践。迭代的方法有很多种,这是有原因的。选择适当的机制。

无返回值数组方法的缺点是什么?

使用.map().filter()而不从回调中实际返回任何内容有以下缺点:

  1. 您的代码具有误导性。.map().filter()的目的是迭代数组并生成新数组。当开发人员阅读一些代码并看到使用.map().filter()时,他们预计应该会有一个返回的数组。当他们不这样做时,他们会感到困惑,一开始会觉得自己不理解代码。如果我对这样的代码进行代码审查,我不会批准这样的代码。

  2. 您的代码不必要地创建了不使用的对象。这是浪费,不是好做法。相反,请使用不会产生输出数组的迭代方法,如for/of、常规的for循环或.forEach()

  3. 您的代码无法链接。Linters对事物提出异议是有原因的。正如林特所说,使用.map().filter()而不从回调中返回任何内容很可能是编程错误,因为这些函数不是这样设计的,当您不想要返回的数组时,有合适的替代方法。

因此,如果您只是尝试在不创建任何结果数组的情况下进行迭代,请使用for/of.forEach()或某些不是专门为创建您不需要的输出数组而专门设计的迭代方案。

相关文章