V2EX = way to explore V2EX 是一个关于分享和探索的地方
Sign Up Now For Existing Member Sign In
推荐关注 Meteor JSLint - a Javascript code quality tool jsFiddle D3.js WebStorm 推荐书目 Javascript 权威指南第 5 版
Closure: The Definitive Guide
Closure: The Definitive Guide 
This topic created in 3570 days ago, the information mentioned may be changed or developed.
function oneMenu() {
var main = document.getElementById("oneMenu");
var links = main.getElementsByClassName("cc-tree-item");
for (var i = 0; i < links.length; i++) {
links[i]. Onclick= function () {
addClass(this, "cc-selected cc-focused")
}
}
}
想在点击一个元素时，删除其他同级元素的 "cc-selected cc-focused" class 。现在点击能追加样式，可不能删除
var main = document.getElementById("oneMenu");
var links = main.getElementsByClassName("cc-tree-item");
for (var i = 0; i < links.length; i++) {
links[i]. Onclick= function () {
addClass(this, "cc-selected cc-focused")
}
}
}
想在点击一个元素时，删除其他同级元素的 "cc-selected cc-focused" class 。现在点击能追加样式，可不能删除
13 replies 2016-07-19 13:46:40 +08:00
 | 1 xjp Jul 18, 2016 ``` links[i]. Onclick= function () { addClass(this, "cc-selected cc-focused") }; ``` 这里的 this 指向的是全局 而不是你希望的 links[i]，这里弄个闭包吧 |
 | 2 narcotics Jul 18, 2016 循环中创建闭包是最常见的 Javascript 大坑之一，参见 https://developer.mozilla.org/cn/docs/Web/Javascript/Closures#Creating_closures_in_loops_A_common_mistake 另外，你这代码哪儿有 「删除样式」的逻辑？ |
 | 3 adv007 Jul 18, 2016 via iPhone 实现这种完全不需要遍历元素啊，楼主在动动脑子。你只需要记录一个当前高亮节点，并且在单击的时候重新指向新的当前节点 |
 | 4 YuJianrong Jul 18, 2016 via iPhone 1 ， 2 楼确定？我记得时间处理函数的 this 就是 node 元素， lz 用法并没有问题。 |
| 5 narcotics Jul 18, 2016 @YuJianrong 是的，我的错~看到循环闭包就觉得有问题，这里的 this 没问题 |
 | 6 sjw199166 Jul 18, 2016 因为你没有清除 ；你在上面加一行 清楚所有那个的样式 |
| 7 sjw199166 Jul 18, 2016 links[i]. Onclick= function () { //这个去除所有的按钮样式 addClass(this, "cc-selected cc-focused") } |
 | 8 iTakeo Jul 18, 2016 via iPhone 这里的 this 并没有问题 |
 | 9 iyangyuan Jul 18, 2016 via iPhone 没看到清除的代码 |
 | 10 LancerComet Jul 18, 2016 没有删除样式的代码 楼主可以递归 this 的 previousElementSibling 跟 nextElementSibling ，或者找 parentNode 的 children |
 | 11 rain0002009 Jul 18, 2016 点击高亮么？ 只要结构好 js 都不用写 纯 css+html 的 radio 标签就可以实现 |
 | 12 yiyizym Jul 18, 2016 links[i]. Onclick= function () { //先删除所有 [].slice.call(links).forEach(function(link){link.classList.remove('cc-selected', 'cc-focused')}); addClass(this, "cc-selected cc-focused") } |
Select Language