将参数传递给回调函数的正确方法是什么？

Question

这是一个工作示例。 正如你可以看到我打电话setElemHeight函数从返回的值getElemHeight 。 但我认为传递值而不是函数是一种不好的做法。

选项 1)我可以将它包装在匿名函数中，如下所示：

function () { getElemHeight('.map-svg') }

选项 2)我可以在setElemHeight函数中调用它并在setElemHeight传递参数，但我认为它会破坏开放封闭原则。

我的问题是：将参数传递给回调函数的正确方法是什么？

谢谢。

function(elem) {
  return $(elem).height()
}

function setElemHeight(elem, callback, offset) {
  let elemHeight = callback
  $(elem).height(elemHeight - offset)
}
$(window).on('load resize', function() {
  setElemHeight('#dealersTabContent', getElemHeight('.map-svg'), 190)
})

Answer 1

将值传递给函数就可以了，除非元素的高度随时间发生变化。

如果它发生变化，您将需要在每次调用事件处理程序时重新计算高度。 如果没有，您可以只传入该值，因为无论如何它将始终使用相同的值。

此外，这一行：

 let elemHeight = callback

如果callback是一个函数，这实际上不会调用回调，它只是将函数分配给一个新变量。
如果您希望elemHeight获得返回值，则需要这样做：

 let elemHeight = callback() // <== Notice the parenthesis

如果它是一个值，您可能想要更改名称。

至于开放封闭原则，我认为在 JS 中，作为一种动态的弱类型语言，这可能主要适用于模块而不是特定的代码片段。 在 JS 中创建封装和只允许在某些地方扩展有一些技巧，但通常你可以在任何地方插入任何你想要的东西，所以我不担心它。