问题描述

请问我这段代码规范吗，有什么语法错误

效果达到了 但是BOSS说我代码不规范

解决方案

你的定时器放在了chgCol方法内，

方法在外部调用时，会先执行chgCol方法 然后再创建一个定时器，然后在chgCol被调用时再一次创建一个定时器，每个定时器都是新一个定时器，并且在第一次执行时 调用了一次方法

应该将定时器放到外围，如果你要用无限循环 请用setInterval()定时器。

解决方案二：

pos初始值为3，那if里面就没必要判断pos<0了，你的方法运用递归调用来实现循环，没有语法错误。至于规范不规范，就仁者见仁智者见智了。

解决方案三：

pos没有定义啊，给pos一个适当的注释

解决方案四：

p++那里有什么问题吗

解决方案五：

是的，你定时任务写在方法里面了，这样会无限循环啊，用setInterval()定时器吧

解决方案六：

建议写个内部变量区分开可能带来的全局变量冲突，传入形参pos,内部var pos=pos || 0;适当修改下下面的写法，尽量封装成一个高可用的函数