jquery set ajax result to an outside variable callback

Question

I need to set a variable to the results of an ajax query. I realise that i cannot just return result in the success function as this is an asynchronous call. but would like to achieve this will a callback function and not set to a synchronous call.

I would like to achieve something like:

myMethod: function() {
   var result = function(callback) {
      this.getResults(params)
   }
},

getResults: function(params) {
   $.ajax({
      type: 'GET',
      url: 'some/url',
      data: params,
      success: function(data) {
         callback(data).call();
      }
   });
}

Where result = data

I know my syntax is not right, I have tried so many variations and havent found something that works. Any help would be soooooo appreciated. Thank-you!

Matt Ball · Accepted Answer · 2011-06-14 14:01:25Z

2

You're quite close. Pass the callback function to getResults.

myMethod: function() {
    this.getResults(params, callback);
},

getResults: function(params, callback) {
   $.ajax({
      type: 'GET',
      url: 'some/url',
      data: params,
      success: function(data) {
         callback(data);
      }
   });
}

Note that I removed the var result = since that's just not going to work given the asynchronicity you mentioned. If you need to perform further processing of data, it's got to happen in a callback.

answered Jun 14, 2011 at 14:01

Matt Ball

361k102 gold badges655 silver badges725 bronze badges

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

6 Comments

MBax Over a year ago

Thanks! If i haven't allocated data to a variable in myMethod? Then how can i refer to it?

Matt Ball Over a year ago

Sorry, did you mean callback instead of data? I don't understand.

MBax Over a year ago

Not sure ;) I want a variable in myMethod to equal data. data is a json response and I want to loop through it and compare it to another variable i have set in myMethod. Is that possible?

Matt Ball Over a year ago

Ok. You can't do that, because of the asynchronous execution. Like I said in my answer, everything that relies on data must be passed as a callback (unless you're willing to accept synchronous execution, which you already said you're not). Take the logic that depends on data out of myMethod and put it into the callback function that you pass to getResults.

MBax Over a year ago

Okay. I think i may need to rethink what im trying to achieve here. At least i know the syntax of a callback now ;)

|

Jason Spick · Accepted Answer · 2011-06-14 14:03:14Z

-1

Did you try setting async to false?

myMethod: function() {
   var result = function(callback) {
      this.getResults(params)
   }
},

getResults: function(params) {
   $.ajax({
      type: 'GET',
      async:false, //<-- Here
      url: 'some/url',
      data: params,
      success: function(data) {
         callback(data).call();
      }
   });
}

answered Jun 14, 2011 at 14:03

Jason Spick

6,09814 gold badges43 silver badges62 bronze badges

2 Comments

Felix Kling Over a year ago

but would like to achieve this will a callback function and not set to a synchronous call.... and your code does not work (callback not defined, etc) don't just copy the OP's code.

Matt Ball Over a year ago

The OP specifically said that they do not want to do this.

Collectives™ on Stack Overflow

jquery set ajax result to an outside variable callback

2 Answers 2

6 Comments

2 Comments

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

6 Comments

2 Comments

Related