Update 4-error-fstat.js

[kzemlyak]

• Uses more undrestandable variable names
• Uses for loop instead of forEach

[tshemsedinov]

• Use camelCase: fileName
• Here is a case for .map

[kzemlyak]

• Here is a case for .map

But if we use map in this case, wouldn't it be semantically incorrect because there will be mutation of the external variable (stats) inside map, while map is intended for immutable behavior?

[tshemsedinov]

But if we use map in this case, wouldn't it be semantically incorrect because there will be mutation of the external variable (stats) inside map, while map is intended for immutable behavior?

You can easily do it without mutations. Reflect array of names to array of stats, we do not even need counters, printResult, and isLastProcessed here.

[kzemlyak]

But for that, I will need to use Promises to do this. Can I use them in this example?

[tshemsedinov]

constpromises=files.map((fileName)=>fs.lstat(fileName));Promise.allSettled(promises).then((stats)=>{console.dir(stats);});

[kzemlyak]

I just thought that it's not worth using Promises in this example since they were not originally included

[tshemsedinov]

We can refactor this even without promises, using closures, I'll provide multiple solutions a bit later