JavaScript：在函数间传递大对象或字符串被视为不良实践

Question

JavaScript：在函数间传递大对象或字符串被视为不良实践

6

在函数之间传递一个大字符串或对象（假设是从ajax响应中获取的），这是否被认为是一种不好的做法？将响应保存在变量中并重复使用该变量有什么好处吗？

因此，在代码中，应该像这样：

var response;

$.post(url, function(resp){
   response = resp;
})

function doSomething() {
  // do something with the response here
}

vs

$.post(url, function(resp){
   doSomething(resp);
})

function doSomething(resp) {
  // do something with the resp here
}

假设 resp 是一个大的对象或字符串，并且可以在多个函数之间传递。

- Mr. Smee

3

传递函数参数比杂乱全局命名空间更可取。 - Blunderfest

1

@Malcoda 它不一定要是全局的，它可以在闭包中。 - Scimonster

5

在JavaScript中传递对象引用并不昂贵。 - Pointy

1

在函数式编程中，你传递一切，这是很好的。 - dandavis

1

将response = resp;这样赋值外部变量的做法必然会让代码难以维护（如果它能工作的话，还要考虑不可避免的竞态条件）。从闭包中赋值可能看起来很有用，但我总是能找到更好的解决方案，通常涉及启动回调，传递直接参数，并查看回调的返回值。如果这种策略似乎太普通，那么它通过有效性和可维护性获得了它的普遍性。 - Keen

显示剩余3条评论

1个回答

网页内容由stack overflow 提供, 点击上面的

可以查看英文原文，
原文链接

- jfriend00 · Accepted Answer

Javascript中的对象和字符串是传递引用的。 (严格来说是按值传递引用，这意味着在函数内部使用=重新分配变量不会影响函数外的变量，但这与本问题无关。)

这意味着将它们传递给函数并不昂贵，因为不会复制任何内容。传递给函数的只是指向原始对象的指针，这是高效的。

你需要认识到你的第一个方案甚至不能正常工作：

var response;

$.post(url, function(resp){
   response = resp;
})

function doSomething() {
  // do something with the response here
}

因为事情的时机，你不知道何时调用doSomething()。正如你展示的$.post()是异步的，因此你无法确定它实际上何时完成。在你的代码中，必须使用完成函数中的resp值。你必须要么在完成函数中使用它，要么从完成函数中调用某些东西并将resp传递给它（就像你的第二个示例）。只有这样，当数据可用时才能正确地控制时间。