asynchrone synchrone attente fin de boucle

Version imprimable

Bonjour à tous,
je m'arrache le peu de cheveux qu'il me reste...
je veux faire un truc tout bête, qui serrais fini en synchrone depuis longtemps mais en asynchrone j'ai du rater un truc.

J'ai fait un exemple simple mais qui reflète déjà bien mon problème.

j'ai une boucle avec un appel externe et à la fin de cette boucle je veux renvoyer le tableau.
Comme vous pouvez le voir sur ce code tout est renvoyé avant que la boucle finisse

Help
j'ai surement du rater un gros truc mais ça alourdi considérablement bien le code le asynchrone, dommage qu'on ne puisse pas activer ou desactiver cette fonctionnalité... :(
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 const Request = require("request"); const pers = ["pascal", "laurent", "marion", "melanie"] const pwd = "https://www.passwordrandom.com/query?command=guid&format=json&count=1" const tab = { "name": "", "pass": "" } var tokenTab = [] function newpass(pe) { return new Promise(function (resolve, reject) { Request(pwd, function (error, response, body) { if (error) { reject(error) } var pw = JSON.parse(body).char[0] resolve(pw) }) }) } async function start() { console.log("start") await step1(); console.log(tokenTab) console.log("end") } function step1() { pers.forEach(el => { //console.log(el) newpass(el).then(z => { //console.log(z) var tt = tab tt.name = el tt.pass = z tokenTab.push(tt) console.log(tt) }) }) } start()

01/04/2020, 10h55
NoSmoking

Bonjour,
regarde du côté de l'utilisation de await qui met l’interpréteur « en pause ».

j'ai déjà essayé mais j'y arrive pas ?
je passe a coté d'un truc...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
 
const Request = require("request");
const pers = ["pascal", "laurent", "marion", "melanie"]
const pwd = "https://www.passwordrandom.com/query?command=guid&format=json&count=1"
const tab = { "name": "", "pass": "" }
var tokenTab = []
 
 
function newpass(pe) {
    return new Promise(function (resolve, reject) {
        Request(pwd, function (error, response, body) {
            if (error) {
                reject(error)
            }
            var pw = JSON.parse(body).char[0]
            resolve(pw)
        })
    })
}
 
async function start() {
    console.log("start")
    var tt = await step1();
    console.log("end")
}
 
function step1() {
    pers.forEach(async (el) => {
        //console.log(el)
        var ret = await newpass(el)
        console.log(ret)
    })
    console.log("fin de boucle")
}
 
start()

L’idée du await rend le code plus simple à comprendre… En théorie. Le problème c’est la méthode .forEach qui ne « comprend » pas quand on lui passe une fonction async. Tu y arriverais avec une boucle for classique, à condition qu’elle soit elle-même dans une fonction async.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 async function step1() { for (const el of pers) { //console.log(el) const z = await newpass(el) //console.log(z) var tt = tab tt.name = el tt.pass = z tokenTab.push(tt) console.log(tt) } }
Mais il y a un inconvénient à cette façon de faire : on perd le parallélisme des requêtes, car elles sont faites les unes à la suite des autres.

Une solution plus efficace consiste à :

lancer toutes les requêtes en même temps dans une boucle synchrone qui n’attend pas les résultats ;
utiliser Promise.all() pour renvoyer le tableau quand toutes les requêtes sont terminées.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 function step1() { const promises = [] pers.forEach(el => { //console.log(el) promises.push( newpass(el).then(z => { //console.log(z) var tt = tab tt.name = el tt.pass = z //tokenTab.push(tt) // tokenTab n’est plus nécessaire console.log(tt) }) ) }) Promise.all(promises).then((tokens) => { console.log(tokens); }); }

Merci Watilin,
j'aime mieux l'idée d'envoyer les requêtes en asynchrone et d'attendre la fin de toutes les promesses.

du coup j'ai fait le script avec ton exemple ça donne ça ...
mais le resutat est surprenant.
le log semble ok, mais le log du tableau des promesses ne fait apparaitre que le meme ????

Code:

1
2
3
4
5
6
7
8
9
10
11
{ name: 'laurent', pass: '3f43385-0cd5-4eac-b57b-17506096fc49' }
{ name: 'marion', pass: 'd9c3eae-9b2b-4fc4-b6dd-47a9b34c1999' }
{ name: 'melanie', pass: '6e14885-e3c3-44c1-9941-81ee7eba874c' }
{ name: 'pascal', pass: 'eca99a8-0670-44f5-b78e-ae24c7c38885' }
 
Resultat du tableau Promise.all ???
 
[ { name: 'pascal', pass: 'eca99a8-0670-44f5-b78e-ae24c7c38885' },
  { name: 'pascal', pass: 'eca99a8-0670-44f5-b78e-ae24c7c38885' },
  { name: 'pascal', pass: 'eca99a8-0670-44f5-b78e-ae24c7c38885' },
  { name: 'pascal', pass: 'eca99a8-0670-44f5-b78e-ae24c7c38885' } ]

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
const pers = ["pascal", "laurent", "marion", "melanie"]
const pwd = "https://www.uuidgenerator.net/api/version4"
const tab = { "name": "", "pass": "" }
var rest = require('rest')
 
async function newpass(per) {
    var pw = await rest(pwd)
    var ret = JSON.parse(JSON.stringify(pw.entity)).substring(1, 36)
    return ret
}
 
function step1() {
    const promises = []
    pers.forEach(el => {
        promises.push(
            newpass(el).then(z => {
                var tt = tab
                tt.name = el
                tt.pass = z
                console.log(tt)
                return tt
            })
        )
    })
 
    Promise.all(promises).then((tokens) => {
        console.log(tokens);
    });
}
 
step1();

02/04/2020, 13h59
NoSmoking

tu utilises un « pointer » sur ton objet tab en écrivant tt = tab, donc à chaque tour tu modifies en fait tab.
Remplace ta ligne :

Code:

var tt = tab

par

Code:

const tt = Object.create(tab)
02/04/2020, 14h10
Troyan

pouaaaaa je m'aperçois que j'ai des mauvais reflexe d'autres langages.
milles mercis je vais pouvoir terminer ce morceau... grace à vous tous.

Merci.