Javascript Zugriff auf Elemente des Arrays funktioniert nicht

[Netron (Gast)]

Hallo,

ich bin ein Neuling in der NodeJS Entwicklung und habe eine Frage. Ich 
habe mir einen kleinen Server geschrieben, der die Ports und Adressen 
der Clients mitloggen soll. Diese werden in einer globalen Variable 
gespeichert.

Leider verhält es sich so, dass die bei mehrmaligen Empfangen von 
Nachrichten die Adressen im Array verloren gehen. Es ist immer nur die 
Adresse des letzten Pakets gespeichert. Kann mir da jemand weiterhelfen? 
Wahrscheinlich habe ich ein grundsätzliches Problem beim Verständnis von 
Javascript. Die Ausgabe nach drei Nachrichten sieht wie folgt aus:

Server listening on 255.255.255.255:7772
new client at 192.168.178.23:47696
[ { addr: '192.168.178.23', port: 47696 } ]
new client at 192.168.178.23:35086
[ { addr: false, port: 47696 },
  { addr: '192.168.178.23', port: 35086 } ]
new client at 192.168.178.23:38922
[ { addr: false, port: 47696 },
  { addr: false, port: 35086 },
  { addr: '192.168.178.23', port: 38922 } ]

var net = require('net');
var dgram = require('dgram');

function ClientList () {
    this.list = [];
    
  this.add = function (addr, port) {
    var obj = {
      addr: addr,
      port: port
    };  
    this.list.push(obj);    
  };
  
  this.find = function (addr, port) {
    for (i = 0; i < this.list.length; i++) {
      if(this.list[i].addr = addr && this.list[i].port == port)
        return true;
    }
    return false;
  };
}

var clients =  new ClientList();
var server = dgram.createSocket('udp4');

server.on('listening', function () {
    var address = server.address();
    console.log('UDP Server listening on ' + address.address + ":" + address.port);
});

server.on('message', function (message, remote) {
      
  if(clients.find(remote.address, remote.port) == false)
  {      
    console.log("new client at " + remote.address + ':' + remote.port );
    
    clients.add(remote.address, remote.port);
    
    console.log(clients.list);    
  }
  
});
server.bind(7772, '255.255.255.255');

[Daniel A. (daniel-a)]

Diese client detections methode ist sehr unzuverlässig. Ich würde eine 
Session variable einführen. Der fehler ist ein Tippfehler:

Ersetze:
> this.list[i].addr = addr && this.list[i].port == port
Durch:
this.list[i].addr == addr && this.list[i].port == port

PS: JS bietet auch möglichkeiten das versehentliche verändern von werten 
zu verhindern. Object.seal und Object.freeze zum beispiel. Dann gibt es 
Object.defineProperty für komplexere Dinge und seit ECMAScript 6 aka 
JS1.7.8 ein const keyword.

[Dirk D. (dicky_d)]

Der Klassiker, Du machst ne Zuweisung statt einen Vergleich.
  if(this.list[i].addr = addr && this.list[i].port == port)

_______________________^

Versuchs da mit mit einem == :)