你代码这样写要实现什么功能?
是要点下按钮切换一个颜色，再点击一下显示另一个，循环往复？
如果想要这样的话,
去掉for循环

oBtn.onclick=function(){
    var color=bgcolor.shift();
    bgcolor.push(color);
    op.style.backgroundColor= color;
    
}

或者

var i=0;
oBtn.onclick=function(){
    if(i>=bicolor.length){
        i=0;
    }
    op.style.backgroundColor= bgcolor[i];
    i++;
}

你的代码需要放在window.onload=function里面啊

修改将for循环中变量i的实时值用闭包传入到onclick中，不然你得到的i全部等于3，所得到的背景色都是red

逻辑不对，for了一圈，最后还是onclick就变成一种红色。其实不需要for，只需每次click 取0到colors.length-1之间的随机数i，然后background-color=colors[i]

同问，我也不懂