Warten, bis alle Versprechen gelöst sind, während ich rekursiv ein Verzeichnis durchläuft

Question

Ich versuche rekursiv einen Verzeichnisbaum zu gehen und alle Dateinamen zu sammeln. Im folgenden Code, jedoch läuft der console.log() am Ende zu früh warten, nicht für die von der Funktion des rekursiven cacheDir Aufruf erzeugt Versprechen zu beheben:

function cacheDir(dir) { 
    return fs.readdir(dir).then(items => { 
    const readDirPromises = [] 

    for (let item of items) { 
     const fullPath = path.join(dir, item) 
     const lstat = fs.lstat(fullPath) 
     readDirPromises.push(lstat) 

     lstat.then(stats => { 
     if (stats.isFile()) { 
      return fullPath 
     } else { 
      return cacheDir(fullPath) 
     } 
     }) 
    } 

    return Promise.all(readDirPromises) 
    }) 
} 

cacheDir("../someDir/") 
    .then(results => console.log('done!', results)) 

Was mache ich falsch? Es scheint mir, dass readDirPromises wird ein Array von strings und verspricht, jedes Versprechen von denen wird sich zu einer Reihe von Strings und verspricht zu lösen.

Answer 1

Das Versprechen, auf das Sie mit Promise.all() warten müssen, muss lstat.then() sein, nicht nur lstat. Denken Sie daran, dass lstat.then() eine neue Zusage zurückgibt und diese neue Zusage alle Versprechungen nachverfolgt, die innerhalb des -Handlers zurückgegeben werden, den Sie aufgrund Ihrer Rekursion benötigen.

Hier ist eine Version, die das tut (und wird von einigen temporären Variablen zu befreien):

function cacheDir(dir) { 
    return fs.readdir(dir).then(items => { 
    const readDirPromises = []; 

    for (let item of items) { 
     const fullPath = path.join(dir, item); 

     readDirPromises.push(fs.lstat(fullPath).then(stats => { 
     if (stats.isFile()) { 
      return fullPath; 
     } else { 
      return cacheDir(fullPath); 
     } 
     })); 
    } 

    return Promise.all(readDirPromises) 
    }) 
}