Discussion :

[Tillt]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
file3.txt
file3.txt
file3.txt

Merci pour votre aide.

[Watilin]

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.)

[Tillt]

Super ! Merci pour ces infos Watilin !

Cette version fonctionne :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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));
    });
  }
});