[SOLUCIONADO] ¿Porqué siempre me coge la misma id?

[SeaPirates]

Hola, estoy intentando hacer que por cada zona que existe inserte todos los objetos que hay si no existen ya. El problema que tengo es que hay 2 objetos y teóticamente tendría que insertar 20 de cada, y me inserta en total 40, hasta ahí bien. Pero me los inserta todos con la misma id, la última. No entiendo porqué sucede, he llegado a pensar que el bucle for se ejecuta antes que el do que tiene dentro..

Código Javascript:

client.query("SELECT id FROM world_areas", function(errors, rows) {
    for(var i = 0;i < rows.length; i++) {
        var map = rows[i].id;
    
        client.query('SELECT * FROM objects WHERE regeneration = "1"', function(errors, rows) {
            
            for(var e = 0;e < rows.length; e++) {
            
                var max_objects = rows[e].max_objects;
                var name = rows[e].name;
                var ammount = rows[e].ammount;
                var type = rows[e].id;
                client.query('SELECT id FROM world_objects WHERE type = "' + type + '" AND map = "' + map + '"', function(errors, rows) {
                    var dif = parseInt(max_objects)-rows.length;
                    var a = 0;
                    do {
                        var x = 1100 + Math.floor(Math.random() * 3600);
                        var y = 1100 + Math.floor(Math.random() * 3600);
                        client.query('INSERT INTO world_objects (name, type, map, x, y) VALUES ("' + name + '", "' + type + '", "' + map + '", "' + x + '", "' + y + '")');
                        a++;
                        console.log('OBJETO[' + a + '] => ' + type );
                    }while(a < dif);
                            
                });
                
            }
        });
    
    
        
        console.log(' [' + date + '] -> GENERADOS OBJETOS EN EL MAPA ' + rows[0].id);
    }
});
 

¿Alguna idea?

[chwc]

Hola ! primero que nada no se en donde esta el problema, pero lo que si que nunca le asignas los datos a objeto[ ] dentro del bucle, solo lo agregas a una BD

Estas haciendo un juego no ? me encanta la idea :D

[chwc]

Mira, de todos modos te recomiendo que no abuses de mysql ni de ninguna BD, te vas a volver loco. trata de utilizarlo solo para almacenar backups del estado del juego, es mejor tener todos los datos a mano en la memoria ram.

es decir, hacer una especie de BD con javascript, con objetos, para asi no tener que lidiar con los callbacks de las bases de datos ( son todas, redis, mongoBD, mysql, etc.. ) y ademas, obviamente agilizar todo

es solo una opinion

[SeaPirates]

Hola, si es un juego.

Lo que pasa es que esta función la utilizo para cuando inicio el servidor para que se reestablezcan todos los objetos en cada mapa.

La variable "object" proviene de una función, te paso el codigo completo:

Código Javascript:

client.query("SELECT id FROM world_areas", function(errors, rows) {
    for(var i = 0;i < rows.length; i++) {
        var map = rows[i].id;
    
        client.query('SELECT * FROM objects WHERE regeneration = "1"', function(errors, rows) {
            
            for(var e = 0;e < rows.length; e++) {
            
                var max_objects = rows[e].max_objects;
                var name = rows[e].name;
                var ammount = rows[e].ammount;
                var type = rows[e].id;
                client.query('SELECT id FROM world_objects WHERE type = "' + type + '" AND map = "' + map + '"', function(errors, rows) {
                    var dif = parseInt(max_objects)-rows.length;
                    var a = 0;
                    do {
                        var x = 1100 + Math.floor(Math.random() * 3600);
                        var y = 1100 + Math.floor(Math.random() * 3600);
                        client.query('INSERT INTO world_objects (name, type, map, x, y) VALUES ("' + name + '", "' + type + '", "' + map + '", "' + x + '", "' + y + '")');
                        a++;
                        console.log('OBJETO[' + a + '] => ' + type );
                    }while(a < dif);
                            
                });
                
            }
        });
    
    
        
        console.log(' [' + date + '] -> GENERADOS OBJETOS EN EL MAPA ' + rows[0].id);
    }
});
 

Funcionar funciona, porque lee cada mapa, y lee cada objeto.

Si pongo un console.log con la id dentro del for de la tabla objetos me devuelve las id correctamente, pero si coloco en console.log dentro del do, me devuelve la misma id todo el rato

Es como si se estuviera sobrescribiendo la id por el ultimo registro

[zerokilled]

buenas...
no he podido interpretar bien todo el código pero se me ocurre que puede ser el problema clásico de los bucles y los eventos asíncronos. recuerda que la mayoría de los componentes en nodejs son asíncronos. a esto sumale que javascript es late-binding. lo que más o menos quiere decir que javascript obtiene los valores al momento de invocarlos. de modo que si esa variable cambia, eventualmente se reflejará. tienes que hacer una especie de bind con closure porque creo que con bind no te funcionará.

básicamente en el bucle creas una función a la que le pasas como argumento el iterador y dentro de esta función utilizas ese argumento para los callbacks.