Из-за чего возникает переполнение стека?

Question

[Sergey Grigorov]

Здравствуйте. Имеется некая функция:

function db_prepare(table, columns){
    var weight = 0;
    var is = Core.dbcfg_tables.indexOf(table);
    if(is){
        weight = 1;
    }
    if(columns != undefined){
        var i;
        for(i in columns){
            i = columns[i].column;
            try{
                if(Core.dbcfg_structures[table].columns.indexOf(i) != undefined){
                    if(weight > 0){
                        weight = 2;
                    }else{
                        weight = 1;
                    }
                }else{
                    if(weight > 0){
                        weight = 1;
                    }else{
                        weight = 0;
                    }
                }
            }catch(Ex){
                if(weight > 0){
                    weight = 1;
                }else{
                    weight = 0;
                }
            }
        }
    }
    if(columns != undefined){
        if(weight < 2){
            return false;
        }else{
            return true;
        }
    }else{
        if(weight < 1){
            return false;
        }else{
            return true;
        }
    }
}

Мне отладчик упорно твердит, что в ней происходит переполнение стека (скорее всего, создается вечный цикл), но никак не возьму в толк -- в чем конкретно здесь проблема. Голова после продуктивной ночи отказывается уже думать.
Заранее спасибо за помощь.

Comments

[Павло Пономаренко]

Что значит "переполнение стека"? Можете процитировать ошибку, пожалуйста?

Answer 1

[vreitech]

i = columns[i].column;
мне вот эта строка не по нраву, сам не знаю почему. и так на неё смотрел, и эдак - не нравится, хоть ты тресни.

Comments

[Павло Пономаренко]

Потому что переменная i уже занята под содержимое шага массива, а потом записывается значением колонки. Тут крайне необходимо использовать другую переменную

[Sergey Grigorov]

Изменил. К сожалению, все равно идет переполнение стека. Указывает именно на эту функцию, и почему-то на строку:

if(is){
        weight = 1; // на эту
    }

[AngReload]

Да, лучше так что-ли

var idx, i;
for(idx in columns){
  i = columns[idx].column;

Answer 2

[Павло Пономаренко]

Еще вы неправильно используете indexOf:

indexOf(i) != undefined

indexOf НИКОГДА не возвращает undefined. Если вы хотите проверить элемент на существование в массиве, то необходимо это делать так:

indexOf(i) >= 0

Comments

[Sergey Grigorov]

Благодарю за совет.:)

[Павло Пономаренко]

Sergey Grigorov, я вообще для начала переписал бы код как-то так:

function db_prepare(table, columns){
    var weight = 0;

    if (Core.dbcfg_tables.indexOf(table) >= 0) weight = 1;

    if (columns){
        for(var i in columns){
            var item = columns[i].column;
            try{
                if (Core.dbcfg_structures[table].columns.indexOf(item) >= 0){
                    weight = (weight > 0) ? 2 : 1;
                } else {
                    weight = (weight > 0) ? 1 : 0;
                }
            }catch(Ex){
                weight = (weight > 0) ? 1 : 0;
            }
        }

        return weight >= 2;
    } else {
        return weight >= 1;
    }
}

Answer 3

[Sergey Grigorov]

Разобрался. Слона и не приметил.
Имеется Core.js, который инициализирует все, что необходим ядру. В нем неверно передавалось название функции для инициализации, из-за чего просто-напросто не создавалась структура таблицы, а поскольку всё лежало в блоке try catch, то он не мог выдать адекватную ошибку. Всем спасибо за ответы и советы.:)