Can't create a server in an object node.js

Question

I'm trying to put my server with its variables inside an object in order to keep my project clean, so I've created core.js file, which holds all server variables and looks like this:

module.exports =
{
    express: require('express'),
    http: require("http"),
    app: this.express(),
    server: this.http.createServer(this.app),
    io: require('socket.io').listen(this.server),

    runServer: function()
    {
        ...........
    }
};

I require this file inside my main file:

var core = require("./server/classes/core");

And when I try to run I get this error:

app: this.express(),

          ^

TypeError: this.express is not a function

What could possibly be the problem?

Aruna · Accepted Answer · 2016-11-06 20:12:34Z

1

When you initialize the json as above in your code, they all execute at the same time and not one by one since they all belong to single line of execution.

Hence, all the below lines will be executed at the same time and not one by one.

{
    express: require('express'),
    http: require("http"),
    app: this.express(),
    server: this.http.createServer(this.app),
    io: require('socket.io').listen(this.server),

    runServer: function()
    {
        ...........
    }
};

So, at the time of execution of the line app: this.express(), this.express would be undefined.

Better you can define them in a function and assign to module.exports as below.

function Core() {
   this.express = require('express');
   this.http = require("http");
   this.app = this.express();
   this.server = this.http.createServer(this.app);
   this.io = require('socket.io').listen(this.server);

   .....
}

module.exports = new Core();

Another alternative is assigning them directly to the exports as below.

var express = require('express');
var http = require("http");
var app = express();
var server = http.createServer(app);
var io = require('socket.io').listen(server);

exports.express = express;
exports.http = http;
exports.app = app;
exports.server = server;
exports.io = io;

answered Nov 6, 2016 at 20:12

Aruna

12k3 gold badges31 silver badges42 bronze badges

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

2 Comments

Donatas Over a year ago

Everything worked out on both ways, though I like first one more. Didn't knew that everything is being defined at the same time in objects...

Aruna Over a year ago

@Donatas, great mate and just upvote it if you like it :-)

Collectives™ on Stack Overflow

Can't create a server in an object node.js

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related