Question

我目前正在实施一个使用Express.js构建的API，遵循我的设计的MVCS模式。在阅读了Async模块之后，我得出的结论是，利用其强大的方法对我来说是一个好主意。不幸的是，我遇到了一个我似乎无法解决的问题，并且想知道是否有人可以给我一些建议。我们来看看代码：

＆＃13;

'use strict'

_ = require 'lodash'
async = require 'async'
validator = require 'express-validator'

class BaseService

  constructor: (@model, @relationships, @fields) ->

  list: (req, callback) ->
    async.waterfall [
      validate(req).bind(this),
      query.bind(this)
    ], callback

  validate = (request) ->
    valid = request
    
    (next) ->
      next null, valid
      return

  query = (valid) ->
    query = @model.find({})

    if valid.query.q
      query = query.where(valid.query.q)

    if valid.query.sort
      query = query.sort(valid.query.sort)

    if valid.query.select
      query = query.select(valid.query.select)

    # Requests hangs at this point. Any ideas why?
    (next) ->
      query.exec next
      return

module.exports = BaseService

＆＃13;

基本上，我想首先验证随请求传递的查询字符串。然后，根据验证对象中是否存在某些字段来构建Mongoose查询。最后，它应该执行查询并将结果传递给回调函数。这里我的实现失败，因为它挂起，并且从不执行回调。我对代码段中的行进行了评论，以便让您更好地了解正在发生的事情。

有人可以指出我正确的方向吗？

Answer 1

您的主要问题是您的query方法签名需要接受回调作为最后一个参数。

query = (valid, callback) ->

对于waterfall，第一个辅助功能需要(callback)，但其余功能需要(previousResult, callback)。

另外，考虑到如何使用query方法，它不需要像你一样返回一个函数。只需接受回调作为参数，并在准备继续完成瀑布时调用它。

旁白：使用＆＃34;查询＆＃34;对于coffeescript类中的方法和局部变量，都容易出错且容易混淆。为局部变量使用不同的名称，例如＆＃34; modelQuery＆＃34;。

Node.js异步瀑布挂起

1 个答案: