Accès au nom du fichier dans fs.readFile

Bonjour,

Je cherche à réutiliser le nom du fichier retourné dans l'événement d'un fs.readFile().

Dans le code ci-dessous, la trace m'indique toujours le nom du dernier fichier pour la variable file...
Savez-vous pourquoi ? Et comment corriger le problème ?

Mon objectif est de scanner un répertoire, de récupérer des données dans les fichiers trouvés et d'écrire ces données et le nom du fichier dans un nouveau fichier.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

var fs = require('fs'); var path = require('path');   var dir = path.join(__dirname, 'files');   fs.readdir(dir, function(err, items) { if (err) throw err; for (var i=0; i<items.length; i++) { var file = path.join(dir, items[i]);   fs.readFile(file, 'utf8', function(err, data) { if (err) throw err; console.log(path.basename(file)); }); } });

---

Pour un dossier contenant : file1.txt, file2.txt et file3.txt
La trace sera

Code:

---

1 2 3

file3.txt file3.txt file3.txt

---

Merci pour votre aide.

La solution la plus simple est d’utiliser let ou const à la place de var.

L’explication est que var ne crée pas de portée spécifique à la boucle for.
Pour chaque appel à ta fonction de rappel function(err, items) { ... } il ne se crée qu’une seule fermeture (closure), et donc il n’y a qu’une seule version des variables i et file pour toutes les itérations de la boucle for. C’est un peu le même problème qu’exposé dans la FAQ Comment attribuer à des objets un onclick faisant appel à une variable de boucle ?

D’autres explications sur la page MDN de let. (const a les mêmes règles de portée que let.)

Super ! Merci pour ces infos Watilin !

Cette version fonctionne :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

var fs = require('fs'); var path = require('path');   const dir = path.join(__dirname, 'files');   fs.readdir(dir, function(err, items) { if (err) throw err; for (let i=0; i<items.length; i++) { let file = path.join(dir, items[i]);   fs.readFile(file, 'utf8', function(err, data) { if (err) throw err; console.log(path.basename(file)); }); } });

---