帮忙看一段 js 代码

推荐关注

› Meteor

› JSLint - a JavaScript code quality tool

› jsFiddle

› D3.js

› WebStorm

推荐书目

› JavaScript 权威指南第 5 版

› Closure: The Definitive Guide

This topic created in 3764 days ago, the information mentioned may be changed or developed.

13 replies • 2016-07-19 13:46:40 +08:00

xjp

Jul 18, 2016

```
links[i]. onclick = function () {
addClass(this, "cc-selected cc-focused")
};
```

这里的 this 指向的是全局而不是你希望的 links[i]，这里弄个闭包吧

narcotics

Jul 18, 2016

循环中创建闭包是最常见的 JavaScript 大坑之一，参见

https://developer.mozilla.org/cn/docs/Web/JavaScript/Closures#Creating_closures_in_loops_A_common_mistake

另外，你这代码哪儿有「删除样式」的逻辑？

adv007

Jul 18, 2016 via iPhone

实现这种完全不需要遍历元素啊，楼主在动动脑子。你只需要记录一个当前高亮节点，并且在单击的时候重新指向新的当前节点

YuJianrong

Jul 18, 2016 via iPhone

1 ， 2 楼确定？我记得时间处理函数的 this 就是 node 元素， lz 用法并没有问题。

narcotics

Jul 18, 2016

@YuJianrong 是的，我的错~看到循环闭包就觉得有问题，这里的 this 没问题

sjw199166

Jul 18, 2016

因为你没有清除；你在上面加一行清楚所有那个的样式

sjw199166

Jul 18, 2016

links[i]. onclick = function () {
//这个去除所有的按钮样式
addClass(this, "cc-selected cc-focused")
}

iTakeo

Jul 18, 2016 via iPhone

这里的 this 并没有问题

iyangyuan

Jul 18, 2016 via iPhone

没看到清除的代码

LancerComet

Jul 18, 2016

没有删除样式的代码

楼主可以递归 this 的 previousElementSibling 跟 nextElementSibling ，或者找 parentNode 的 children

rain0002009

Jul 18, 2016

点击高亮么？
只要结构好 js 都不用写纯 css+html 的 radio 标签就可以实现

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")
}

ssyz1988

Jul 19, 2016

@adv007 总得有一个取消上一个高亮节点的处理吧？