[SOLUCIONADO] Callback node.js

[dunga007]

Hola!, tengo un problema bastante avanzado..., espero me puedan ayudar, se los planteo:

Estoy haciendo un chat para usuarios que tengo en una base de datos, el desafio:

Hacer que cuando el mismo usuarios se conecte desde otra hubicacion cierre la sesion que tenia abierta e incie la nueva

Lo que tengo es:

Cuando un usuario se conecta:

Código HTML:

for (i in usuariosConectados)
                {
                   if(usuariosConectados[i].id == rows[0].id) {
                      //le envio un mensaje a la hubicacion anterior informandoleio.sockets.socket(usuariosConectados[i].socket).emit("alerta_desconectar");
                   }
                }

                //Lo agrego al objeto
                usuariosConectados[rows[0].id]={
                                        'id':rows[0].id,
                                        'nombre': rows[0].display_name,
                                        'socket': socket.id,                                            
                                        };

Como se puede ver primero reviso si el usuario estaba conectado, en caso de estar conectado, llamo a la funcion alerta_desconectar como se muestra arriba, esta funcion del lado del cliente es:

Código HTML:

function alerta_desconectar(){
                alert("Se ha cerrado la sesion porque ha iniciado sesion  desde otra ubicacion, pestaña o ventana");
                socket.emit('desconectar');
             }

y desde esta funcion se llama a la funcion desconectar en el servidor que es:

Código HTML:

socket.on('desconectar', function()
    {
      socket.disconnect()
    });

Y luego continuo agregando el usuario al objeto usuariosConectados.

Parece que todo va bien pero al forzar la desconeccion se dispara automaticamente la funcion socket.on('disconnect') que tengo tambien definida y que lo que hace es, quitar al usuario de la lista de conectados y borrarlo del objeto usuariosConectados. La consecuencia:

Al ingresar el cliente desde otra hubicacion:
a) lo desconecta de la hubicacion anterior
b) borra al usuario actual del objeto usuariosConectados provocando que todo ande mal

Se me ocurrio solucionarlo con callbacks para que no agrege la nueva conexion al objeto usuariosConectados hasta que haya terminado con todo ese proceso pero nose como deberia hacerlo ni si es la mejor opcion o la mas correcta, espero alguien pueda ayudarme ya que este problema excede mis conocimientos. Muchisimas gracias!!!

[utan]

Hola,

Ya que usuariosConectados[i].id == rows[0].id son iguales , solo abrio otra ventana o se logeo con otro browser, lo que deves hacer es actulizar tu objeto con el nuevo socket.io..

Prueba creando una variable salvas el viejo socket.id como el id de la base de datos es el mismo entonces actualizas tu objeto , rows[0].display_name y socket.id después mandas el emit a usuario al que le pertenece el viejo id el de la variable , sin necesidad de borrarlo de usuariosConectados..

Ya que como todo es asyncronico cuando mandas el disconnect a la ventana antigua , la ventana nueva es salvada en tu usuariosConectados borrandolo y quizás mandando el disconnect también a la nueva ventana.

[dunga007]

Iniciado por utan

Hola,

Ya que usuariosConectados[i].id == rows[0].id son iguales , solo abrio otra ventana o se logeo con otro browser, lo que deves hacer es actulizar tu objeto con el nuevo socket.io..

Prueba creando una variable salvas el viejo socket.id como el id de la base de datos es el mismo entonces actualizas tu objeto , rows[0].display_name y socket.id después mandas el emit a usuario al que le pertenece el viejo id el de la variable , sin necesidad de borrarlo de usuariosConectados..

Ya que como todo es asyncronico cuando mandas el disconnect a la ventana antigua , la ventana nueva es salvada en tu usuariosConectados borrandolo y quizás mandando el disconnect también a la nueva ventana.

Hola utan!, muchas gracias por tu colaboracion, entiendo perfectamente lo que decis y tenes toda la razon en el razonamiento... el tema es que cuando lo desconecto al usuario viejo se dispara automaticamente esto (que lo tengo para cuando el usuario cierra el navegador):

Código HTML:

socket.on('disconnect', function () 
		{
			//Eliminamos al usuario del objeto
			delete usuariosConectados[socket.identificador];			
			//Mandamos la información a las Sockets para que se borre de la lista
			socket.broadcast.emit("usuarioDesconectado",socket.identificador);
		});

En socket.identificador es:

Código HTML:

socket.identificador=rows[0].id;

Por eso se me habia ocurrido usar callbacks para que luego de que haya hecho todo el proceso de desconexion recien se actualice o se agrege al objeto, pero no se como hacerlo . Espero pueden ayudarme porque esta muy complicado...

