因为你的if else依次执行了
当你if条件ok的时候 login的style设置为block，此时，else条件又成立，所以依次执行，回到原点。
方法1：
使用jquery的toggle切换函数
login
$("#login").toggle(
function () {
$(this).show();
},
function () {
$(this).hide();
}
);
方法2：原始方法
设置全局标记位
var flag = false;
if(flag){
login.style.display="block";
flag=false;
}
else{
login.style.display="none";
flag=true;
}
