应该是对那个对象的值用indexOf，而不是对那个对象用indexOf所以应该是..document.getElementById("mail").value.indexOf('@')==-1
document.getElementById("mail").indexOf('@')==-1 意思就是 mail 这个对象的值里面没有包含‘@’这个字符。是否有误你自己测试一下就知道了
