Обещания JavaScript и оператор if / else

Когда я использую функцию filemanager для каталога (/), код работает хорошо, но когда я вызываю файл (/index.html), код возвращает ошибку.

Я вижу, что проблема в операторе if / else (readdir выполняется, даже если isDir вернула false), но я не знаю, как правильно использовать его с обещаниями.

var fs = require('fs'),
    Q = require('q'),
    readdir = Q.denodeify(fs.readdir),
    readFile = Q.denodeify(fs.readFile);

function isDir(path) {
    return Q.nfcall(fs.stat, __dirname + path)
        .then(function (stats) {
            if (stats.isDirectory()) {
                return true;
            } else {
                return false;
            }
        });
}

function filemanager(path) {
    if (isDir(path)) {
        return readdir(__dirname + path)
            .then(function (files) {
                return files.map(function (file) {
                    return ...;
                });
            })
            .then(Q.all);
    } else {
        return readFile(__dirname + path, 'utf-8')
            .then(function (content) {
                return ...;
            });
    }
}

filemanager('/').done(
    function (data) {
        ...
    },
    function (err) {
        ...
    }
);

person inetbug    schedule 20.02.2014    source источник


Ответы (2)


isDir возвращает обещание, которое всегда является истинным значением. Вам нужно будет поместить условие в обратный вызов then, чтобы получить доступ к логическому значению:

function isDir(path) {
    return Q.nfcall(fs.stat, __dirname + path)
        .then(function (stats) {
            return stats.isDirectory()
        });
}

function filemanager(path) {
    return isDir(path).then(function(isDir) {
        if (isDir) {
            return readdir(__dirname + path)
                .then(function (files) {
                    return files.map(function (file) {
                        return ...;
                    });
                })
                .then(Q.all);
        } else {
            return readFile(__dirname + path, 'utf-8')
                .then(function (content) {
                    return ...;
                });
        }
    });
}
person Bergi    schedule 20.02.2014

Ваш вызов isDir(path) оценивается как обещание. Таким образом, вы не можете получить результат непосредственно из этой функции. Скорее, вы должны дождаться разрешения этого возвращенного обещания, а затем оценить его значение. Итак, вам нужна конструкция типа isDir(path).then(...) вместо if (isDir(path)), которую вы сейчас используете.

person Brian H    schedule 20.02.2014