0

I know this question hast probably been asked a thousand times, but i cannot seem to find the answer. I want result to be the data returned from the ajax-request, which should be a json-data array (the result of console.log(data)).

  var result = $.ajax({
    type: 'GET',
    url: dataPath,
    dataType: 'json',
    success: function(data) {
      console.log(data)
      },
    error: function(){
      //alert("damn");  
      },
    data: {},
    aync: false
  });

  console.log(result);

However, console.log(result); will return some strange object, which I don't know how to handle. Why isn't result = data ?

Comfort Eagle
  • 2,112
  • 2
  • 22
  • 44

5 Answers5

1

Typo.

Change this:

aync: false

to:

async: false

And the ajax method still returns the jqXHR object doing the request, not the result. Use the data parameter in the success call and store it somewhere.

Guffa
  • 687,336
  • 108
  • 737
  • 1,005
  • Why the downvote? If you don't explain what you think is wrong, it can't improve the answer. – Guffa Sep 30 '12 at 11:00
  • 1
    I didn't downvote but I think this isn't *really* helping OP. `aync: false` is a typo bug, but `async: false` is generally a conception bug. – Denys Séguret Sep 30 '12 at 11:23
  • @dystroy :It's *really* what's wrong with the code. Whether the method is good or not, that's another matter. – Guffa Sep 30 '12 at 11:48
1

First of all remove the aync: false from your code. It should be spelled async: false but you don't need it to achieve your goal and what it actually does is block the entire browser's user interface resulting in a terrible user experience. Remember that "A" in AJAX means Asynchronous.

The result of an $.ajax() call is a promise which is not the same as your data, but it can still be useful to get to your data. You just need to use it in a certain way.

Try changing:

 console.log(result);

to:

result.done(function (data) { 
    console.log(data);
});

or:

result.done(function (data) { 
    console.dir(data);
});

or even this might work - untested:

result.done(console.dir);

See this answer for a better explanation.

Community
  • 1
  • 1
rsp
  • 107,747
  • 29
  • 201
  • 177
  • No, the result isn't a promise. The OP is trying to do a synchronous call, not an asynchronous call. – Guffa Sep 30 '12 at 11:01
  • 1
    @Guffa In the case of an synchronous call, that promise will be already resolved, but it will still be a promise. Using this promise is marked as _deprecated_ in the docs, but apparently it is still the case. – lanzz Sep 30 '12 at 11:05
  • 2
    Blocking the entire browser isn't a smart thing to do in this case. It's much better to use asynchronous call with a promise. My advice is to **NEVER** use **synchronous** AJAX (SJAX?) **EVER**. – rsp Sep 30 '12 at 11:07
  • @rsp: Even if it's not the best thing to do, that is what the op *is* trying to do. – Guffa Sep 30 '12 at 11:11
  • 1
    @Guffa yes which is clearly the *wrong* way to do it and our responsibility is to help with better ways. – rsp Sep 30 '12 at 11:15
  • @rsp: Yes, but first you should point to what is wrong with *the code*, then you can talk about the limitations and drawbacks with the method. – Guffa Sep 30 '12 at 11:19
  • 1
    @Guffa That's true that I should do that and incidentally that's what I did. :) – rsp Sep 30 '12 at 11:23
  • @rsp: Oh, you added that too. – Guffa Sep 30 '12 at 11:49
0

Initialize result inside success function.

 var result;
  $.ajax({
        type: 'GET',
        url: dataPath,
        dataType: 'json',
        success: function(data) {
          result = data;
          console.log(data)
          },
        error: function(){
          //alert("damn");  
          },
        data: {},
        async: false
      });

      console.log(result);
Anoop
  • 23,044
  • 10
  • 62
  • 76
0

You mentioned console.log(result) returns a strange object, actually this strange object is known as xhr (XMLHttpRequest) object.

Since the call is syncronous because of async: false so it's easy to get the returned data like

var result = $.ajax({...}); // get the xhr object in to result
console.log(result.responseText); // xhr object has a "responseText" property

As because result.responseText will be available only after the request completes and there is no chance to execute this console.log(result.responseText); because of async:false, before the request completes because a syncronous ajax request hangs on everything before it finish the request.

In your success callback data will be an object because of dataType: 'json' but outside of success callback, i.e. console.log(result.responseText); will be only text so to use it as an object you have to convert it to an object using $.parseJSON(result.responseText).

The Alpha
  • 143,660
  • 29
  • 287
  • 307
0

There is a small spelling mistake aync: false should read async: false assuming of course that you require the request to run synchronously i.e. for the remainder of your code to wait for this result.

I think that the main issue here is that the result you are trying to output to the console is not being referenced by the ajax request.

It is entirely your choice how you reference the data returned by the ajax request, you chose the word data this could just as easily have been result or json_Data or return_Data or ....

Hence to send the result of the ajax request to the console I would suggest:

var result = $.ajax({
    type: 'GET',
    url: dataPath,
    dataType: 'json',
    success: function(result) {
      console.log(result)
      },
    error: function(){
      //alert("damn");  
      },
    data: {},
    async: false
  });
codepuppy
  • 1,130
  • 2
  • 15
  • 25