javascript - Calling a Node.js module function from within the module - Stack Overflow

admin管理员组

文章数量:1344973

I'm attempting to write a node module in order to clean up my code and separate it out into different files.

Consider the below code:

module.exports = {
    Hello : function(request, reply) {
        return reply("Hello " + World());
    },
    World : function() {
        return "World";
    }
}

If I import the above module and use the Hello function as handler for a specific route, i get an HTTP 500 internal server error.

I've narrowed the problem down to the call to World(), if I change the Hello function to

Hello : function(request, reply) {
    return reply("Hello World");
}

Then it works fine, so it seems that it is being tripped up when calling another function from within the export object

Does anyone know why this is happening and how to resolve it?

I'm attempting to write a node module in order to clean up my code and separate it out into different files.

Consider the below code:

module.exports = {
    Hello : function(request, reply) {
        return reply("Hello " + World());
    },
    World : function() {
        return "World";
    }
}

If I import the above module and use the Hello function as handler for a specific route, i get an HTTP 500 internal server error.

I've narrowed the problem down to the call to World(), if I change the Hello function to

Hello : function(request, reply) {
    return reply("Hello World");
}

Then it works fine, so it seems that it is being tripped up when calling another function from within the export object

Does anyone know why this is happening and how to resolve it?

• javascript
• node.js
• module
• node-modules

Share Improve this question Follow asked Aug 20, 2017 at 16:09 MichaelMichael 17744 silver badges1111 bronze badges 8

• jsfiddle/rc62tuw8 – Andy Commented Aug 20, 2017 at 16:13
• Stange. It works in fiddle with this. appended, however it my application doesn't seem to like it – Michael Commented Aug 20, 2017 at 16:24
• @Michael: It depends on how Hello is called. If it's not called as method of the exported object, this won't refer to the object. – Felix Kling Commented Aug 20, 2017 at 16:27
• From where exactly do you call your myModule.Hello function..? – Redu Commented Aug 20, 2017 at 16:27
• Its being called from my main app.js file, with the following server.route({ method: 'GET', path: '/user/new/{q}', handler: testy.Hello }); There seems to be something not quite right, however Ahmads answer works fine – Michael Commented Aug 20, 2017 at 16:35

 |  Show 3 more ments

3 Answers 3

Sorted by: Reset to default 5

You should call it as follows:

module.exports = {
    Hello: function(request, reply) {
        return reply("Hello " + module.exports.World());
    },
    World: function() {
        return "World";
    }
}

If you are aiming for cleaner code, I suggest that you change the code to this:

function World() {
  return "World";
}
function Hello(request, reply) {
  return reply("Hello " + World());
}
module.exports = {
    Hello,
}

This will make your code more readable and you will only be exporting what you actually need. This question has other solutions to your issue.

Well let's demonstrate the this

this doesn't define the object that the function resides in. It defines from where the function is called. So while;

var obj = {
    Hello : function(request, reply) {
        return reply("Hello " + this.World());
    },
    World : function() {
        return "World";
    }
};
obj.Hello("test", console.log);

would work just fine; This wouldn't;

var obj = { Hello : function(request, reply) {
                      return reply("Hello " + this.World());
                    },
            World : function() {
                      return "World";
                    }
          };

setTimeout(obj.Hello,100,"test",console.log);

This is just because the obj.Hello will be assigned an argument in setTimeOut function's definition and that argument will be invoked as window being the this for that function. So you should instead do like;

var obj = { Hello : function(request, reply) {
                      return reply("Hello " + this.World());
                    },
            World : function() {
                      return "World";
                    }
          };

setTimeout(obj.Hello.bind(obj),100,"test",console.log);
//or 
setTimeout(obj.Hello.bind(obj,"test",console.log),100);
//or
setTimeout((x,y) => obj.Hello(x,y),100,"test",console.log);

You'll need to add this to your invocation of World() -

module.exports = {
    Hello : function(request, reply) {
        return reply("Hello " + this.World());
    },
    World : function() {
        return "World";
    }
}

World is an attribute of the export object, not a variable within the accessible scope - so you need to specify that the function belongs to this.

本文标签： javascriptCalling a Nodejs module function from within the moduleStack Overflow