node.js中的回调

Question

我读过并且听说我不应该在函数中返回值，因为它是一个阻塞操作，并且它可能会拒绝任何请求，直到操作完成。

所以这里是一个我编码的小函数，我想知道如果我正确处理它。我这样说是因为我刚开始使用节点，而且我想以正确的方式进行编码，这也是因为在函数内部有一个测试条件而另一个测试回调的感觉很奇怪。

function isWithinSplit(path, target, separator, callBack) 
{ 
var response = ""; 
var readStream = fs.createReadStream(path); 

readStream.on('data', function (data) { 
    response += data; 
}); 

//Data complete, process it 
readStream.on('end', function (close) 
{ 
    var array = response.split(separator); 

    for (var idx=0 ; idx < array.length; idx++) 
    { 
     if(array[idx] != "" && array[idx] == target) 
      callBack("true"); 
     else 
      callBack("false"); 
    } 
}); 

} 

电话：

fileHelper.isWithinSplit(__dirname + ROOM_LIST_PATH, "hello", "|", function(data){ 
    if(data == "true") 
    console.log("hurray!"); 
}); 

我只是想知道这是否是人们如何做，如果它是有效的。

Answer 1

你忘

• 错误处理
• 使用===
• 缓存阵列长度
• 命名匿名函数

function isWithinSplit(path, target, separator, callBack) { 
    var response = ""; 
    var readStream = fs.createReadStream(path); 

    readStream.on('data', function _aggregateData(data) { 
     response += data; 
    }); 

    //Data complete, process it 
    readStream.on('end', function _findTarget(close) { 
     var array = response.split(separator); 

     for (var idx = 0, len = array.length; idx < len; idx++) { 
      if (array[idx] === target) { 
       return callBack(null, true); 
      } 
     } 
     callback(null, false); 
    }); 

    readStream.on('error', callBack); 
} 

fileHelper.isWithinSplit(__dirname + ROOM_LIST_PATH, "hello", "|", printIfSuccess); 

function printIfSuccess(err, data) { 
    if (data === true) { 
     console.log("hurray!"); 
    } 
} 

你可以也改善它使用Array.prototype.any

readStream.on('end', function(close) { 
    callback(null, response.split(seperator).any(function _doesItMatchTarget(val) { 
     return val === target; 
    })); 
});