hvad er det galt i dette simple script?

Jeg har fået det til at virke, her er det færdige:

function subm(thisform) {

    inputStr1 = thisform.elements[0].value;
    if (inputStr1 <= 0 || inputStr1 >=30000) {
    alert("Du skal skrive et tal mellem 0 - 30.000")
    return false; }
    else {
    return true; }
   
}