Question

我一直在研究一个函数，该函数将采用相对目录名，递归遍历目录及其所有子目录，并创建包含文件的整个结构的数组。

我从这里完成的工作开始：https://gist.github.com/kethinov/6658166

我正在开发一个独立的应用程序，可以利用ES6 / ES7。

以下代码有效。但是，我的主要目标是提高我的编码技能，所以我想知道是否有更好的方法？更高效？功能更强大？

我知道我可以将命名函数直接移动到.map中并使用箭头函数但我应该这样做吗？在这种情况下，我并没有真正返回任何内容，所以使用更简洁的语法不太明确？我没有明确地返回并利用该返回而是依赖于副作用（？）这一事实是不起作用的。

const fs = require('fs');
const path = require('path');


function walkSync(dir, array = []) {

  function _process(collectionElement) {

    const nextJoin = path.join(dir, collectionElement);
    array.push(nextJoin.replace(/\\/g, '/'));

    if (fs.statSync(nextJoin).isDirectory()) {

      walkSync(nextJoin, array);

    }

  }

  fs.readdirSync(dir).map(_process);

  return array;

}

console.log(walkSync('directory'));

编辑于4/23/17

所以我相信以下内容更具功能性和“更纯粹”。不确定我是否还能做得更好？

const fs = require('fs');
const path = require('path');


// Builds an array of all directories and files
function processDirectory(content, directory, array) {

  const item = path.join(directory, content);

  // reformat for Windows environment
  array.push(item.replace(/\\/g, '/'));

  if (fs.statSync(item).isDirectory()) {

    // eslint-disable-next-line no-use-before-define
    return walkDirectorySync(item, array);

  }

  return array;

}

function walkDirectorySync(directory, array = []) {

  // node reads the directory SYNCHRONOUSLY (maintains order & BLOCKS)
  fs.readdirSync(directory).map(content => processDirectory(content, directory, array));

  return array;

}


console.log(walkDirectorySync('world'));

Answer 1

在这里带来了lodash模块。首先，您可以在walkDirectorySync中将map更改为reduce，并将其传递给一个数组而不是注入一个数组。其次，你可以讨论processDirectory函数，它允许你将目录压入processDirectory并将curried函数传递给reduce的每次迭代。这会将processDirectory转换为reduce函数。最后，你可以通过克隆processDirectory中的数组而不是修改原始数据来更纯粹 - 当然是以性能成本，但这对你有用。

这应该代表一种更纯粹/更实用的方法。我相信总会有更多改进的空间，但希望这能让你有所思考。

const fs     = require('fs'),
      lodash = require('lodash'),
      path   = require('path');

function walkDirectorySync(directory) {
  return fs.readdirSync(directory).reduce(processDirectory(directory), []);
}

const processDirectory = lodash.curry(function (directory, accumulator, content) {
  accumulator = accumulator.slice(); // trying to be pure here

  const item = path.join(directory, content).replace(/\\/g, '/'); // reformats for Windows environment

  accumulator.push(item);

  if (fs.statSync(item).isDirectory()) {
    // eslint-disable-next-line no-use-before-define
    accumulator.push(...walkDirectorySync(item));
  }

  return accumulator;
});

清除函数以从文件目录结构

1 个答案: