Модуль возвращает undefined, почему?

Question

[Ярослав Корнилов]

Есть мой модуль

const fs = require('fs')

const getDataObj = () => {
  const regex = /<email>(\D+?)<\/email>/g;
  fs.readFile('uploads/dataset.xml', "utf-8", function(err, data) {
    let m;
    let result = "";
    while ((m = regex.exec(data)) !== null) {
      if (m.index === regex.lastIndex) {
        regex.lastIndex++;
      }
      m.forEach((match, groupIndex) => {
        if (groupIndex === 1) {
          result += match + '\n'
        }
      });
    }
    return result
  })
}

exports.getDataObj = getDataObj;

При вычислении функции он возвращает нужный результат. Но при вызове этой же функции, импортированной в файл server.js возвращается undefined. В чем может быть дело?

const express = require('express')
const pug = require('pug');
const path = require('path');
const parser = require('./parser');

const app = express()

app.set('view engine', 'pug')
app.use('/static', express.static(__dirname + '/static'))

const multer = require('multer')
var upload = multer({
  dest: 'uploads/'
})

app.get('/', function(req, res) {
  console.log(parser.getDataObj());
  res.render(path.join(__dirname, 'views/index.pug'), {
    pageTitle: 'Hey',
    message: parser.getDataObj()
  })
})

app.listen(3000);

Comments

[alex]

потому что асинхронность

Answer 1

[Дмитрий Лузанов]

fs.readFile работает асинхронно. Передавайте в getDataObj callback или юзайте промисы.
Примерно так:

const getDataObj = (cb) => {
  const regex = /<email>(\D+?)<\/email>/g;
  fs.readFile('uploads/dataset.xml', "utf-8", function(err, data) {
    let m;
    let result = "";
    while ((m = regex.exec(data)) !== null) {
      if (m.index === regex.lastIndex) {
        regex.lastIndex++;
      }
      m.forEach((match, groupIndex) => {
        if (groupIndex === 1) {
          result += match + '\n'
        }
      });
    }
    return cb(result)
  })
}

app.get('/', function(req, res) {
  parser.getDataObj((message) => {
    res.render(path.join(__dirname, 'views/index.pug'), {
      pageTitle: 'Hey',
      message: message
    });
  });
});

Comments

[Ярослав Корнилов]

все равно не работает
и я возвращал же значение внутри callback а, т.е. он сначала читал файл, а потом уже возвращал результат

[Ярослав Корнилов]

насколько плохо readFileSync тут применить, не будет ли это костылем?

[Дмитрий Лузанов]

Ярослав Корнилов: решать только вам.

Что именно с коллбэком не работает?

[Ярослав Корнилов]

Predve4niy: ошибка cb is not function

[Дмитрий Лузанов]

Ярослав Корнилов: а вызываете вы как? Вот енто parser.getDataObj
Там UPD в ответе.

[Ярослав Корнилов]

загрузил как вы посовтеовали в качестве примера

и я возвращал же значение внутри callback а, т.е. он сначала читал файл, а потом уже возвращал результат

все же не понимаю почему не так работает)

а с readFileSync все хорошо
думаю так и оставлю, потому что это единственное IO в данном случае, но хочу разобраться все же почему возникла ошибка

[Дмитрий Лузанов]

Ярослав Корнилов:

parser.getDataObj((message) => {
    res.render(path.join(__dirname, 'views/index.pug'), {
      pageTitle: 'Hey',
      message: message
    });
  });

[Дмитрий Лузанов]

Ярослав Корнилов: readFileSync в любом случае не гуд

[Ярослав Корнилов]

Predve4niy: а почему? (знаю что блокирующая операция, но это же не абсолютное зло)

[Дмитрий Лузанов]

Ярослав Корнилов: https://learn.javascript.ru/screencast/nodejs тут ответы на все ваши вопросы

[Ярослав Корнилов]

Predve4niy: я смотрел, но уже давно, сейчас вот все восстанавливаю в памяти
кстати, понял как сделать с колбэками и с промисами тоже
p.s спасибо большое)