sname = document.S.newname.value();
改成
sname = document.S.newname.value;value 不是方法,是属性