javascript - 自己写了一个图片轮播，能不能给我指明一下问题在哪？

Question

https://github.com/gonewithfa...

这是我的github地址，图片文件夹没有放上去。有没有人能指明一下js代码可以怎样的去改进？

自学了两个半月，上面的图片轮播可以算是我目前js最高的水平了，最近看了一下underscore源码发现根本看不懂.....大二学生，希望有人能指导一下，学习方向之类的问题，简单点一两句就很感谢了

Answer 1

我大二也和你差不多，现在大三，稍微好一点了。没有什么技巧，只要你喜欢这行，可以一直学下去，渐渐的知道的就多了。

Answer 2

不要急，大二的时候 CSS 我都没听过

如果要学习 JavaScript，建议看高程，以前学的时候看过太多烂书 ...

Answer 3

图片最好按需加载，而且这么写不会有抖动吗，图片全部是在window.onload里加载的，而且之前没有图片站位，大二以及很不错了啊，我大二的这个时候连github都不会用，多看看基础，有机会出来实习下就好啊，欢迎来review下一个大三狗写的轮播图

Answer 4

轮播图？轮播图，其实是把几张图不停的轮换显示在某一特定区域。要是使用原生语法手写的话，给个建议，下面是步骤：
1、定时器，让里面的函数能定时执行，就能够实现动起来啦
2、定时器里面的函数，就是更换显示出来的那个区域标签代码
3、或者，使用切换隐藏和显示语法也行
下面是个示例：

//自动轮播
setInterval(function(){
    // 显示出来图片的序号
    var indexDantip = imagesLi();
    indexDantip++;
    // 当到底了
    if (indexDantip > coun) {
        indexDantip = 1;
    }
    prependLi(indexDantip);
},5000);

//取得当前显示的对象
function imagesLi () {
    // 选择出备选图片子元素
    var obj = $(".banbox_pic li:first-child");
    var lid = obj.attr("lid");
    var index = lid.split("_")[1];
    // 图片序号
    return index;
}

//设置需要显示的对象
function prependLi (indexDantip) {
    // 拼接标签
    var obj = $("li[lid=li_"+indexDantip+"]");
    // 设置标签
    $(".banbox_pic").prepend(obj);
}