Hey @willaeng
I would start off by not calling on value twice. You call on value here:
var userId = document.myForm.userName.value;
and again here:
var userId_len = userId.value.length; ... else if (userId.value.match(alphaNumber)) {Try removing the extra calls on .value and then go from there.