Because a single element doesn't form an array. Also your loop is potentially skipping array elements. You can do it like this:
Originally Posted by bdmovies
or you could use Array.push.
for (var i=0, j=0; i < theForm.servee.length; i++)
if(theForm.servee[ i ].checked == true)
serviceID[ j++ ] = theForm.servee[ i ].value;
serviceID = theForm.servee.value;
Where used, return should be executed unconditionally and always as the last statement in the function.
That's my signature, it's not part of the damn post!