推荐关注
› Meteor
› JSLint - a JavaScript code quality tool
› jsFiddle
› D3.js
› WebStorm
推荐书目
› JavaScript 权威指南第 5 版
› Closure: The Definitive Guide
› Closure: The Definitive Guide
这是一个创建于 3481 天前的主题，其中的信息可能已经有所发展或是发生改变。
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 条回复 • 2016-07-19 13:46:40 +08:00
 |
1
xjp 2016 年 7 月 18 日
```
links[i]. onclick = function () { addClass(this, "cc-selected cc-focused") }; ``` 这里的 this 指向的是全局 而不是你希望的 links[i]，这里弄个闭包吧 |
 |
2
narcotics 2016 年 7 月 18 日
循环中创建闭包是最常见的 JavaScript 大坑之一，参见
https://developer.mozilla.org/cn/docs/Web/JavaScript/Closures#Creating_closures_in_loops_A_common_mistake 另外，你这代码哪儿有 「删除样式」的逻辑？ |
 |
3
adv007 2016 年 7 月 18 日 via iPhone
实现这种完全不需要遍历元素啊，楼主在动动脑子。你只需要记录一个当前高亮节点，并且在单击的时候重新指向新的当前节点
|
 |
4
YuJianrong 2016 年 7 月 18 日 via iPhone
1 ， 2 楼确定？我记得时间处理函数的 this 就是 node 元素， lz 用法并没有问题。
|
|
5
narcotics 2016 年 7 月 18 日
@YuJianrong 是的，我的错~看到循环闭包就觉得有问题，这里的 this 没问题
|
 |
6
sjw199166 2016 年 7 月 18 日
因为你没有清除 ；你在上面加一行 清楚所有那个的样式
|
|
7
sjw199166 2016 年 7 月 18 日
links[i]. onclick = function () {
//这个去除所有的按钮样式 addClass(this, "cc-selected cc-focused") } |
 |
8
iTakeo 2016 年 7 月 18 日 via iPhone
这里的 this 并没有问题
|
 |
9
iyangyuan 2016 年 7 月 18 日 via iPhone
没看到清除的代码
|
 |
10
LancerComet 2016 年 7 月 18 日
没有删除样式的代码
楼主可以递归 this 的 previousElementSibling 跟 nextElementSibling ，或者找 parentNode 的 children |
 |
11
rain0002009 2016 年 7 月 18 日
点击高亮么？
只要结构好 js 都不用写 纯 css+html 的 radio 标签就可以实现 |
 |
12
yiyizym 2016 年 7 月 18 日
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