如何用 Ramda.js 重构这个组合函数？

Question

我正在使用ramda 和data.task 编写一个小型实用程序，它可以从目录中读取图像文件并输出它们的大小。我让它像这样工作：

const getImagePath = assetsPath => item => `${assetsPath}${item}`

function readImages(path) {
  return new Task(function(reject, resolve) {
    fs.readdir(path, (err, images) => {
      if (err) reject(err)
      else resolve(images)
    })
  })
}

const withPath = path => task => {
  return task.map(function(images) {
    return images.map(getImagePath(path))
  })
}

function getSize(task) {
  return task.map(function(images) {
    return images.map(sizeOf)
  })
}

const getImageSize = dirPath => compose(getSize, withPath(dirPath), readImages)

问题在于 withPath 函数将正确的图像路径添加到图像文件名，但强制我的 api 两次传入 directoryName：一次用于读取文件，第二次用于读取路径。这意味着我必须像这样调用getImageSize 函数：

const portfolioPath = `${__dirname}/assets/`

getImageSize(portfolioPath)(portfolioPath).fork(
  function(error) {
    throw error
  },
  function(data) {
    console.log(data)
  }
)

有没有办法将dirname 作为参数只传递一次？我希望 api 像这样工作：

getImageSize(portfolioPath).fork(
  function(error) {
    throw error
  },
  function(data) {
    console.log(data)
  }
)

Answer 1

我设法通过将Task 分辨率传递给一个像这样的对象来解决这个问题：

function readImages(path) {
  return new Task(function(reject, resolve) {
    fs.readdir(path, (err, images) => {
      if (err) reject(err)
      else resolve({ images, path })
    })
  })
}

const withPath = task => {
  return task.map(function({ images, path }) {
    return images.map(getImagePath(path))
  })
}

...然后将其从任务负载中销毁，现在我的 compose 函数如下所示：

module.exports = (function getImageSize(dirPath) {
  return compose(getSize, withPath, readImages)
})()

我的 api 调用如下所示：

getImageSize(portfolioPath).fork(
  function(error) {
    throw error
  },
  function(data) {
    console.log(data)
  }
)

Answer 2

你不应该像那样手动构建路径

Node 更好的 API 之一是 Path module——我建议您将 readImages 包装器设为通用 readdir 包装器，而是解析 path.resolved 文件路径的数组

const readdir = dir =>
  new Task ((reject, resolve) =>
    fs.readdir (dir, (err, files) =>
      err
        ? reject (err)
        : resolve (files.map (f => path.resolve (dir, f)))


const getImagesSizes = dir =>
  readdir (dir) .map (R.map (sizeOf))

仅仅为了返回 Task 而包装 Node 持续传递样式 API 会很麻烦，不是吗？

const taskify = f => (...args) =>
  Task ((reject, resolve) =>
    f (...args, (err, x) =>
      err ? reject (err) : resolve (x)))

const readdir = (dir, ...args) =>
  taskify (fs.readdir) (dir, ...args)
    .map (R.map (f => path.resolve (dir, f)))

const getImagesSizes = dir =>
  readdir (dir) .map (R.map (sizeOf))

您可能还应该注意归档 目录 的文件路径 - 除非您的 sizeOf 实现处理该路径