javascript - 创建一个函数来执行一个小的重复任务是一种反模式吗？

Question

我正在尝试学习编程的最佳实践，并希望让我的代码尽可能干净，但也可维护。例如，我正在运行一个等待创建每个元素的程序。

a.waitForElementToAppear(50000);
b.waitForElementToAppear(50000);
c.waitForElementToAppear(50000);

现在，我添加了一个函数，这样我就可以轻松更改 50000，而无需手动编辑每个:

function waitForElement(element) {
    element.waitForElementToAppear(5000);
}

并将上面的代码更改为:

waitForElement(a);
waitForElement(b);
waitForElement(c);

创建一个新函数来调用一个相对较小的任务是否是一种反模式？有没有更好的方法？

Answer 1

Is it an anti-pattern to create a new function to call for a relatively small task?

不，绝对没问题!虽然你是对的，你的功能太小了，几乎没有任何收获。

Is there a better approach?

为了实现轻松更改共享参数值的目标，您可以或者(不一定“更好”)也将其放入变量中:

const time = 50000;
a.waitForElementToAppear(time);
b.waitForElementToAppear(time);
c.waitForElementToAppear(time);

最后但同样重要的是，该代码仍然有点重复。另一种选择是循环(尽管这三项只是合理循环的阈值):

for (const element of [a, b, c]) {
    element.waitForElementToAppear(5000);
}