(...args) => fn(...args) 有没有垃圾回收的问题？

This topic created in 2074 days ago, the information mentioned may be changed or developed.

js 函数柯里化或者实现语法糖经常用到这种逻辑：

const adder = (...args) => dispatch(...args)。 // dispatch 可以任意普通函数。

我看到 redux 源码里面也是这么用，甚至很多开源 react 项目是这么写。最近和面试官讨论，对方说这样会产生闭包，甚至有内存泄漏的风险。

我回头 chrome 再测试调试一下，好像没有看到有闭包。不知道是不是我的理解有问题

args

闭包

dispatch

函数

10 replies 2020-09-12 13:27:58 +08:00

KuroNekoFan

Sep 11, 2020

你面试官记混了吧,是 arguments object 有可能导致内存泄漏，而且跟闭包没关系
https://developer.mozilla.org/en-US/docs/Web/Javascript/Reference/Functions/arguments

ethusdt

Sep 11, 2020

「有内存泄漏的风险」这句话怎么说都是正确的, 谁也不能保证浏览器是否释放应该被释放的变量. 但「 (...args) => dispatch(...args)」这么写并没有内部变量呀, 有啥好泄漏的 ?

不过还是要讲个例子(浏览器的 bug): https://bugs.chromium.org/p/chromium/issues/detail?id=315190

var someClass = function() {
console.log('some');
};

function getter() {
var some = new someClass();
if(true) {
return function() { //I'm done with some and don't need it
return null;
};
} else {
return function() {
return some;
};
}
}

window.f = getter();

这个代码执行完理论上变量 some 会被回收, 因为 closure 没有 catch 住 some, 但通过 devtools 里 memory - [heap snapshot] 可以发现变量 some 没有被 GC.

所以, 这就是定义一个变量都有「有内存泄漏的风险」.

az22c

Sep 11, 2020

@FaiChou 你这个例子是特殊情况，编译的时候没法分析出 if true 。感觉是两回事，话题扯得比较远了。
@KuroNekoFan 是，问题讨论集中到对 args 的引用上，有没有闭包。搞个测试代码测试一下，还是挺明显的

soulmt

Sep 11, 2020

what 这会产生闭包？应该说有可能会产生吧，es6 这种用法不是很多么，赶紧跑吧，这还面啥

az22c

Sep 11, 2020

@soulmt
@KuroNekoFan
好像明白了
像 react-redux，这个“dispatch 函数”是作为函数参数传入本函数中，没有闭包问题。

function mapDispatchToProps(dispatch) {
return {
adder: (...args) => dispatch('function name', ...args),
}
}

connect(mapStateToProps, mapDispatchToProps)(Component)

-------------------------

而如果这个 dispatch 函数作为当前函数的内声明的函数，并且该函数返回 adder，是有闭包。

KuroNekoFan

Sep 11, 2020