[utan]

hola,
Ok, mira te borra el disconnect el usuario porque estas usando el mismo id de la base de datos..

La nueva ventana y el socket anterior utilizan el mismo id de la base de datos, por lo tanto cuando el nuevo entra el otro emite un disconnect y como el id es el mismo de la bd borra el que acaba de entrar..

Lo que tienes que hacer es re trabajar en tu lógica ya que no la puedes trabajar así...

Una idea seria que crearas otra row en tu base de datos y que esta tuviera un timestamp del nuevo login, el mismo id del usuario lo tendrías y no molestaría tu código mucho..

y podrías comprobar algo así.

Código Javascript:

socket.on('disconnect', function () 
        {
            //usas en vez del id del db, la timestamp de su reciente login
            delete usuariosConectados[socket.timestamp];            
            //Mandamos la información a las Sockets para que se borre de la lista
            socket.broadcast.emit("usuarioDesconectado",socket.identificador);
        });
 

Usas en vez del id del db, la timestamp de su antiguo login el cual es el mismo id de la db del usuario pero la timestamp lo define como diferente .. así no borrara al mas reviente en tu objeto..
Este es un ejemplo mas claro de mi idea , puede haver bugs:

Código Javascript:

var clients = {};
    
    server.on( 'connection' , function( socket ){
        // obtenemos de la BD
        //db.id , db.screen_name,db.timestamp
        for ( i in clients ){
            // si no existe crear nuevo
            if( clients[i] && clients[i].id !== db.id ){
                socket.identifier   =   db.timestamp;
                
                clients[db.timestamp] = {
                    id          : db.id,
                    screen_name   : db.screen_name,
                    sockId      : socket.id,
                     timestamp  :db.timestamp
                }
            }else{
            // inicializa el disconnect de el socket antiguo
            var old_socket  =   socket.sockets.sockets[clients[i].sockId];
                old_socket.json.emit('user_lef' , 'user left the app');
                old_socket.disconnect();
            }           
            
        }
        
        
        server.on('disconnect' , function(){
            if( clients[socket.timestamp] ) delete clients[socket.timestamp];
        });
    });
 

[dunga007]

Iniciado por utan

hola,
Ok, mira te borra el disconnect el usuario porque estas usando el mismo id de la base de datos..

La nueva ventana y el socket anterior utilizan el mismo id de la base de datos, por lo tanto cuando el nuevo entra el otro emite un disconnect y como el id es el mismo de la bd borra el que acaba de entrar..

Lo que tienes que hacer es re trabajar en tu lógica ya que no la puedes trabajar así...

Una idea seria que crearas otra row en tu base de datos y que esta tuviera un timestamp del nuevo login, el mismo id del usuario lo tendrías y no molestaría tu código mucho..

y podrías comprobar algo así.

Código Javascript:

socket.on('disconnect', function () 
        {
            //usas en vez del id del db, la timestamp de su reciente login
            delete usuariosConectados[socket.timestamp];            
            //Mandamos la información a las Sockets para que se borre de la lista
            socket.broadcast.emit("usuarioDesconectado",socket.identificador);
        });
 

Usas en vez del id del db, la timestamp de su antiguo login el cual es el mismo id de la db del usuario pero la timestamp lo define como diferente .. así no borrara al mas reviente en tu objeto..
Este es un ejemplo mas claro de mi idea , puede haver bugs:

Código Javascript:

var clients = {};
    
    server.on( 'connection' , function( socket ){
        // obtenemos de la BD
        //db.id , db.screen_name,db.timestamp
        for ( i in clients ){
            // si no existe crear nuevo
            if( clients[i] && clients[i].id !== db.id ){
                socket.identifier   =   db.timestamp;
                
                clients[db.timestamp] = {
                    id          : db.id,
                    screen_name   : db.screen_name,
                    sockId      : socket.id,
                     timestamp  :db.timestamp
                }
            }else{
            // inicializa el disconnect de el socket antiguo
            var old_socket  =   socket.sockets.sockets[clients[i].sockId];
                old_socket.json.emit('user_lef' , 'user left the app');
                old_socket.disconnect();
            }           
            
        }
        
        
        server.on('disconnect' , function(){
            if( clients[socket.timestamp] ) delete clients[socket.timestamp];
        });
    });
 

Muy biena idea!, lo voy a intentar y te cuento como me va. Gracias!

[dunga007]

Muchisimas gracias Utan, ya lo solucione!, solo usando el timestamp, no lo habria logrado sin tu ayuda!. Gracias de nuevo!!!