You may also need to bind `onConnection` to `this` otherwise `this.onData` would be invalid as well.
On Monday, March 25, 2013 9:28:06 AM UTC-7, mgutz wrote: > > > Shouldn't `stream.on('data', onData);` be `this.onData`? > > > > > > On Monday, March 25, 2013 8:44:02 AM UTC-7, Perrier wrote: >> >> Hi, >> >> I cannot reach some of the fields of my class and I don't understand why. >> Someone please take a look at my code. This is the Nth one, I was changing >> this to var, to TCPServer.prototype.sg etc. Nothing helped. >> I've started with a sample tcp server from Anwajler's blog >> here<http://anwajler.com/node.html>. >> It was working well, but I would like to put it into a seperate file, into >> a module. I've created my TCPServer class as follows: >> >> var net = require('net'); >> var GatewayClass = require('./Gateway.js'); >> >> exports.TCPServer = TCPServer; >> >> function TCPServer(host, port) { >> this.HOST = host; >> this.PORT = port; >> var tcpServer = net.createServer(); >> this.gatewayList = []; >> this.findByAddressAndPort = function (address, port) { >> var i; >> for(i = 0; i < gatewayList.length; i++){ >> if(gatewayList[i] !== undefined) { >> if(gatewayList[i].IP === address && gatewayList[i].Port === port) { >> return gatewayList[i]; >> } >> } >> } >> }; >> this.dataHandler = function (stream, data) { >> var gateway = this.findByAddressAndPort(stream.remoteAddress, >> stream.remotePort); >> gateway.Buffer.Add(data); >> }; >> >> this.onData = function(data) { >> this.dataHandler(this, data); >> } >> >> this.onClose = function(){ >> console.log('CONNECTION CLOSED'); >> }; >> this.onError = function(exception) { >> console.log('SOCKET ERROR: ' + exception); >> }; >> >> this.onListen = function() { >> tcpServer.on('connection', this.onConnection); >> tcpServer.on('error', this.onError); >> tcpServer.on('close', this.onClose); >> console.log('Server listening on ' + this.HOST +':'+ this.PORT); >> }; >> >> this.onConnection = function (stream) { >> var gw=new GatewayClass.Gateway(stream); >> gatewayList.push(gw); >> stream.on('data', onData); >> console.log("Clients: ", gatewayList.length); >> }; >> this.Open = function() { >> tcpServer.listen(this.PORT, this.HOST, this.onListen); >> } >> } >> >> TCPServer.prototype.Listen = function() { >> this.Open(); >> } >> >> >> In my main application I create an instance of TCPServer and start >> listening this way: >> >> var TCPServerClass = require('./tcpserver.js'); >> >> var TCPServer = new TCPServerClass.TCPServer(GLOBALS.HOST, GLOBALS.PORT); >> TCPServer.Listen(); >> >> >> Thanks for your help. This version says: >> >> events.js:130 >> throw TypeError('listener must be a function'); >> ^ >> TypeError: listener must be a function >> at TypeError (<anonymous>) >> at Server.EventEmitter.addListener (events.js:130:11) >> >> Peter >> >> -- -- Job Board: http://jobs.nodejs.org/ Posting guidelines: https://github.com/joyent/node/wiki/Mailing-List-Posting-Guidelines You received this message because you are subscribed to the Google Groups "nodejs" group. To post to this group, send email to nodejs@googlegroups.com To unsubscribe from this group, send email to nodejs+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/nodejs?hl=en?hl=en --- You received this message because you are subscribed to the Google Groups "nodejs" group. To unsubscribe from this group and stop receiving emails from it, send an email to nodejs+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out.