How do I avoid deeply nested code in node.js?

Question

In node.js, it being event-driven, all I/O is done via callbacks. So I end up writing code that looks like this:

app.get('/test', function (req, res) {
  http.get('some/place', function (req1, res1) {
    if (res1.statusCode == 200) {
      res1.on('data', function (data) {
        http.get('other/place?q=' + data, function (req2, res2) {
          if (res2.statusCode == 200) {
            res2.on('data', function (data) {
              db.query(data).on('data', function (rows) {
                res.writeHead(200)
                res.end(JSON.stringify(rows))
              })
            })
          }
        })
      })
    }
  })
})

And that doesn't even include error handling.

What can I do to unwind this mess?

WOW! Well that's nested all right!

Trufa
– Trufa

2011-02-04 21:34:49 +00:00
Commented Feb 4, 2011 at 21:34 — Trufa
– Trufa, Commented Feb 4, 2011 at 21:34
Possible duplicate: stackoverflow.com/questions/4234619/…

Anderson Green
– Anderson Green

2012-09-23 04:59:11 +00:00
Commented Sep 23, 2012 at 4:59 — Anderson Green
– Anderson Green, Commented Sep 23, 2012 at 4:59

Alfred · Accepted Answer · 2011-02-05 02:30:27Z

6

You could use async module to avoid this.

answered Feb 5, 2011 at 2:30

Alfred

62k33 gold badges151 silver badges186 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

mikerobi · Accepted Answer · 2012-05-05 16:28:25Z

6

Don't use anonymous functions.

EDIT

Your code isn't even valid. You aren't closing most of your function calls.

If you switched to named functions it would look something like this: Updated to reflected comment about global namespace

(function () {
    app.get('/test', f0)

    function f0(req, res) {
      http.get('some/place', f1)
    }

    function f1(req1, res1) {
        if (res1.statusCode == 200) {
          res1.on('data', f2)
        }
     }
    function f2(data) {
        http.get('other/place?q=' + data, f3)
    }
    function f3(req2, res2) {
      if (res2.statusCode == 200) {
        res2.on('data', f4)
      }
    }

    function f4(data) {
          db.query(data).on('data', f5)
        }

    function f5(rows) {
        res.writeHead(200)
        res.end(JSON.stringify(rows))
    }
})()

edited May 5, 2012 at 16:28

answered Feb 4, 2011 at 21:38

mikerobi

21k5 gold badges49 silver badges43 bronze badges

4 Comments

nornagon Over a year ago

ah, that was why it wasn't syntax highlighting properly. Fixed.

mikerobi Over a year ago

@nornagon, that doesn't change my answer.

mikerobi Over a year ago

@nornagon, there is a very simple solution for that, I have updated my example.

tbranyen Over a year ago

This code is invalid. Ironic. You cannot immediately invoke an unnamed function declaration.

nornagon · Accepted Answer · 2011-02-13 02:12:16Z

I wrote a library based on node-seq, which looks like this:

app.get('/test', function (req, res) {
  Seq()
    .seq(function () {
      http.get('some/place', this.next)
    })
    .seq(function (req1, res1) {
      if (res1.statusCode == 200) {
        res1.on('data', this.next)
      }
    })
    .seq(function (data) {
      http.get('other/place?q=' + data, this.next)
    })
    .seq(function (req2, res2) {
      if (res2.statusCode == 200) {
        res2.on('data', this.next)
      }
    })
    .seq(function (data) {
      db.query(data).on('data', this.next)
    })
    .seq(function (rows) {
      res.writeHead(200)
      res.end(JSON.stringify(rows))
    })
})

The code is here.

Also, there's a lengthy discussion on the nodejs mailing list about this issue.

Step is another library for doing this stuff.

George · Accepted Answer · 2011-07-19 20:38:29Z

1

Please have a look at Streamline; it is a JavaScript preprocessor that allows you to write simple 'streamlined' code and transforms it to callback-heavy code.

answered Jul 19, 2011 at 20:38

George

111 bronze badge

Comments

Pappa · Accepted Answer · 2013-02-04 22:23:19Z

Another (and perhaps better) way to clean up this kind of thing is to use EventEmitters rather than callbacks. Here's an example showing several EventEmitters in use:

    var events = require('events'),
    util = require('util');

    var Search = function () {
    "use strict";

    var search,
        requestPageData,
        makeReturnObject,
        sendResults,

        emptyObj = {};

    events.EventEmitter.call(this);

    search = function (query) {
        this.emit("requestPageData", query);
    };

    requestPageData = function (query) {
        var resultsArray = [];

        // some logic

        if (.....some condition....) {
            this.emit("makeReturnObject", resultsArray);
        } else {
            this.emit("sendResults", emptyObj);
        }
    };

    makeReturnObject = function (resultsArray) {
        var resultsObj = {};

        if (magnetArray) {

            // some logic

            this.emit("sendResults", resultsObj);
        } else {
            this.emit("sendResults", emptyObj);
        }
    };

    sendResults = function (obj) {
        // finally, do whatever it is you want to do with obj
    };

    this.on("requestPageData", requestPageData);

    this.on("makeReturnObject", makeReturnObject);

    this.on("sendResults", sendResults);

    this.search = search;

};

util.inherits(Search, events.EventEmitter);
module.exports = new Search();

phani · Accepted Answer · 2014-03-19 22:13:19Z

0

You can use promises. Check this out https://github.com/kriskowal/q

answered Mar 19, 2014 at 22:13

phani

1,1341 gold badge11 silver badges24 bronze badges

Collectives™ on Stack Overflow

How do I avoid deeply nested code in node.js?

6 Answers 6

Comments

4 Comments

Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

Comments

4 Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related