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

[az22c]

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

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

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

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

[KuroNekoFan]

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

[FaiChou]

「有内存泄漏的风险」这句话怎么说都是正确的, 谁也不能保证浏览器是否释放应该被释放的变量. 但 「 (...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]

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

[soulmt]

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

[az22c]

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

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

connect(mapStateToProps, mapDispatchToProps)(Component)


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

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

[KuroNekoFan]

@az22c 按照闭包的定义，在一个 function 里在定义一个 function 就形成了一个闭包，但是闭包跟内存泄漏没有必然联系...

[finalwave]

@FaiChou #2 这不是 V8 同一作用域函数共用内存和块状作用域内函数变量提升搞出来的问题嘛，换个引擎就好了（狗头

[FaiChou]

@finalwave #7 GC 是引擎自己实现的, 代码都是有 bug 的, 所以它有泄漏的风险 ( 逃

[usw]

咋着，还指望把 dispatch 给你回收掉

[wxsm]

遇到这种情况建议追问：具体是那一部分的内存泄漏？怎么个泄露法？怎么写才能不泄露？如果对方不懂，那就暴露了。如果真懂，那就学到了。