EN Using new api "Classes"

[Ofir Gal]

Hey, which is more efficient?

registerPlugin....

var engine = require('engine');

sinusbot.on('chat', function(ev) {
    engine.log('Hello from a script!');
});

sinusbot.on('poke', function(ev) {
        engine.log('We got poked by ' + ev.clientNick + ': ' + ev.msg);
});

or

registerPlugin....

sinusbot.on('chat', function(ev) {
    var engine = require('engine');
    engine.log('Hello from a script!');
});

sinusbot.on('poke', function(ev) {
     var engine = require('engine');
     engine.log('We got poked by ' + ev.clientNick + ': ' + ev.msg);
});

 

[Multivitamin]

i would recommend to use the first solution, they both do the same but with the first you are much cleaner and need to write less
But the Second Solution probably does not even use more Ram

You could even do
require('ENGINE').log("bla")

 

[kanalumaddela]

The 1st one obviously....I don't see why'd you want to define the same backend twice.

 

[flyth]

The first one, but I think it doesn't really matter a lot in that particular case. You should however use the new event module as well.

 

[Ofir Gal]

Thx for the quick answers, I have just wondered if its waste of memory

@flyth, Im still reading the new api thx for noticing.

 

[Xuxe]

Thx for the quick answers, I have just wondered if its waste of memory

We are not in C/C++ here, so we don't need to think much about Memory Leeking. Behind the Engine runs a Garbage Collector, so mostly the memory should be clean


But +1 for the interest in writing clean code.