Question

以下代码的目标是创建一个二维数组，使其

Clip是一个自定义对象
银行是一组（8）个剪辑
Banks是一组（8）银行
银行[a] [b]可以访问每个剪辑，其中a是银行中的索引（银行），b是剪辑中的索引（剪辑）

在当前状态下，它返回undefined，唉，我无法解释原因。任何关于我做错的建议都将不胜感激

var banks = []

function Clip(a,b)

{
this.track = a
this.slot = b
}

function Bank(w)

{

for (var j, j = 0; j <= 7; j++) {
    var clips = []
    var aClip = new Clip(w,j);
    //post(i)
    //post(aClip.length)
    clips[j] = aClip
}
//post();
return clips
}

function makeBanks()

{

for (var k, k = 0; k <= 7; k++) {
    var aBank = Bank(k);
    //post(i)
    //post (aClip.length)
    banks[k] = aBank
}
}

makeBanks();

console.log(banks[0][0])

非常感谢提前

Answer 1

你最大的错误就在这里（Bank内）：

for (var j, j = 0; j <= 7; j++) {
    var clips = []
    var aClip = new Clip(w,j);
    //post(i)
    //post(aClip.length)
    clips[j] = aClip
}

每次循环都会重新初始化clips，因此唯一保留的值是最后一个（因为循环在你有机会再次覆盖它之前终止）。为了说明这一点，这是第一次调用Bank时返回的内容：

[undefined, undefined, undefined, undefined, undefined, 
     undefined, undefined, Clip { track=0, slot=7}]

在循环之外移动声明解决了这个基本问题：

var clips = [];
for (var j = 0; j <= 7; j++) {
    var aClip = new Clip(w, j);
    clips[j] = aClip;
}
return clips;

额外清理

现在我们已经解决了基本问题，我们可以做更多的事情来清理它：

始终使用分号
除非您打算将它们用作构造函数，否则不要使用大写字母命名函数
始终缩进
不要依赖全局变量
一步声明并分配（即var j, j = 0应为var j = 0）

结果：

function Clip(a, b) {
    this.track = a;
    this.slot = b;
}

function makeBank(w) {
    var clips = [];
    for (var j = 0; j <= 7; j++) {
        var aClip = new Clip(w, j);
        clips[j] = aClip;
    }
    return clips;
}

function makeBanks() {
    var banks = [];
    for (var k = 0; k <= 7; k++) {
        var aBank = makeBank(k);
        banks[k] = aBank;
    }
    return banks;
}

var banks = makeBanks();
console.log(banks[0][0]);

创建自定义对象的二维数组

1 个答案:

额外清理