在javascript中实例化循环中的类使用最后一个值

时间:2013-07-03 11:45:25

标签: javascript class loops closures

首先,我知道JavaScript中有很多关于闭包的问题,​​特别是涉及循环时。我已经阅读了很多,但我似乎无法弄清楚如何解决我自己的特定问题。我的主要经验在于C#,C ++和一些ASM,并且它已经习惯了JavaScript。

我正在尝试在一些for循环中用一个类的新实例(称为Tile)填充一个三维数组。我想要做的就是将一个其他类(称为Group)的引用传递给第一个循环中实例化的(并且还添加到另一个数组中)。正如您可能已经猜到的,在完成循环之后,Tile类的每个实例都引用了同一个Group对象,即最后一个要创建的对象。

显然,不是传递对Group对象的引用,而是传递对函数本地的某个变量的引用,该引用在循环的每次迭代中都会更新。我的假设是解决这个问题与闭包有关,因为我在寻找解决方案时遇到了很多类似的问题。

我发布了一些修剪过的代码,在jsFiddle上公开了问题的核心:

//GW2 namespace
(function( GW2, $, undefined ) {

    //GW2Tile class
    GW2.Tile = function(globalSettings, kineticGroup)
    {
        //Private vars
        var tilegroup = kineticGroup;
//      console.log(tilegroup.grrr); //Shows the correct value
        var settings = globalSettings;

        this.Test = function(){
            console.log(tilegroup.grrr);
        }

        this.Test2 = function(group){
            console.log(group.grrr);
        }

    } //Class

}( window.GW2 = window.GW2 || {}, jQuery ));


var zoomGroups = [];
var tiles = [];
var settings = {};

InitArrays();
tiles[0,0,0].Test(); //What I want to work, should give 0
tiles[0,0,0].Test2(zoomGroups[0]); //How I'd work around the issue

function InitArrays(){
    var i, j, k, zoomMultiplier, tile;

    for(i = 0; i <= 2; i++){
        zoomGroups[i] = {};
        zoomGroups[i].grrr = i;

        tiles[i] = [];
        zoomMultiplier = Math.pow(2, i);
        for(j = 0; j < zoomMultiplier; j++){
            tiles[i,j] = [];
            for(k = 0; k < zoomMultiplier; k++){
                tile = new GW2.Tile(settings, zoomGroups[i]);
                tiles[i,j,k] = tile;
            }
        }
    }
}               

到目前为止,在使用JavaScript时,我通常会对代码进行一些调整以使其工作,但我已经厌倦了使用看似凌乱的解决方法,因为我知道应该有一些相当简单的解决方案。我只是不喜欢寻求帮助,但这真的让我感到满意。非常感谢任何帮助。

1 个答案:

答案 0 :(得分:2)

多维数组

问题

上面代码的第一个问题是您尝试创建多维数组的方法。

您使用的语法是:

tiles[0,0,0]

然而,JavaScript将解释这一点的方式是:

tiles[0]

访问multidim数组

如果您想访问multidim阵列,则必须使用:

tiles[0][0][0]

要创建multidim数组,您需要执行以下操作:

tiles = [];
tiles[0] = [];
tiles[0][0] = [];
tiles[0][0][0] = 'value';

或:

tiles = [[['value']]];


关于您的代码

在您的代码中,您应该使用:

tiles[i][j][k] = tile;

但是你应该确保每个子数组在设置它的值之前实际存在,否则你会得到undefinedillegal offset错误。

您可以通过以下方式执行此操作:

(typeof tiles[i] === 'undefined') && (tiles[i] = []);
(typeof tiles[i][j] === 'undefined') && (tiles[i][j] = []);
tiles[i][j][k] = tile;

显然,上面的内容可以根据您遍历循环的方式进行优化,但最好在进入tiles[i]循环之前确保[j]级别作为数组存在,并且然后不用担心在踩j时再次检查它是否存在。


其他选项

根据您的数据集的内容,或者至少您希望对tiles数组做什么,可能值得考虑使用对象:

/// set up
tiles = {};

/// assignment
tiles[i+','+j+','+k] = 'value';

然而,这种方法可能会慢一些,尽管我的假设和不同的JavaScript解释器已被多次证明是错误的。这可能是jsPerf将是你的朋友。


优化

使用tiles[i][j][k]方法的一个好处是,它为您提供了优化引用的机会。例如,如果您要在多维数组的一个级别处理许多操作,则应该这样做:

/// set up
var ij = tiles[i][j];

/// use in loops or elsewhere
ij[k] = 'value'

如果你不止一次访问同一级别,这只会带来好处。