最新消息:雨落星辰是一个专注网站SEO优化、网站SEO诊断、搜索引擎研究、网络营销推广、网站策划运营及站长类的自媒体原创博客

node.js - How to write a function that returns an error in javascript - Stack Overflow

programmeradmin6浏览0评论

In mongoose.js after every query the functions returns (error, result). I would like to write function of my own that does the same.

I thought the answer was to use promises so I wrote this

  login: (user) ->
    q.Promise (resolve, reject, notify) ->
      console.log resolve
      if _.has(user, 'password')
        dashboard.users.find(user).exec().then (err, results) ->
          if err then reject("Error occurred with the database")
          if results then resolve(results) else reject("User not found")
      else
        reject("A password is required for login")

Auth.login(testuser).then (d) ->
    console.log d, "done"

but after research I found that this is an anti pattern and the code does not work (promise is never resolved/rejected). So now I am not sure what to do.

In mongoose.js after every query the functions returns (error, result). I would like to write function of my own that does the same.

I thought the answer was to use promises so I wrote this

  login: (user) ->
    q.Promise (resolve, reject, notify) ->
      console.log resolve
      if _.has(user, 'password')
        dashboard.users.find(user).exec().then (err, results) ->
          if err then reject("Error occurred with the database")
          if results then resolve(results) else reject("User not found")
      else
        reject("A password is required for login")

Auth.login(testuser).then (d) ->
    console.log d, "done"

but after research I found that this is an anti pattern and the code does not work (promise is never resolved/rejected). So now I am not sure what to do.

Share Improve this question edited Nov 26, 2014 at 6:37 grasshopper asked Nov 26, 2014 at 6:28 grasshoppergrasshopper 9584 gold badges14 silver badges30 bronze badges 5
  • How do you know the promise is never resolved/rejected? I don't see that you're using the actual promise anywhere. Did you mean to return the promise from the login() method? – jfriend00 Commented Nov 26, 2014 at 6:35
  • yes I did I forgot to add that code I'll add it now – grasshopper Commented Nov 26, 2014 at 6:36
  • Like I said, you have to return the promise from the login() method so the callers of login() can use it. – jfriend00 Commented Nov 26, 2014 at 6:38
  • my original code had this but it is still the same result you are saying I should have deferred.promise returned at the end right? – grasshopper Commented Nov 26, 2014 at 6:51
  • Avoid the Promise constructor antipattern! – Bergi Commented Aug 21, 2022 at 14:13
Add a ment  | 

1 Answer 1

Reset to default 2

To return errors from functions as promises, there are generally two ways to produce rejected promises:

  • The reject function of your Promise library
  • Throwing an exception in a then callback or returning a rejected promise from it will reject the resulting promise

This has the same effect as building a promise for an async function that calls its node-style callback with an error argument. Your code would be rewritten to

  login: (user) ->
    if _.has(user, 'password')
      q(dashboard.users.find(user).exec()).then (results) ->
        if results 
          results
        else
          throw new Error("User not found")
      , (err) ->
        throw new Error("Error occurred with the database")
    else
      q.reject("A password is required for login")
发布评论

评论列表(0)

  1. 暂无评论