CODEWITHSUNDEEP
javascriptnode.jscoffeescriptpromise
grasshopper
grasshopper

Reputation: 958

How to write a function that returns an error in javascript

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.

Upvotes: 3

Views: 24510

Answers (1)

Bergi
Bergi

Reputation: 665456

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")

Upvotes: 2

Related Questions