将额外参数传递给 NodeJs Http 回调方法
Pass extra parameter to NodeJs Http callback method
我正在用 NodeJs 编写一个 API 包装器,并且正在尝试干净地处理来自 http.get() 的响应。问题的根源是我讨厌所有教程中匿名内联定义回调方法的不干净的编码风格。
// Bad. Can get ugly if the handle function has multiple lines or nested callbacks
http.get("www.example.com", function(response){/* handle response */});
// Better. Follows clean-code guidelines. Code is more reusable. Etc
var handleResponse = function(response){
// handle response
}
http.get("www.example.com", handleResponse);
虽然我更喜欢后者,但我似乎无法将额外的参数传递给 handleResponse,特别是我希望 handleResponse 调用的回调。
我目前的有效方法:
module.exports = function() {
return {
apiGet: function(url, callback) {
http.get(url, function(response) {
var result = handleResponse(response);
callback(null, result);
});
}
}
}
我想要的(但不起作用)
module.exports = function() {
var handleResponse = function(response) {
var result = handleResponse(response);
callback(null, result);
};
return {
apiGet: function(url, callback) {
http.get(url, handleResponse);
}
}
}
此代码的问题在于回调函数未在方法 handleResponse() 中定义。我似乎无法解决这个问题。
我尝试过的东西。
// hoping extra parameters get passed to function. Nope.
return {
http.get(url, handleResponse, callback);
}
// Trying out a suggestion from a random blog I found while googling for an answer. Nope.
return {
http.get(url, handleResponse.bind({callback: callback});
}
所以我发现了一些在 this related thread 中有效的东西。
return {
http.get(url, function(response){ handleResponse(response, callback); });
}
这似乎是一个合理的妥协。我可以将一个长匿名函数变成一个小匿名函数,它只需使用所需信息调用我的显式函数。
在开始之前,有人对我要完成的工作有任何其他建议吗?我一直在寻求最高级别的代码可读性。
如果你有一个主要工作是做某事然后将控制流传递给其他函数的函数,那么也许你应该考虑编写一个 makeResponseHandler()
而不是编写 handleResponse()
函数。基本上,使用函数工厂:
function makeResponseHandler (callback) {
return function (response) {
// deal with response here
callback(null, result);
}
};
return {
apiGet: function(url, callback) {
http.get(url, makeResponseHandler(callback));
}
}
注意:如果仔细观察,您实际上并没有将 makeResponseHandler
传递给 http.get()
,而是调用 makeResponseHandler()
并将其 returns 传递给 http.get()
.
为什么不使用:
var req = http.get("www.example.com");
req.SOME_VARIABLE = "something";
req.on("response", function(res){
//Do something with req.SOME_VARIABLE
});
我正在用 NodeJs 编写一个 API 包装器,并且正在尝试干净地处理来自 http.get() 的响应。问题的根源是我讨厌所有教程中匿名内联定义回调方法的不干净的编码风格。
// Bad. Can get ugly if the handle function has multiple lines or nested callbacks
http.get("www.example.com", function(response){/* handle response */});
// Better. Follows clean-code guidelines. Code is more reusable. Etc
var handleResponse = function(response){
// handle response
}
http.get("www.example.com", handleResponse);
虽然我更喜欢后者,但我似乎无法将额外的参数传递给 handleResponse,特别是我希望 handleResponse 调用的回调。
我目前的有效方法:
module.exports = function() {
return {
apiGet: function(url, callback) {
http.get(url, function(response) {
var result = handleResponse(response);
callback(null, result);
});
}
}
}
我想要的(但不起作用)
module.exports = function() {
var handleResponse = function(response) {
var result = handleResponse(response);
callback(null, result);
};
return {
apiGet: function(url, callback) {
http.get(url, handleResponse);
}
}
}
此代码的问题在于回调函数未在方法 handleResponse() 中定义。我似乎无法解决这个问题。
我尝试过的东西。
// hoping extra parameters get passed to function. Nope.
return {
http.get(url, handleResponse, callback);
}
// Trying out a suggestion from a random blog I found while googling for an answer. Nope.
return {
http.get(url, handleResponse.bind({callback: callback});
}
所以我发现了一些在 this related thread 中有效的东西。
return {
http.get(url, function(response){ handleResponse(response, callback); });
}
这似乎是一个合理的妥协。我可以将一个长匿名函数变成一个小匿名函数,它只需使用所需信息调用我的显式函数。
在开始之前,有人对我要完成的工作有任何其他建议吗?我一直在寻求最高级别的代码可读性。
如果你有一个主要工作是做某事然后将控制流传递给其他函数的函数,那么也许你应该考虑编写一个 makeResponseHandler()
而不是编写 handleResponse()
函数。基本上,使用函数工厂:
function makeResponseHandler (callback) {
return function (response) {
// deal with response here
callback(null, result);
}
};
return {
apiGet: function(url, callback) {
http.get(url, makeResponseHandler(callback));
}
}
注意:如果仔细观察,您实际上并没有将 makeResponseHandler
传递给 http.get()
,而是调用 makeResponseHandler()
并将其 returns 传递给 http.get()
.
为什么不使用:
var req = http.get("www.example.com");
req.SOME_VARIABLE = "something";
req.on("response", function(res){
//Do something with req.SOME_VARIABLE
});