Where are constructors?

[piecioshka]

Where are constructors?

[robreczarek]

@piecioshka All three objects are now present. Is there something that should be changed in the animal object?

[piecioshka]

Constructor in JS are:

function Animal() {
  //...
}
Animal.prototype.xxx = function () {
  // ...
}

😄

[robreczarek]

@piecioshka I have revised the code, but am running into issues. Can you please take a look at commit #7

My intent was to mix things up to have properties that are assigned during object instantiation, ones that are private to the internal object, and ones that are exposed via "public" properties.

I'm not sure why I'm not able to push animal objects via the addAnimal method. Also, why do the getGroup and getDiet properties return the function body and not the literals for each?

[piecioshka]

@robreczarek nie dam rady chyba po ang to wytłumaczyć 😄 ale spróbuje...

When you defined:

var animals = [];

this is a local variable. So the scope of access starts at 1 line of index.js file and ends in 5 line (when brackets ends).

You should make animals as object member.

this.animals = [];

And in each "method" you should use this as reference to object instance. eg

return this.animals.push(animal);

[piecioshka]

@robreczarek btw. please make seconds file test.js where you import index.js file (by require global function) and move whole console.log block to it.

[robreczarek]

@piecioshka Możemy też po polsku. Powiedzmy że będzie mi szło raczej koślawo ;)

Usunąłem var i zamieniłem na this, ale nadal mam problem z tym że wypluwa mi funkcję zamiast wartości.

No i teraz jak mam test w oddzielnym pliku, to nie wiem jak zrobić żeby odpaliło. Wyskakują mi ReferenceError.

Probowałem też
var index = require('./index.js');

I w console.log
console.log( 'Welcome to ' + index.myzoo.zooName );

Ale też mam wtedy ReferenceError

[piecioshka]

@robreczarek nie nie, piszemy po angielsku 😄

I was created Pull Request with some changes: #4
Please review it.

[robreczarek]

@piecioshka I see that you added object definitions for Lion and Elephant, plus a section for module.exports.

This is to expose them for consumption when you require a file, correct? Is this a node.js specific thing?

[piecioshka]

This is called "CommonJS syntax".
In definition file you exports, and in setup file you require.

[robreczarek]

@piecioshka Got it. Gonna read up on the spec. Seems like this part is a wrap, so I'm closing this out.