Почему при колбеке теряется переменная из scope?

Question

[kpoxas]

Имеется парсер html страницы, который заносит данные в DB, c колбеком (ниже). На одной странице имеется несколько опций, которые надо занести в таблицу отдельно. Их перебираю с помощью
$(...).each, затем проверяю существование в таблице ключа product_id-option_id, обновляя или записывая затем новые данные.

...
"callback":function(error,result,$) {
        if (error) {
            logger.error(error);
            return;
        }
        logger.info('(%d/%d) Url %s %d', this.idx, this.total, this.uri, result.statusCode);
        if (result.statusCode!=200) {
            return;
        }
        // $ is a jQuery instance scoped to the server-side DOM of the page
        var product_new = {
            product_id: this.product_id         
        };        
        // options 
        $("#pr .option").each(function(index,a)                   
                  
                product_new.option_id = $(a).data('opt')||0;        
             
                logger.debug('>> pid=%d, opt=%d',product_new.product_id, product_new.option_id);

                Product.find({product_id: product_new.product_id, option_id: product_new.option_id||0}).success(function(product_old){     
                    logger.debug(index);
                    logger.debug('<< pid=%d, opt=%d',product_new.product_id, product_new.option_id);
                });             
            });  
    }

Проблема в том, что методу (объект ORM Sequelizejs) Product.find передается по очереди несколько пар параметров, но в колбеке после поиска их в БД они устанавливаются во всех случаях в последнюю пару:

[Friday, January 31, 2014 11:52:59,379] DEBUG >> pid=76, opt=13
[Friday, January 31, 2014 11:52:59,379] DEBUG >> pid=76, opt=308
[Friday, January 31, 2014 11:52:59,380] DEBUG >> pid=76, opt=310
[Friday, January 31, 2014 11:52:59,380] DEBUG >> pid=76, opt=311
[Friday, January 31, 2014 11:52:59,502] DEBUG 0
[Friday, January 31, 2014 11:52:59,503] DEBUG << pid=76, opt=311
[Friday, January 31, 2014 11:52:59,789] DEBUG 2
[Friday, January 31, 2014 11:52:59,789] DEBUG << pid=76, opt=311
[Friday, January 31, 2014 11:52:59,793] DEBUG 3
[Friday, January 31, 2014 11:52:59,793] DEBUG << pid=76, opt=311
[Friday, January 31, 2014 11:52:59,802] DEBUG 1
[Friday, January 31, 2014 11:52:59,802] DEBUG << pid=76, opt=311

При чем, переменная индекса перебора передается та, которая была при вызове метода Product.find, то есть нормально. Грешил на то что product_new не в скоупе из-за того, что определяется выше перебора опций, но если на каждую опцию product_new копировать локальной переменной, будет тоже самое, но ведь index же нормально передается!

Answer 1

[gro]

Например, просто сделать в начале each()

var product_id = product_new.product_id,
    option_id = product_new.option_id;

и дальше их использовать.

Comments

[kpoxas]

а почему все же var new_product_tmp = new_product; в начале each() не срабатывает?

[kpoxas]

единственное, что приходит на ум, что product_new -это объект, если просто его присвоить новой перемнной, то так установится ссылка на объект, а если надо его клонировать, то надо перебрать и установить все его атрибуты

[kpoxas]

Да, действительно msdn.microsoft.com/ru-ru/library/d53a7bd4(v=vs.94).aspx

Answer 2

[kpoxas]

Помогло просто разделить переменные, что касается общих данных - product_new, а на каждую опцию создал option_data_new

Answer 3

[gro]

callback один раз вызван -> есть только один контекст -> product_new определён в нём -> разделяется всеми вложенными функциями

анонимка из each() вызывается для каждого элемента -> каждый раз свой контекст, в котором определён index -> анонимка из success() имеет доступ к тому index, который определён в том же контексте, что и она

Comments

[kpoxas]

если внутри each делать var new_product_tmp = new_product;
а потом передавать new_product_tmp в Product.find, то будет тоже самое.

[kpoxas]

может знаете, как можно обойти эту проблему?