problema con HashMap

[cucuru]

hola, tengo un HashMap que almacena una MAC y un timer:

Código:

    public static HashMap<Long, KeepAliveTimer> kaTimer;

La clase KeepAliveTimer es un timer que relaciona la MAC con la IP:

Código:

	public KeepAliveTimer(long mac, String ipAddress)

creo 7 elementos del tipo KeepAliveTimer y los inserto en el HashMap:

Código:

    keepAliveTimer keepAlive = new keepAliveTimer(mac, ip); 
    kaTimer.put(mac, keepAlive);

Y finalmente extraigo el objeto KeepAliveTimer del HashMap:

Código:

   KeepAliveTimer keepAlive = kaTimer.get(mac);
   keepAlive.update();

Sin embargo, si de ese elemento KeepAliveTimer extraigo el valor de "ip" siempre obtengo la ip del último elemento que inserté en el Hash en lugar de la "pareja" de la mac.

¿qué estoy haciendo mal?

Gracias!

[Xerelo]

Pues o todas las mac las insertas con la misma ip, o estás metiendo siempre la misma mac por lo que en realidad solo tendrás un timer en el map.

En debug mira qué contiene el map.

[cucuru]

Ya lo he probado, y aparentemente está todo bien.

Para simplificarlo todo, he reducido el número de elementos de 7 a 2, y añadido a mi código en la parte de extraer:

Código:

System.out.println("tamaño" + kaTimer.size());
Collection col = kaTimer.values();
	for (Object obj : col) {
		System.out.println("temporizador " + obj);
     }

	System.out.println("La MAC" + String.valueOf(MACAddress));
	keepAliveTimer keepTimer = kaTimer.get(MACAddress);
	System.out.println("y su timer" +keepTimer);

Y lo que me reporta, para el primer elemento:

Código:

 temporizador spb.KeepAliveTimer@105bd58
 temporizador spb.KeepAliveTimer@edf3f6
 La MAC 26954208388119
 y su timer spb.KeepAliveTimer@edf3f6

Y para el segundo:

Código:

 temporizador spb.KeepAliveTimer@105bd58
 temporizador spb.KeepAliveTimer@edf3f6
 La MAC 26954208388118
 y su timer spb.KeepAliveTimer@105bd58

por lo que parece que eso está bien, no??

[Xerelo]

Tienes dos objetos distintos con dos mac distintas, por lo que si te aparece la misma ip es que cuando los creas tienen la misma ip.

Revisa cómo obtienes la ip cuando creas el objeto.

[cucuru]

He descubierto un poquito más, pero no llego a nada.

He empezado de cero:

Código:

public MyTimer(long mac, String ipAddress){
    this.macAddress = mac;
    this.ip = ipAddress;

    timer = new Timer();

    TimerTask timerTask = new TimerTask() {

        @Override
        public void run() {
            log.info("mac " + String.valueOf(macAddress) + " ip " + ip);

        }
    };

    timer.schedule(timerTask, 30*1000,30*1000);
    log.info("timer created, mac:" String.valueOf(macAddress) + " ip " + ip);
}

Desde otra clase

Código:

MyTimer timer1 = new MyTimer(mac1, ip1);
MyTimer timer2 = new MyTimer(mac2, ip2);

Cuando se crean:

Código:

Oct 23, 2012 1:36:43 PM spb.MyTimer <init>
INFO: timer created, mac: 26954208388119 ip 77.209.184.53
Oct 23, 2012 1:36:43 PM spb.MyTimer <init>
INFO: timer created, mac 26954208388118 ip 178.139.7.27

Cuando se alcanza el timer la cosa falla

Código:

Oct 23, 2012 1:39:13 PM spb.MyTimer$1 run
INFO: mac 26954208388119 ip 178.139.7.27
Oct 23, 2012 1:39:13 PM spb.MyTimer$1 run
INFO: mac 26954208388118 ip 178.139.7.27

También he probado con "MyTimer extends TimerTask" pero nada, que no doy con ello...

[Xerelo]

Lo que preguntaste al principio no es lo mismo que lo que preguntas después.

De todas formas, el código que has puesto funciona bien

Código Javascript:

public class TimerC {
 
    public static void main(String[] args) {
        MyTimer timer1 = new MyTimer(26954208388119L, "77.209.184.53");
        MyTimer timer2 = new MyTimer(26954208388118L, "178.139.7.27");  
    }
}
 
class MyTimer{
    
    Long macAddress;
    String ip;
    
    public MyTimer(long mac, String ipAddress){
        this.macAddress = mac;
        this.ip = ipAddress;
 
        Timer timer = new Timer();
        TimerTask timerTask = new TimerTask() {
            @Override
            public void run() {
                System.out.println("mac " + String.valueOf(macAddress) + " ip " + ip);
            }
        };
 
        timer.schedule(timerTask, 30*1000,30*1000);
        System.out.println("timer created, mac:" + String.valueOf(macAddress) + " ip " + ip);
    }
}
 

Da como resultado

timer created, mac:26954208388119 ip 77.209.184.53
timer created, mac:26954208388118 ip 178.139.7.27
mac 26954208388118 ip 178.139.7.27
mac 26954208388119 ip 77.209.184.53
mac 26954208388119 ip 77.209.184.53
mac 26954208388118 ip 178.139.7.27
mac 26954208388118 ip 178.139.7.27
mac 26954208388119 ip 77.209.184.53

[cucuru]

Hola, sí, ya se que no es el mismo error, es que he visto que el error del HashMap no es del Hash sino de la clase KeepAliveTimer (o MyTimer que es la misma), lo puse por intentar ser más clara, y veo que he conseguido lo contrario

Visto que el código te funciona probaré donde está mi error, porque está claro que debe estar en otro sitio.

Gracias!