Как заставить код читать ответ сервера?

Question

[Павел Чуев]

Этот код не работает, т.к по непонятным мне причинам он не читает ответ сервера.

function RequestItems() {
    xhr.onreadystatechange = function() {
        if (xhr.readyState == 4) {
            var data = xhr.responseText;
            data.forEach(function(e){
                var ubid = e.ui_bid;
                +ubid&&my_func(ubid);
            })
        }};
    xhr.open('GET', 'https://csgo.tm/api/Trades/?key=' + secretkey, false);
    xhr.send();
}

А если немного его изменить, то:

function RequestItems() {
    xhr.onreadystatechange = function() {
        if (xhr.readyState == 4) {
            var data = xhr.responseText;
            console.log(data);
        }};
    xhr.open('GET', 'https://csgo.tm/api/Trades/?key=' + secretkey, false);
    xhr.send();
}

в консоль я все таки получаю ответ сервера.
Как изменить код, чтобы первый вариант кода перебирал ответ сервера(массив) и выполнял нужные мне действия с ним?

Answer 1

[Денис]

Ответ с сервера вам приходит в виде строки. Если вам нужно передать массив/объект, то предварительно его нужно преобразовать в строку, а если точнее, то в строку формата JSON. В PHP за это отвечает функция json_encode(). Получив такую json-строку на клиенте, вам надо сделать обратное преобразование, распарсить эту строку:
var data = JSON.parse(xhr.responseText);

Comments

[Павел Чуев]

У меня и без преобразования получалось, но сейчас я получил вот такую ошибку:
TypeError: data.forEach is not a function
Но нужный мне запрос выполнился.

[Денис]

Метод forEach() работает с массивами и ошибка говорит о том, что он у вас применяется не по назначению. Скорее всего, что в data у вас объект. Можете показать ответ сервера?

[Виталий Инчин ☢]

AllDecay: это как раз свидетельствует тому, что у вас не получилось.