使用闭包限制函数调用
使用闭包限制函数调用
提问于 2016-04-22 03:57:29
问题陈述：
我们的web应用程序中有许多功能被调用得太频繁了。我们希望您创建一个函数，给定一个函数和一个时间间隔，返回一个新函数，我们可以随时调用它，但确保每个间隔调用原始函数的次数不超过一次。示例用法: limitedDoSomething = rateLimit(doSomething，500)；现在，对
limitedDoSomething的调用只会调用doSomething，但不会超过每500毫秒调用一次。
测试用例：
function logHello(){ // just a sample function
console.log('Hello!');
}
limitedDoSomething = rateLimit(logHello, 500);
limitedDoSomething(); // should log “Hello!”
window.setTimeout(function(){
limitedDoSomething(); // should not log “Hello!"
}, 400);
window.setTimeout(function(){
limitedDoSomething(); // should log “Hello!"
}, 500);我的解决方案：
function rateLimit(func, limit){
var lastInvokedTimestamp;
return function(){
if(typeof lastInvokedTimestamp === 'undefined' || Date.now() - lastInvokedTimestamp >= limit){
lastInvokedTimestamp = Date.now();
console.log('go ahead!');
return func();
} else {
console.log('too early!');
return;
}
};
}
// run test cases
function logHello(){ // just a sample function
console.log('Hello!');
}
limitedDoSomething = rateLimit(logHello, 500);
limitedDoSomething(); // should log “Hello!”
window.setTimeout(function(){
limitedDoSomething(); // should not log “Hello!"
}, 400);
window.setTimeout(function(){
limitedDoSomething(); // should log “Hello!"
}, 500);我很想知道是否有办法改进我的解决方案。
回答 1
Code Review用户
回答已采纳
发布于 2016-04-22 07:02:34
以下是我能想到的一些小改进。没什么大不了的：
- 初始化
lastInvokedTimestamp，这样就不会是undefined了。 - 缓存当前时间。
- 跟踪下一次呼叫的阈值，这样它就变成了每个间隔加一次，而不是每个入口减一次。
- 将参数传递到已取消的函数中。
- 简化测试用例的
setTimeout。
function rateLimit ( func, interval ) {
var nextInvokeTimestamp = 0;
return function(){
var now = Date.now();
if ( now < nextInvokeTimestamp ) {
return console.log('too early!');
}
nextInvokeTimestamp = now + interval;
console.log('go ahead!');
return func.apply( this, arguments );
};
}
// run test cases
function logHello( message ){
console.log( message || 'Hello!' );
}
limitedDoSomething = rateLimit( logHello, 500 );
limitedDoSomething( "Welcome!" ); // should log “Welcome!”
setTimeout( limitedDoSomething, 400 ); // should not log “Hello!"
setTimeout( limitedDoSomething, 500 ); // should log “Hello!"您可能会注意到，David是完全不同的做这事。
- 使用
setTimeout将实际调用延迟到稍后的事件循环，而不是当前事件，以使事情稳定下来一些。 - 由于延迟(可以低到0，即4ms)，他可以使用间隔中的最新参数进行调用，例如鼠标的最新移动。更大的间隔可以让您捕捉更多的呼叫，而响应更慢。
这并不是好或坏，只是不同的需要不同的减速带而已。如果您的要求只是“将多个调用简化为一个”，那么您做得很好。
页面原文内容由Code Review提供。腾讯云小微IT领域专用引擎提供翻译支持
原文链接：
https://codereview.stackexchange.com/questions/126390
复制相关文章
相似问题
腾讯云开发者
