首页 / JavaScript

不要纠结于实现的圈套中

小小子，simaopig 2010-07-22 20:05:56 浏览 2,341 次

标签 DOM 闭包

有的时候，过于钻牛角尖并不是什么好事，只会把自己越套越牢。换个思路，其实方法很简单。近来忙着改一些东西，需求来的时候真是锐不可挡。被越来越多的任务压的喘不过气来。

今天在写一个适应于所有游戏充值页面的js，希望大多数页面可以共用一套js，方便维护。在这个过程中，遇到了如下问题：

首先写实现如下：

var sndaRadios = document.getElementsByName("snda_amount");
for (var i = 0, len = sndaRadios.length; i < len; i++) {
sndaRadios[i].onclick = function(){
changePaytype('pay_snda', this.value);
}
}

var junnetRadios = document.getElementsByName("junnet_amount");
for (var i = 0, len = junnetRadios.length; i < len; i++) {
junnetRadios[i].onclick = function(){
changePaytype('pay_junnet', this.value);
}
}

var szxRadios = document.getElementsByName("szx_amount");
for (var i = 0, len = szxRadios.length; i < len; i++) {
szxRadios[i].onclick = function(){
changePaytype('pay_szx', this.value);
}
}

其实这都是一样的东西，无非就是每个radio组的名字不同。所以我想用一个数组循环来做：

var payTypes = ['snda', 'junnet', 'szx'];
for (var i = 0, len = payTypes.length; i < len; i++) {
var Radios = document.getElementsByName(payTypes[i] + "_amount");
for (var j = 0, rLen = Radios.length; j < rLen; j++) {
Radios[j].onclick = function(){
changePaytype('pay_' + payTypes[i], this.value);
}
}
}

但是这样又会发现页面报错，因为在写onclick的时候，其实是做了一个闭包，这时候的payTypes[i]其实是payTypes[3]即undefined。

难道要这样？

var payTypes = ['snda', 'junnet', 'szx'];
for (var i = 0, len = payTypes.length; i < len; i++) {
var Radios = document.getElementsByName(payTypes[i] + "_amount");
for (var j = 0, rLen = Radios.length; j < rLen; j++) {
Radios.type = i;
Radios[j].onclick = function(){
changePaytype('pay_' + this.type, this.value);
}
}
}

不过为DOM元素新建属性，怎么感觉都是怪怪的。

于是，纠结于此现象中无法自拔，苦苦思索怎么使用这种结构，却不破坏DOM对象的属性。头发都抓掉了好几根，仍然无果。

出去

建议继续学习

PHP Simple HTML DOM Parser 是一个不错的html/xml分析类（阅读 6,943）
for 循环为何可恨？（阅读 5,383）
理解Javascript的闭包（阅读 4,746）
GC与JS内存泄露（阅读 4,603）
仅100行的JavaScript DOM操作类库（阅读 4,381）
phpQuery:像jQuery一样处理DOM （阅读 4,363）
JavaScript的闭包问题（阅读 4,304）
获取Dom元素的X/Y坐标（阅读 4,040）
对大量子节点DOM操作的最佳实践方式（阅读 3,982）
在回调和闭包中的内存泄漏（阅读 3,983）