Error when exporting a variable in nodejs

Question

I want to export a variable. but this happens

1st file

const commando = require('discord.js-commando');
const ytdl = require('ytdl-core');


class testCommand extends commando.Command {
    constructor(client) {
        super(client,{
            name: 'test',
            group: 'music',
            memberName: 'test',
            description: 'test',
        });

    }

    async run(message, args) {

        var Testo = 'hello'

    }

}

module.exports.Testo = Testo;
module.exports = testCommand;

2nd file

const commando = require('discord.js-commando');
const ytdl = require('ytdl-core');
var Testotest = require('./test.js')


class pauseCommand extends commando.Command {
    constructor(client) {
        super(client,{
            name: 'pause',
            group: 'music',
            memberName: 'pause',
            description: 'Pause music',
        });

    }


    async run(message, args) {

        message.channel.send(Testotest.Testo())

    }

}

module.exports = pauseCommand;

Error

ReferenceError: Testo is not defined
    at Object.<anonymous> (/Users/andrew/Desktop/NullBot_/commands/music/test.js:27:24)
    at Module._compile (module.js:652:30)
    at Object.Module._extensions..js (module.js:663:10)
    at Module.load (module.js:565:32)
    at tryModuleLoad (module.js:505:12)
    at Function.Module._load (module.js:497:3)
    at Module.require (module.js:596:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/Users/andrew/Desktop/NullBot_/commands/music/pause.js:3:17)
    at Module._compile (module.js:652:30)

Why does it give an error?

looks like you are instantly overriding exports.testo, try changing the last export line to something along these lines module.exports.testCommand = testCommand; — Dale
– Dale, Commented Sep 1, 2018 at 15:03

hong4rc · Accepted Answer · 2018-09-03 10:25:23Z

1

You define Testo in method run if you run method run Testo = 'hello', but you define class testCommand, so Testo is undefined, you shold run method run one time to define Testo.

This code

module.exports.Testo = Testo;

set module.exports = {Testo: Testo}

but you use

module.exports = testCommand;

set module.exports = testCommand

When you call Testotest.Testo is testCommand.Testo (undefined)

change your code in 1st file:

module.exports = testCommand;
module.exports.Testo = Testo;

edited Sep 3, 2018 at 10:25

answered Sep 3, 2018 at 10:06

hong4rc

4,0994 gold badges24 silver badges42 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Fupeng Wang · Accepted Answer · 2018-09-01 15:08:18Z

0

Your mayby gaonna wanna to use module.exports in this way.

module.exports = {
    key1: val1,
    key2: val2
}

So, your code module.exports.Testo = Testo; module.exports = testCommand; can use this format, and It will not throw errors.

answered Sep 1, 2018 at 15:08

Fupeng Wang

942 bronze badges

2 Comments

Codec Over a year ago

what if I need to export a function that has to be run but I also need the function in a variable to be exported to be used in another file?

Fupeng Wang Over a year ago

You can declear testCommand.Testo = Testo, and module.exports = testCommand;

Jose Mato · Accepted Answer · 2018-09-01 16:35:02Z

0

You have this file defined, I understand is test.js:

const commando = require('discord.js-commando');
const ytdl = require('ytdl-core');


class testCommand extends commando.Command {
    constructor(client) {
        super(client,{
            name: 'test',
            group: 'music',
            memberName: 'test',
            description: 'test',
        });

    }

    async run(message, args) {
      var Testo = 'hello'
    }
}

// Testo is not defined because is under method run
module.exports.Testo = Testo;
module.exports = testCommand;

You can see the problem better now because is well indented. This module is loading sync and you are exporting Testo following sync way, so, the error is expected. If you want to fix this you need to define "var Testo" outside of "run" method or make this module async.

Regards

answered Sep 1, 2018 at 16:35

Jose Mato

2,8091 gold badge20 silver badges18 bronze badges

2 Comments

Codec Over a year ago

so is there a way to export a variable in a ‘run’ function?

Codec Over a year ago

without putting the module.exports inside the run function

Collectives™ on Stack Overflow

Error when exporting a variable in nodejs

3 Answers 3

Comments

2 Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

2 Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related