作者B9 (叶酸酸)
看板Ajax
标题Re: [问题] 新手绕不出来的Javascript 问题
时间Sat Jul 30 13:30:09 2011
※ 引述《phreat (雷)》之铭言：
: 这是程式,主要目的是希望能抓取值
: 尝试过将k改成数字是可行的但因为会跑回圈所以放在里面
: 怎麽试都不行 希望有大大指点迷津一下
: <SELECT id="test1" onchange="test('test1')">
: <option value="100">Ａ</option>
: <option value="200">Ｂ</option>
: <option value="300">Ｃ</option>
: </SELECT>
: <SELECT id="test2" onchange="test('test2')">
: <option value="100">Ａ</option>
: <option value="200">Ｂ</option>
: <option value="300">Ｃ</option>
: </SELECT>
: for (i=1; i<=2; i=i+1)
: {
: k=String(i);
: alert(k);
: cc=Number(document.getElementById("test"+k)[document.getElementById("test"+k).selectedIndex].value);
: }
http://jsbin.com/osatuf
请问这是你要的结果吗？触发 select element 的 change 时间可以看到抓到的数值。
我改的地方只有：
多了 test function，这是你在 select.onchange 里面会 call 的 function。
我有几个小建议：
- namespace 的建议:
你这段程式码会在 global namespace 留下几个变数 i, k, cc, test，
在 global namespace 里面留下的足迹越少越好。方法就是用 var 宣告这些变数。
例如： function test() { var i, k, cc; }，这样就只会有一个 test。
甚至更好的方法是
(function()
{
用 getElementById 抓 html element；
抓到的element.onchange = function(){ 做点事情 };
})()
这样的写法一点也不会污染 global namespace。
- styling 的建议:
cc=Number(document.getElementById("test"+k)[document.getElementById("test"+k).selectedIndex].value);
拆开比较好，我会这样写：
element = document.getElementById("test"+k);
selected = element.selectedIndex;
value = element[selected].value;
cc = value;
可读性好很多，或者我自以为好很多：P
- loose coupling 的建议：
你的 html 跟 js 太亲密了。我指把 js 放到 html 的 attribute 里面这件事情。
例如： onchange="whatever()"
假设将来这份 js 要给其他 html 档案用，那你现在就有两个 html，
那你做好 html 档案之後，还要一个一个 element 去加入 onchange="whatever()"
过了一段时间你不想要这个功能了，你把 js 丢掉，然後就要打开这两个 html 档案，
一个一个把 onchange attribute 拿掉。
- type conversion 的建议：
k=String(i); 是不必要的，因为之後的 "test"+k 会自动把 k 转成 string，
然後跟 "test" 结合。
cc=Number( value )
这我没有意见，不过有个小秘诀：
cc = value - 0 或 cc = +value 都可以做 type conversion，
可以少打几个字不知道有没有吸引力。XD
- performance 的建议：
document.getElementById("test"+k)[document.getElementById("test"+k).selectedIndex].value
document.getElementById("test"+k) 被执行两次，其实只要执行一次就可以了。
如果其实你有注意到这里的效能可以改善，那你应该留个 comment 例如：
/* 这里效能可以更好。 */
--
Oni devas ami animalojn. Ili estas tiel bongustaj.
One should love animals. They are so tasty.
每个人都应该爱动物，他们是如此美味。
--
※ 发信站: 批踢踢实业坊(ptt.cc)
◆ From: 175.180.52.143
※ 编辑: B9 来自: 175.180.52.143 (07/30 13:30)
1F：推 phreat:不愧是神人级小弟刚接触jsc所以很多小细节不够拘谨谢谢提醒 07/30 15:06