-1

此代码有效，但我不知道它是否看起来不错以及 ramda.js 是否正确使用。它看起来实用且可读吗？

我也用 Q 表示承诺。

readDir 读取目录下的所有文件，包括统计信息，这个函数也用在其他地方

你怎么看？

 function readDir(p) {

        return R.pipeP(
            function() {
                return fs.readdir(p);
            },
            R.map(function(file) {

                 return fs.stat(path.join(p, file))
                    .then(function(stats) {

                        return {
                            file: file,
                            stats: stats,
                            fullfilename: path.join(p, file)
                        };
                    })
                    .fail(function(err){
                        log.error(err);
                    });
            }), Q.all
        );
    }



//delete last, filter using statistics
function deleteFilesExceptOne(folder) {

        return R.pipeP(
            function() {
                return readDir(folder)();
            },
            R.filter(function(fl) {
                return (fl.stats.isFile() &&  timespan.fromDates(fl.stats.mtime, new Date()).totalSeconds() > 120);


            }),
            R.tail,
            R.map(function(fl) {

                return fs.unlink(fl.fullfilename);
            }),
            Q.all
        );
    }

非常感谢。

4

1 回答 1

2

虽然我是 Ramda 的作者之一，但我用pipeP的不多，更喜欢Future/Task实现而Promises不是 ，所以我可能不是回答这个问题的最佳人选。但我肯定会建议这不是使用 Ramda 的最惯用的方式。这是您的第二个函数的完全未经测试的版本，其中包含几个辅助函数：

var isFile = function(file) {
    return file.stats.isFile();
};

var olderThan = R.curry(function(seconds, file) {
    return timespan.fromDates(file.stats.mtime, new Date()).totalSeconds() > seconds;
});

var deleteFilesExceptOne = R.pipeP(
    readDir,
    R.filter(R.both(isFile, olderThan(120))),
    R.tail,
    R.map(R.pipe(R.prop('fullfilename'), fs.unlink)),
    Q.all
);

请特别注意由此带来的变化：

function deleteFilesExceptOne(folder) {
    return R.pipeP(
        function() {
            return readDir(folder)();
        },
        //...

对此：

var deleteFilesExceptOne = R.pipeP(
    readDir,
    // ...

如果还有一些有用的清理工作要做，我不会感到惊讶olderThan。

正如我所说，这些都没有经过测试。

于 2015-10-02T18:57:01.250 回答