using traditional javascript objects in node JS

Question

I am trying my might at doing a NODE JS project, and learning the in's and outs of the Node JS structure. I have looked all over the internet to find out how to do this with little or no luck. The problem is as follows:

I have a class called player, which was created in a file called player.js

var playa = function ()
 {

       this.createPlayer = function(){

       }

 }

In my index.js file I link my player

      var playa = require("./player.js");

however, when I try to reference my player

  io.on('connection', function(socket){
       var player =   new playa();
        player.createPlayer();
});

the console tells me that playa is not a function and the server stops.

Am I even allowed to use traditional OOP JS in node? If I move this class to the index.js file it works splendidly, however it will get sloppy since the class is large and I plan on using at least 5 classes.

2.If it is possible too structure my objects this way, how do I link them/reference them correctly?

if neither or possible, what would be the correct way to structure my objects? I am hoping not to get away from my current convention since that is what I am used to and I need to create multiple instances of player.

I don't think you're properly exporting the symbol from your module. — Pointy
– Pointy, Commented Sep 3, 2014 at 13:43
Currently you do not export anything (or undefined as a default value) from your player.js module. Hence the error. — Sirko
– Sirko, Commented Sep 3, 2014 at 13:45
I tried using module export.. When I did cosnole told me I had a type error and playa was not a function @Sirko — Dnaso
– Dnaso, Commented Sep 3, 2014 at 13:48

Pointy · Accepted Answer · 2014-09-03 13:50:43Z

4

If you export the constructor as a property of module.exports, then you'll have to refer to it by that as a property of the value returned from require():

var playa = require("./player.js");
var player = new playa.playa();

If you want require() to return the function itself, you can write

module.exports = function()
{
   this.createPlayer = function(){

   }
}

answered Sep 3, 2014 at 13:50

Pointy

415k62 gold badges600 silver badges631 bronze badges

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

3 Comments

Dnaso Over a year ago

So first i have to reference the namespace ie the export than the underlying class. Jeez i feel sheepish,. Thank you @Pointy

Pointy Over a year ago

@Dnaso think of it this way: the return value of require() is whatever the value of module.exports ended up being when the imported module runs. It starts off as a simple empty object, but you can make it be whatever you want - in this case, a function.

Dnaso Over a year ago

interesting. I kind of like the way Node makes you structure your scripts.

tier1 · Accepted Answer · 2014-09-03 13:51:34Z

3

Add this to the bottom of your player.js file

module.exports.playa = playa;

This will allow that "class" to be available

EDIT

You will need to create an instance like the following:

var p = require('./player.js')
var player = new p.playa();

edited Sep 3, 2014 at 13:51

answered Sep 3, 2014 at 13:45

tier1

6,4636 gold badges47 silver badges76 bronze badges

3 Comments

Dnaso Over a year ago

var player = new playa(); ^ TypeError: object is not a function

Dnaso Over a year ago

that was the console output, followed by my server crashing

Sirko Over a year ago

@tier1 Currently you have a mistake with upper/lower case letters for the function. You export as lowercase, but use as uppercase.

Collectives™ on Stack Overflow

using traditional javascript objects in node JS

2 Answers 2

3 Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related