求大量范围型if else代码的优化思路

[weixin_39354423]

最近写了一段根据滚轮值修改路由的代码。
十分臃肿，代码全是重复的if else 只有参数的下标一直在递增，请问有什么办法缩减优化吗？？
代码片段如下：
if (scrollValue < scrollArr[0].split(':')[1]) {
if (this.props.match.params.anchor != scrollArr[0].split(':')[0]) {
this.props.history.push('/accountManage/' + scrollArr[0].split(':')[0]); // 0
}
} else if (scrollValue < scrollArr[1].split(':')[1]) {
if (this.props.match.params.anchor != scrollArr[1].split(':')[0]) {

this.props.history.push('/accountManage/' + scrollArr[1].split(':')[0]); //1
}
} else if (scrollValue < scrollArr[2].split(':')[1]) {
if (this.props.match.params.anchor != scrollArr[2].split(':')[0]) {

this.props.history.push('/accountManage/' + scrollArr[2].split(':')[0]); //2
}
} else if (scrollValue < scrollArr[3].split(':')[1]) {
if (this.props.match.params.anchor != scrollArr[3].split(':')[0]) {
this.props.history.push('/accountManage/'+ scrollArr[3].split(':')[0]); // 3
}

} else if (scrollValue < scrollArr[4].split(':')[1]) {
if (this.props.match.params.anchor != scrollArr[4].split(':')[0]) {
this.props.history.push('/accountManage/'+ scrollArr[4].split(':')[0]);// 4
}
}else ..............................

[weixin_39354423]

for(var i =0;i<scrollArr.length;i++){ var menu = scrollArr[i].split(':')[0]; var nextScroleNum =parseInt(scrollArr[i].split(':')[1]); if(scrollValue<nextScroleNum){ if(this.props.match.params.anchor != menu ){ this.props.history.push('/accountManage/' +menu); break; } } } 用for来搞似乎也不对。小弟初学比较菜，就大神指点一下

[潇湘忆梦]

用for循环没有问题的，或者提取出来一个函数调用也可以的，我用了for循环

 for (var i = 0; i < scrollArr.length; i++)
        {
            var menu = scrollArr[i].split(':')[1];
            var nextScroleNum = scrollArr[i].split(':')[0];
            if (scrollValue < menu) {
                if (this.props.match.params.anchor != nextScroleNum) {
                    this.props.history.push('/accountManage/' + nextScroleNum);    // 0
                    continue;
                }
            }
        }

[weixin_39354423]

引用 2 楼 潇湘忆梦 的回复:

用for循环没有问题的，或者提取出来一个函数调用也可以的，我用了for循环

 for (var i = 0; i < scrollArr.length; i++)
        {
            var menu = scrollArr[i].split(':')[1];
            var nextScroleNum = scrollArr[i].split(':')[0];
            if (scrollValue < menu) {
                if (this.props.match.params.anchor != nextScroleNum) {
                    this.props.history.push('/accountManage/' + nextScroleNum);    // 0
                    continue;
                }
            }
        }

大佬我实际运行中似乎有问题，这个方法是放在滚轮监听事件中的，每次滚轮值变动都会去跑这个for，但是调试的时候发现每次跑完一个循环，即使我没有再触发这个事件，这个滚轮值会一直在增加。很是奇怪

[J_Vernon]

https://www.runoob.com/jsref/jsref-foreach.html

[weixin_39354423]

目前我用了新的办法，已经解决了。 改造如下： for(var i =0;i<scrollArr.length;i++){ var menu = scrollArr[i].split(':')[0]; var nextScroleNum =parseInt(scrollArr[i].split(':')[1]); if(scrollValue<nextScroleNum){ this.goHash(menu).bind(this); } } 当if条件满足时，直接去执行另一个函数。这样就不会继续循环下去了。。。并不知道原理。。

[weixin_39354423]

引用 4 楼 J_Vernon 的回复:

https://www.runoob.com/jsref/jsref-foreach.html

大佬你好，我这个是范围型的if判断，不是单一的某个值，用遍历的方式去处理应该不对吧。

[J_Vernon]

用foreach循环遍历，https://www.runoob.com/jsref/jsref-foreach.html
但是这个地址只是一个思路，你也可以在网上查找foreach的其他写法