4

Estou em um projeto utilizando NodeJS, e seu framework Express.

No arquivo principal app.js eu chamo uma função com o objetivo de ler um arquivo, e atribuo a uma variável global:

global.google_sheet_credentials = readFileCredentials("spread_sheet.txt");

O código da função readFileCredencials é:

readFileCredentials = function(file){
    fs.readFile('data-source/credentials/'+file, 'utf8', function(err, data){
        if(err){
            console.log("Could not open file: %s", err);
        }else{
            console.log(data);
            return data;
        }
    });
};

module.exports = readFileCredentials;

Após isso, eu redireciono para outro arquivo:

rek('dashboards/ti/main_it');

Neste arquivo main_it.js, eu tento utilizar essa variável global google_sheet_credentials:

console.log(google_sheet_credentials);

porém avisa que ela está indefinida, alguém pode me explicar o motivo, e uma forma para que eu consiga o resultado certo?

5
  • 1
    Você pode criar um modulo para ler o .txt e importar ele no arquivo que você deseja. Commented 27/04/2016 às 12:53
  • Esse readFileCredentials é síncrono?
    – Sergio
    Commented 27/04/2016 às 13:02
  • Ele é uma função que recebe o nome do arquivo, e retorna o conteudo desse arquivo, não sei lhe dizer se é sincrono ou assincrono.
    – Jonathan
    Commented 27/04/2016 às 13:06
  • Podes colocar o código dessa função aqui?
    – Sergio
    Commented 27/04/2016 às 13:10
  • Coloquei o código na pergunta.
    – Jonathan
    Commented 27/04/2016 às 13:20

2 Respostas 2

4

Essa função é assíncrona, ou seja esse return não é usado da maneira que pensas.

Tens de usar uma callback (ou via Promise como foi sugerido noutra resposta). Assim, só podes contar com o valor setado nesse global.google_sheet_credentials quando a callback for chamada.

Uma sugestão seria assim:

readFileCredentials = function(file, cb){
    var _path = 'data-source/credentials/' + file;
    fs.readFile(_path, 'utf8', function(err, data){
        if(err){
            console.log("Could not open file: %s", err);
        }
        cb(err, data); 
    });
};

module.exports = readFileCredentials;

e depois usares assim:

readFileCredentials("spread_sheet.txt", function(err, data){
    global.google_sheet_credentials = data;
    // agora a variável está setada. Se precisares de correr outro código tens de o ter aqui dentro, ou chamando funções a partir daqui
});
9
  • Obrigado pela resposta, mas infelizmente não funciono. Eu criei um repositório no git, e importei a parte que está dando o problema, para ficar melhor de visualizar: bitbucket.org/JGonczoroski/teste . Quando puder dar uma olhada
    – Jonathan
    Commented 27/04/2016 às 14:22
  • @JonathanGonczoroski onde estás a precisar de google_sheet_credentials? não vejo isso no teu código
    – Sergio
    Commented 27/04/2016 às 15:12
  • No arquivo que se encontra: dashboards/implantation/pushers/teste.js , la encontra-se um print dessa variável
    – Jonathan
    Commented 27/04/2016 às 15:15
  • @JonathanGonczoroski porque não usas a lógica normal de fazer requires nos ficheiros em vêz de estares a usar o espaço global? Isso foi uma das grandes novidades e vantagens do Node.js...
    – Sergio
    Commented 27/04/2016 às 15:19
  • 1
    @JonathanGonczoroski não. Cada ficheiro importa o que precisa. Nalguns casos faço require de exatamente a mesma coisa em ficheiros diferentes. A regra é fazer require dos ficheiros/modulos que esse ficheiro precisa.
    – Sergio
    Commented 27/04/2016 às 15:44
3

Claramente o problema do seu código é com assincronismo. Promises podem resolver o seu problema:

function readFileCredentials(file) {
    return new Promise(function(resolve, reject) {
        fs.readFile('data-source/credentials/' + file, 'utf8', function(err, data){
            if (err) {
                console.log("Could not open file: %s", err);
                reject(err);
            } else {
                resolve(data);
            }
        });
    });
};

module.exports = readFileCredentials;

Na chamada, basta usar o then para dar continuidade no processo da Promise:

readFileCredentials("spread_sheet.txt").then(function(googleSheetCredentials) {
    global.google_sheet_credentials = googleSheetCredentials;
});

Atualização:

Conforme o link que foi passado, tente modificar o seu código App para funcionar da seguinte maneira:

var App = (function(){

    // Method Construct, your objetive is loading modules and utils that will be used in this project.
    function App() {
        this.define_global_utils(function() {
          this.define_global_modules();
        }.bind(this));

        this.init();
    }

    // Method responsible for initiating the application.
    App.prototype.init = function(){
        this.load_modules();
    };

    // Method responsible for importing the modules used in this project.
    // The modules were imported globally, that is, can be used in anywhere in the code.
    App.prototype.define_global_modules = function() {
        global.rek = require('rekuire');
        global.fs = require('fs');
    };

    // Method responsible for imported the utilities.
    App.prototype.define_global_utils = function(callback){
        global.readFileCredentials = rek('data-source/utils/readFileCredentials');

        readFileCredentials("spread_sheet.txt", function(err, data){
            global.google_sheet_credentials = data;
            callback();
            // agora a variável está setada. Se precisares de correr outro código tens de o ter aqui dentro, ou chamando funções a partir daqui
        });
    };

    // Method responsible for loading the dashboards.
    App.prototype.load_modules = function(){
        rek('dashboards/implantation/main_implantation');
    };

    return App;
})();
1
  • Obrigado pela resposta, mas infelizmente não funciono. Eu criei um repositório no git, e importei a parte que está dando o problema, para ficar melhor de visualizar: bitbucket.org/JGonczoroski/teste . Quando puder dar uma olhada. O código está com a solução do Sérgio, porém eu também testei a sua, e também não deu certo. Provavelmente é algo relacionado ao meu código, se puder me ajudar agradeço
    – Jonathan
    Commented 27/04/2016 às 14:24

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .