Check form value

Hvad er der galt med følgende stykke kode?

<html>
<head><title>Untitled</title></head>

<script language=\"JavaScript\">

var formnavne = new Array(\"fnavn\", \"enavn\", \"adr\", \"bynavn\", \"pnr\");

function checkform()
{
for (var i = 0; i < formnavne.length; i++)
{
    if(eval(document.bestilform.formnavne[i].value==\"\"))
    {
        alert(\"Alle felter skal være udfyldte før din bestilling kan godkendes\");
        return false;
    }
  else
    {
    document.bestilform.submit();
    return true;
    }
}
}

</script>

<body>

                <form name=\"bestilform\" method=\"get\" action=\"gem_bestilling.asp\">
               
                <table border=\"0\" width=\"700\" align=\"center\">
                <tr>
                <td width = \"100\">Fornavn</td><td><input type=\"text\" name=\"fnavn\" value=\"\" size=\"37\"></td>
                </tr>
                <tr>
                <td width = \"100\">Efternavn</td><td><input type=\"text\" name=\"enavn\" value=\"\" size=\"37\"></td>
                </tr>
                <tr>
                <td width = \"100\">Adresse</td><td><input type=\"text\" name=\"adr\" value=\"\" size=\"37\"></td>
                </tr>
                <tr>
                <td width = \"100\">By</td><td><input type=\"text\" name=\"bynavn\" value=\"\" size=\"37\"></td>
                </tr>
                <tr>
                <td width = \"100\">Postnr</td><td><input type=\"text\" name=\"pnr\" value=\"\" size=\"8\"></td>
                </tr>
                </table>
               
                <table border=\"0\" width=\"700\" align=\"center\">
                <tr>               
                <td width = \"350\">Ja tak</td><td><input type=\"checkbox\" name=\"valg\" value=\"ok\"></td>
                </tr>                                                       
                </table>
               
                <table border=\"0\" width=\"700\" align=\"center\">
                <tr>
                <td width = \"100\" align=\"center\"><input type = \"button\" value = \"OK\" onclick = \"checkform()\" ></td>
                </tr>
                           
                </form>
</body>
</html>