Не могу получить массив jquery из json

Question

Получаю данные из php

let pars,  lastMonth = [];
$.post('data.php',data => {
pars = $.parseJSON(data);
for(let prop in pars)
lastMonth.push(pars[prop]);
});

Через консоль смотрю

console.log(lastMonth);

Вроде как массив НО если сделать так

console.log(lastMonth[0]);

то undefined.

Что делаю не так?

[![фото2][2]][2]

«если сделать так» — сделать где и когда? Это важно, ведь массив заполняется не мгновенно — andreymal, Jan 07 '22 at 11:33
Массив заполняется, я же не писал, что он НЕ ЗАПОЛНЯЕТСЯ, еще раз, если console.log(lastMonth); то я вижу массив НО если я обращаюсь по индексу console.log(lastMonth[0]); то undefined — spoon 100500, Jan 07 '22 at 11:49
Ещё раз: где и когда? Покажите код, в котором вы делаете console.log; если в браузерной консоли — покажите скриншот вашего ввода — andreymal, Jan 07 '22 at 11:50
Если пытаться запустить близкий к вашему код целиком в консоли, то всё работает нормально, lastMonth[0] успешно выводит значение из массива, ваша проблема не воспроизводится https://i.stack.imgur.com/7zgdo.png — andreymal, Jan 07 '22 at 12:00
А теперь попробуйте сделать console.log(JSON.stringify(lastMonth)) и вы увидите, что массив на самом деле НЕ ЗАПОЛНЯЕТСЯ, потому что вы вызываете console.log слишком рано и не дожидаетесь момента, когда он заполнится. — andreymal, Jan 07 '22 at 12:04
Объяснение того, почему без JSON.stringify массив выглядит якобы заполненым, есть здесь: Странное поведение Javascript при передаче объекта по ссылке — andreymal, Jan 07 '22 at 12:07
О том, как всё-таки дождаться заполнения массива: Как вернуть значение из события или из функции обратного вызова? Или хотя бы дождаться их окончания — andreymal, Jan 07 '22 at 12:07
И да, ещё рекомендую для наглядности воспользоваться отладчиком и поставить точку останова на этом console.log — он приостановит выполнение страницы, и тогда даже без JSON.stringify вы в отладчике сможете увидеть, что массив на самом деле ещё не успел заполниться — andreymal, Jan 07 '22 at 12:21

Oleh Pohorilyy · Accepted Answer · 2022-01-07T12:56:32.217

1

Массив это ссылочный тип, выводя его в консоль он при открытии будет иметь актуальные данные, то есть, пока ты выводил пустой массив и нажимал на кнопку раскрыть, он успел его заполнить и отобразить то что в нем находится на момент открытия а не на момент выполнения. Для наглядности выведи массив через JSON.stringify.

А решение собственно это - дождаться выполнения асинхронного кода и потом оперировать полученным значением. Например так

async function doRequest() {
    const months = await new Promise(resolve => {
        $.post('data.php',data => {
            const lastMonth = [];
            pars = $.parseJSON(data);
        for(let prop in pars)
            lastMonth.push(pars[prop]);

        resolve(lastMonth);
    });
});

console.log(months);

}
doRequest();

edited Jan 07 '22 at 12:56

answered Jan 07 '22 at 12:35

Oleh Pohorilyy

26

Пробывал так, не работает let lastMonth = []; fetch('data.php') .then(response => response.json()) .then(pars => { for(let prop in pars){lastMonth.push(pars[prop])}; }); console.log(lastMonth[0]); – spoon 100500 Jan 07 '22 at 12:49
Ваш код выдаёт ошибку lastMonth is not defined – spoon 100500 Jan 07 '22 at 12:50
@spoon100500 в этом вашем пробовании та же самая ошибка что и с jquery — вы выполняете console.log слишком рано и не пытаетесь дождаться момента, когда fetch примет ответ на запрос и заполнит массив – andreymal Jan 07 '22 at 12:51
Ваш код выдаёт ошибку lastMonth is not defined? – spoon 100500 Jan 07 '22 at 12:52
@spoon100500 обновил ответ – Oleh Pohorilyy Jan 07 '22 at 12:57
А разве then() не выполняет ожидание? – spoon 100500 Jan 07 '22 at 13:03
1

@spoon100500 нет, then просто добавляет функцию-обработчик и мгновенно завершает работу, ничего не ожидая – andreymal Jan 07 '22 at 13:05
1

then выполнит ожидание текущего промиса, но не текущей строки кода, и передаст значение в лямбду, то есть, запрос запускается параллельно выполнению твоего кода, пока запрос идет, интерпретатор уже переходит к строке console.log, и уже потом учитывая что запрос выполняется дольше, сначала произойдет вывод в консоль а потом запись в lastMonths. – Oleh Pohorilyy Jan 07 '22 at 13:09
Спасибо, ох уж эти промисы)) – spoon 100500 Jan 07 '22 at 13:15
months находится внутри doRequest() т.е. doRequest() это набор промисов, как присвоить результат переменной? Добавил фото3. – spoon 100500 Jan 07 '22 at 14:27
1

@spoon100500 снаружи doRequest — никак, в javascript это принципиально невозможно – andreymal Jan 07 '22 at 14:38
...ну или разве что top-level await использовать, если он поддерживается во всех нужных браузерах – andreymal Jan 07 '22 at 14:41

Не могу получить массив jquery из json

1 Answers1