【问题标题】:Instantiating class in loop in javascript uses last value在javascript中循环实例化类使用最后一个值
【发布时间】:2013-07-03 11:45:25
【问题描述】:

首先,我知道在 JavaScript 中存在许多关于闭包的问题,​​尤其是在循环方面。我已经阅读了其中的许多内容,但我似乎无法弄清楚如何解决我自己的特定问题。我的主要经验在于 C#、C++ 和一些 ASM,并且需要一些时间来适应 JavaScript。

我正在尝试在某些 for 循环中使用类的新实例(称为 Tile)填充 3 维数组。我要做的就是传递对在第一个循环中实例化的其他类(称为 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 时,我通常会稍微修改代码以使其正常工作,但我已经厌倦了使用看起来很混乱的变通办法,因为我知道实际上应该有一些相当简单的解决方案.我只是不喜欢寻求帮助,但这真的让我很头疼。非常感谢任何帮助。

【问题讨论】:

  • 您希望通过tiles[0,0,0] 实现什么目标? -- 这个结构不会像我想的那样做......不是在 JavaScript 中。
  • 首先引起我注意的是“语法”,例如tiles[0,0,0] - 我猜你的意思是tiles[0][0][0][0,0,0] 本身构造一个数组
  • 使用tiles[i]tiles[i][j]tiles[i][j][k]
  • 我猜我使用的符号实际上是其他东西的有效代码,不会引发错误。 爬到角落里抽泣

标签: javascript class loops closures


【解决方案1】:

多维数组

问题

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

你使用的语法是:

tiles[0,0,0]

然而,JavaScript 的解释方式是:

tiles[0]

访问多维数组

如果你想访问一个 multidim 数组,你必须使用:

tiles[0][0][0]

要创建一个多维数组,您需要执行以下操作:

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;

显然,上述内容可以根据您遍历循环的方式进行优化,即最好在进入[j] 循环之前确保tiles[i] 级别作为数组存在,然后不用担心在踩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'

这仅在您多次访问同一级别时才有用。

【讨论】:

  • 哎呀,这确实让我觉得自己很愚蠢。感谢你的回答!当然,它总是你不看的东西:)
  • @Sizzling 不用担心——从其他语言翻译过来时很容易出错 :) 不得不说 [0,0,0] 语法完全让我困惑了一段时间......呵呵,无论如何希望它有帮助。
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2013-10-30
  • 1970-01-01
  • 2021-11-23
相关资源
最近更新 更多