Question

我正在通过learnyounode练习。特别是“过滤ls”。如果您不熟悉此练习，请参阅this对其的说明。

var fs = require('fs');
var path = require('path');
fs.readdir(process.argv[2], function (err, list) {
  list.forEach(function (filename) {
    if (path.extname(filename) === '.' + process.argv[3]) {
      console.log(filename);
    }
  });
});

以下是我的解决方案：

var fs = require('fs');
var path = require('path');
var directory = process.argv[2];
var fileExtension = "." + process.argv[3];

function sortFiles (callback){

    fs.readdir(directory, function(err,files){

        if(err) return callback(err);

        callback(null,files);
    });
};

sortFiles(function(err,files){
    var fileExt = "";
    for(var i = 0; i < files; i++){
        debugger;
        fileExt = path.extname(files[i]);

        if(fileExt === fileExtension){
            console.log(files[i]);
        };
    };
});

问题：

我的代码什么也没输出。据我所知，我的代码和官方解决方案之间的最大区别如下：

      list.forEach(function (filename) {
        if (path.extname(filename) === '.' + process.argv[3]) {
        console.log(filename);
      }

VS

    for(var i = 0; i < files; i++){

        fileExt = path.extname(files[i]);

        if(fileExt === fileExtension){
            console.log(files[i]);
        };
    };

2个问题：

1）如何调试此代码？我试过了node debug node.js。从中可以看出，当我的目录是包含cs文件和js文件的文件夹时，files是一个数组，但每个索引都包含“”。

2）为什么我的简单for循环方法不起作用？他们为什么选择.foreach？除了它更简洁之外还有什么理由吗？

Answer 1

1）https://github.com/node-inspector/node-inspector

2）忘记检查length：

for(var i = 0; i < files.length; i++){
                         --^--

但是forEach读得更好，而且更加惯用。

想知道如何调试Node.js learnyounode“filter ls”练习

1 个答案: