Vänta på forEach (Javascript)

Varför göra det så komplicerat? En vanlig for loop är inte asynk.

for(var i = 0; i < newArr.length; i++)
        this.updateData(element);

Men vem vet, jag kanske är för old school och vill ha enklaste lösning

.forEach är inte async, allt som händer i den kommer hända innan någon rad efter exekverar.

const numbers = [1, 2, 3, 4, 5];
numbers.forEach((number) => console.log(number));
console.log("done");

Kodsnutten kommer alltid skriva
1
2
3
4
5
done

Du sparar datan fel och läser nog in något annat eller så. Eller så skriver du över den någon annan stans

jag tyckte det var skumt att forEach var async, men jag brukar inte använda den pga kompatibilitet. Så jag hade egentligen ingen koll på just den, tyckte bara föreslagna förslag var onödigt komplicerade

@we_man1: vad för updateData ? Gör den något async?

const numbers = [1, 2, 3, 4, 5];
numbers.forEach((number) => console.log(number));
console.log("done");

Skickades från m.sweclockers.com

@we_man1: Yes det har jag

Såhär brukar jag göra, it ain't pretty but it works

Har bara skrivit den här koden på frihand, men det är bara för att du ska få lite koll på logiken. Så får du skriva om den själv. Det går självklart bra att använda promises, det är väl egentligen att föredra, men jag kan inte den syntaxen i huvudet...

function fetchData(onDone){
    //Hämta data och när den är hämtad så kör functionen nedan. Det går självklart att använda promises istället, det här är bara för att jag ska skriva det lite snabbare :)
    if(typeof onDone === "function")
        onDone(result)
}

var arr = [1,2,3,4],
    fetchCounter = arr.length; //håll koll på hur många requests som kommer att göras

for(var i = 0; i < arr.length; i++) {
    fetchData(arr[i], function(res){
        fetchCounter--;//här har vi fått svar på en request, så då drar vi bort en från vår variabel som håller koll på det
        if(fetchCounter < 1) { //Här har alla requests fått svar och är behandlade
            localStorage.setItem('blah');
        }
    });
}

Vad sägs om att göra en funktion som heter updateDataArray() som tar hela arrayen av data istället för bara ett element i taget, sedan när den är klar returnerar den en Promise. Då behöver du bara hantera det såhär:

const oldArr = JSON.parse(localStorage.getItem('theData'))

updateDataArray(oldArr).then((resultArr) => {
	localStorage.setItem('theData', JSON.stringify(resultArr));
}.catch((err) => {
	// Något blev fel
});

Kanske sent svar men det låter som du behöver async/await, tror det kom med es6. Kolla gärna vidare då det hanterar asynchrona kallelser utan att behöva egna callbacks. Jag har använt det vis utveckling av react-native när mobilen har behövt accessa localstorage

Skickades från m.sweclockers.com

Async är dock en del av nästa standard vill jag minnas. I så fall får man använda något som Babel.

Här är en enkel json fetch där du låter två separata promises köras och returnerar den som blir klar först (antingen får du json från servern eller så körs en timeouts om det tog för lång tid).

fetchJSON(server, timeout)
{
	let timer = new Promise((resolve, reject) => {
	  setTimeout(reject, timeout, 'request timed out');
	});
	let f = new Promise((resolve, reject) => {
	  fetch(server, {
			method: 'GET',
			dataType: 'json',
			headers: {
			  'Accept': 'application/json',
			  'Content-Type': 'application/json',
			}})
		.then(response => response.json())
		.then(json => resolve(json))
		.catch(reject)
	});
	return Promise
	  .race([timer, f])
	  .then(json => this._successConnection(json))
	  .catch(err => this._errorConnection(err));
}

_successConnection(json)
{
  return json;
}

_errorConnection(err)
{
  return false;
}

Är medveten eftersom det var Promise jag nämnde och kom med exempel på i posten innan. Svarade bara på att Async som den andra skribenten nämnde (som antagligen kommer ersätta Promise i många fall) kräver Babel just nu.

const KEY = 'theData';

function updateData(data) {
    return data; // identity update
}

const oldData = JSON.parse(localStorage.getItem(KEY));
const newData = oldData.map(updateData);
localStorage.setItem(KEY, JSON.stringify(newData));

Det här känns som den bästa lösningen.