0

トップスレッドと呼ばれる投稿の配列があり、配列の配列を取得しようとしています。各配列には、元の配列の1つの投稿に対するすべての返信が含まれています。これがトップスレッドを取得するための私のコードです

exports.topThread = function(req, res){
// gets the leaf id from URL and assigns to OPID
var OPID = req.params.id;
var thread = [];
// finds the post with id OPID from the db
titles.findOne({postID: OPID}, function (err, post) {
if (err) throw err; 


getBestThread(OPID, thread, function(result){

    branchList.getBranches(function(branches){

        // renders content with data retrieved from the post
        res.render('content', {title: post.title, content: post.body, OPID: post.postID, currentThread: result, branches: branches});
        console.log(result);

    });
    });
});

};

コードのこの部分は機能します。結果は、必要な投稿の配列です。次に、結果配列の各要素をgetAllOtherReplies関数に渡します。

//calback to retrieve other replies for a parent comment
    getOtherReplies = function(parentID, callback){
        postdb.otherReplies(parentID, function(err, commentsArray){
            if (err) throw err;
            callback(commentsArray);
        })
    }

    getAllOtherReplies = function(threadArray, returnArray, callback) {
        async.each(threadArray, 
            function(value, callback) {
                getOtherReplies(value.commentID, function(commentsArray) {
                    console.log(value.commentID);
                    if (commentsArray)
                        returnArray.push(commentsArray);
                });
                callback();
            },
            function(err) {
                if (err) throw err;
            }
        );
        callback(returnArray);
    }

そしてこれがpostsDBの関数です：

//retrieves other replies from db
    exports.otherReplies = function(parentID, callback){

        titles.find({type: "comment", parentID: parentID}).sort({hotness: -1}).toArray(function(err, result){
            if (err) throw err;
            if (result.length > 1)
                callback(result.splice(0,1));
            else callback(null);
        });
    };

ここで、元のコードを変更してgetAllOtherReplies関数を呼び出そうとします。

    exports.topThread = function(req, res){
        // gets the leaf id from URL and assigns to OPID
        var OPID = req.params.id;
        var thread = [];
        var allOtherReplies = [];
        // finds the post with id OPID from the db
        titles.findOne({postID: OPID}, function (err, post) {
        if (err) throw err; 


        getBestThread(OPID, thread, function(result){
            getAllOtherReplies(result, allOtherReplies, function(returnArray){
                console.log(returnArray);

                    branchList.getBranches(function(branches){

                    // renders content with data retrieved from the post
                    res.render('content', {title: post.title, content: post.body, OPID: post.postID, currentThread: result, branches: branches});
                    console.log(result);

                });
            });
            });
        });
    };

結果の各コメントに対するすべての応答の配列の配列を返す代わりに、各配列が結果の最初のコメントのみへの応答である配列の配列を返します。n回繰り返されます。nは結果の投稿の数です。私は私が見逃している非同期の問題があることを知っています、どんな助けもいただければ幸いです。

4

1 に答える 1

1

async.eachループ内では、コールバック内にgetOtherReplies追加commentsArrayしていますが、直後に非同期コールバックを呼び出しており、コールバックをgetOtherReplies待っていません。

非同期コールバックを に移動getOtherRepliesすることは良い出発点であり、問​​題をうまく解決できる可能性があります。

getOtherReplies(value.commentID, function(commentsArray) {
    console.log(value.commentID);
        if (commentsArray) {
            returnArray.push(commentsArray);
        }
        callback();
    });
于 2013-02-18T21:28:27.853 に答える