Я посмотрел на эти вопросы, но я все еще застрял на этом Замыкание внутри цикла в JavaScript - простой практический пример
Вот мой код
template[prop['Name']] = [];
var processedCounter = -1;
$.each(prop.Properties, function (key, value) {
console.info(processedCounter + 'outside');
$.getJSON(value['Url']).done(function (jsres) {
console.info(processedCounter + 'inside');
var numItems = jsres[itemCount].length;
if (template[prop['Name']].length == 0) {
console.info('this should only be printed once');
for (var i = 0; i < numItems; i++) {
template[prop['Name']].push({});
}
}
processedCounter += 1;
});
});
Есть несколько проблем. Во-первых, он два раза печатает сообщение «это следует печатать только один раз». Во-вторых, значение processingCounter должно быть -1 вместо 0, потому что значение увеличивается не в то время.
Сообщение, которое будет напечатано только один раз, будет печататься один раз для каждого вызова getJSON (), который вы делаете, что должно быть равно количеству props в prop.Properties. И счетчик должен дважды регистрировать -1 outside. Это хороший пример того, почему я использую Promise.all () в сочетании с urls.map (fetch (url)), чтобы избежать подобных ситуаций.
Я закончил рефакторинг кода, чтобы все было по-другому, и он заработал.



![Безумие обратных вызовов в javascript [JS]](https://i.imgur.com/WsjO6zJb.png)


Проблема здесь, скорее всего, в состоянии гонки. Каждый из вас запускает несколько запросов getJSON очень быстро (javascript работает очень быстро). Таким образом, в зависимости от того, насколько быстро эти запросы завершаются, возможно, что оба они могут выполнять свою логику выполнения и пройти условное if до того, как кто-либо из них обновит элемент шаблона.