Как правильно писать код под node js?

Question

[IvankoPo]

let obj;
fs.readFile("config.json", (err, data) => {
    if (err){
        console.log(err);
    } else {
        obj = JSON.Parse(data);
    }
});
console.log(obj);

Проблема в том что obj будет undefined, но мне нужно как то дальше работать с этой переменной, и не хочется писать огромную логику в ветке else коллбэка, как эта проблема решается ? Sync использовать нельзя.

Comments

[Сергей Горностаев]

Использования глобальных переменных стоит вообще избегать, а в конкурентном коде особенно.

Answer 1

[Дмитрий Беляев]

new Promise((resolve, reject) => {
  fs.readFile("config.json", (err, data) => {
    if(err) {
      return reject(err);
    }
    resolve(JSON.Parse(data));
  });
}).then(obj => {
  console.log(obj);
  // your code
}).catch(console.error);

Comments

[RidgeA]

const {promisify} = require('util');
const fs = require('fs')
const readFileP = promisify(fs.readFile)

readFileP(/*...*/)
.then(/* ... */)
.catch(/* ... */)

[Дмитрий Беляев]

RidgeA, только стоит оговорится, node 8+, а то мало ли
кроме того для node 10+ есть еще require('fs').promises

const fsP = require('fs').promises;

fsP.readFile(/*...*/)
.then(/* ... */)
.catch(/* ... */)

[RidgeA]

Дмитрий Беляев, согласен, но 8 - уже давно lts, а 10 еще нет :-)

Answer 2

[RidgeA]

async/await, promise, callback

Answer 3

[Interface]

Чтобы не делать "огромную логику в ветке else коллбэка" можно использовать early-return https://forum.freecodecamp.org/t/the-return-early-...

Чтобы не делать большую вложенность и callback-hell нужно проделать путь: callback -> promise -> async/await

Ваш код станет примерно таким

async function foo() {
    try {
        const content = await fs.readFile("config.json", "utf-8");
        const obj = JSON.parse(content);
        console.log(obj);
    } catch(error) {
        console.error(error);
    };
}

При этом функция останется ассинхронной и не будет блокировать поток.

Comments

[Виктор П.]

что-то тут не сходится

const content = await fs.readFile("config.json");
const obj = JSON.parse(data);

читаете в переменную content, а парсите несуществующую data

[Interface]

Виктор П., опечатка, конечно

Answer 4

[Lynn «Кофеман»]

Конкретно тут проще воспользоваться синхронным чтением файла, а учитывая, что это вообще json, let obj = require('./config.json');