完成得很出色,基本实现了任务要求,并实现了简单的插件封装,如果可以完善以下几点将会更好:

1. 对于频繁获取的节点应该缓存到一个变量里,而不是每次重新获取,在DOM树里查找节点是很消耗资源的,应该尽可能缓存

  $(this).siblings().removeClass("active");
  $(this).addClass("active");
  i = $(this).index();

更改为:

  var $this = $(this);
  $this.siblings().removeClass("active");
  ...

2. 该插件无法实现多用,即一个页面仅允许有一个轮播图,如果有多个将出现错乱. 功能结构比较简单,插件内部将轮播图元素给写死了,例如引用插件的方式

$(".show").carousel();

改成

$(".show2").carousel();

也同样可以运行,即插件内部的this并没有指向元素节点$(".show")或$(".show2").这一部分可能比较难理解,如果可以解决这些问题对你学习JS一定会有很大帮助.

作为第一次任务完成的还行，以下是修改建议：

1. 页面乱码，请设置编码格式
2. 样式过于简单，布局比较乱，输入框/按钮等缺少样式
3. 注意HTML格式， meta标签应位于head标签内，标签应该有合理的缩进
4. 尽可能少使用实体，用样式来控制边距缩进
5. document.getElementById("ico").title = "显示密码";
   document.getElementById("ico").src="ico.png";
   连续两次在同一个函数中获取了id=ico的节点，应该将该节点赋给一个变量，操作变量即可，而多次查找节点比较消耗性能。

登录界面

• 不符合题目要求，并没有使用boots框架
• 内容太过于简单

学生管理系统

• 在功能上并没有显示出所有学生的选项，此外，所有操作都在一个页面上显得太乱了，建议使用标签页
• 操作符之间应该有适当的空格，注意代码缩进规范

name=document.getElementById("n").value;
		sex=document.getElementById("s").value;
		id=document.getElementById("i").value;
		grade=document.getElementById("g").value;
		pro=document.getElementById("p").value;
		var stu=new Array();
		stu[0]=name;stu[1]=sex;stu[2]=id;stu[3]=grade;stu[4]=pro;
		localStorage.setItem(id,stu);

• 建议不要把单个学生信息保存为数组通过索引来表示不同的字段，而应该保存为对象，通过键值对来记录学生信息

stu[0]=name;stu[1]=sex;stu[2]=id;stu[3]=grade;stu[4]=pro;