How to make sync function from async in node.js?

Question

I've a return type function:

var responseData = '';
function getResponseFromKey(key){

    client.get(key, function(err, reply){
        responseData = reply;
    });
    return responseData;
}

When I call this function first time it returns blank and then second time returns my value what I want after running again.

I'm calling this function to print in html page.

http.createServer(function(request, response){
    response.writeHead(200, {'Content-Type':'text/plain'});
    getResponseFromKey("my_key");   
    console.log(responseData);
}).listen(8083);

As I'm familiar with node the function is going in asynchronous way. Can you please help me to make synchronous way? Do I need to use generators in that case?

Help would be appreciated!

Possible duplicate of How do I return the response from an asynchronous call? — Ben Fortune
– Ben Fortune, Commented Feb 3, 2017 at 9:06

ThomasThiebaud · Accepted Answer · 2017-02-03 09:08:13Z

You can use callback, promise or generator.

Using callback, you need to send a callback function and call it instead of returning a value

function getResponseFromKey(key, callback){
  client.get(key, function(err, reply){
    callback(reply);   
  });
}

http.createServer(function(request, response){
  response.writeHead(200, {'Content-Type':'text/plain'});
  getResponseFromKey("my_key", function() {
    console.log(responseData);
  });   
}).listen(8083);

Using promise, you need to return a Promise

function getResponseFromKey(key) {
  return new Promise(function(resolve, reject) {
    client.get(key, function(err, reply) {
      return resolve(reply);   
    });
  })
}

http.createServer(function(request, response){
  response.writeHead(200, {'Content-Type':'text/plain'});
  getResponseFromKey("my_key").then(function(responseData) {
    console.log(responseData);
  });   
}).listen(8083);

If you are using and up-to-date version of nodejs, you can use arrow functions, to make your code more readable.

Asif Saeed · Accepted Answer · 2017-02-03 09:07:40Z

0

Your client.get function is async try returing return responseData; inside the client.get body

var responseData = '';
 function getResponseFromKey(key ,cb){

 client.get(key, function(err, reply){
    responseData = reply;
    return cb(responseData);
 });

}

http.createServer(function(request, response){
 response.writeHead(200, {'Content-Type':'text/plain'});
 getResponseFromKey("my_key", function(responseData) {
  console.log(responseData);
});   
}).listen(8083);

answered Feb 3, 2017 at 9:07

Asif Saeed

2,06516 silver badges25 bronze badges

3 Comments

baba ramdev Over a year ago

I'm getting this error while running your code TypeError: cb is not a function

Asif Saeed Over a year ago

have you written this correcty getResponseFromKey("my_key", function(responseData) { console.log(responseData); });

Asif Saeed Over a year ago

callback is the second parameter in a function call.

BD-Joy · Accepted Answer · 2017-02-03 09:18:16Z

0

In many case, the callback is not a good choose.

You can use async,promises,generators and async functions which supported in es7.

I recommend promises beacuse the promise a key features in ES6.

I think this article will help you.

https://blog.risingstack.com/node-js-async-best-practices-avoiding-callback-hell-node-js-at-scale/

answered Feb 3, 2017 at 9:18

BD-Joy

1151 silver badge9 bronze badges

Collectives™ on Stack Overflow

How to make sync function from async in node.js?

3 Answers 3

Comments

3 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related