Javascript 表单验证不起作用

Question

我正在用 JavaScript 编写表单验证脚本。提交表单后，我希望在进入下一页之前对其进行验证。

正在使用 Perl Interchange 从另一个页面调用此页面。对表单上的三个字段执行验证。

---

更新：这是完整的代码：

    <FORM  ACTION="[process]" name="outofstock_form" METHOD=POST onsubmit="return      validate_outofstockform();" >
    <INPUT TYPE=hidden NAME="mv_todo"  VALUE="return">
    <INPUT TYPE=hidden NAME="mv_nextpage" VALUE="outofstock_wish_submit">
    <INPUT TYPE=hidden VALUE="[perl scratch session]$which_search;[/perl]" NAME="search_key">

    <script type=javascript>
        function validate_outofstockform() {
            var m = document.forms["outofstock_form"]["email"].value
            var e = document.outofstock_form.email.value
            var f = document.forms["outofstock_form"]["name"].value
            var p = documnet.forms["outofstock_form"]["wish_product"].value

            var atpos = e.indexOf("@");
            var dotpos = e.lastIndexOf(".");

            if (document.outofstock_form.email.value == "") {
                alert("The Email field is required.");
                return false;
            }
            if (document.outofstock_form.name.value == "") {
                alert("The Name field is required.");
                return false;
            }
            if (document.outofstock_form.wish_product.value == "") {
                alert("The Product field is required.");
                return false;
            }
            if (atpos < 1 || dotpos < atpos + 2 || dotpos + 2 >= e.length) {
                alert("Please enter a valid e-mail address");
                return false;
            }

            if (f == null || f == "" || f == "First Name") {
                alert("Please enter your first name");
                return false;
            }
            if (p == null || p == "" || p == "Product") {
                alert("Please enter your first name");
                return false;
            }
            return false;
        }
    </script>

    <br/>
    *Fields in bold are required.<br/>

    <table cellpadding="1" cellspacing="5" width="360px" border="0">
        <tr>
            <td><b>Name:</b></td>
            <td>
                <input type="text" id="name" name="name" size="40">
            </td>
        </tr>
        <tr>
            <td><b>E-mail:</b></td>
            <td>
                <input type="text" id="email" name="email" size="40">
            </td>
        </tr>
        <tr>
            <td>Phone:</td>
            <td>
                <input type="text" name="phone" size="40">
            </td>
        </tr>
        <tr>
            <td> State/ Province:</td>
            <td>[include pages/ord/widget_state.html]</td>
        </tr>
        <tr>
            <td > Zip/Postal Code:</td>
            <td><INPUT TYPE="text" NAME="zip" VALUE="" size="40" maxlength="10"></td>
        </tr>

        <br/>

        <tr>
            <td valign="bottom">Country:</td>
            <td>[include pages/ord/widget_country_s.html]</td>
        </tr>
        <tr>
            <td>&nbsp;</td>
            <td>&nbsp;</td>
        </tr>
        <tr>
            <td>Salesperson:</td>
            <td align=left colspan=2>
                <SELECT NAME="salesrep" class="field">
                    <OPTION VALUE="WEB">(Optional)
                    [query list=1 prefix=s sql="SELECT id AS username, real_name AS disp_name, everest_id AS int_id FROM employee WHERE sales_ddown = 'Y' AND everest_id != '' ORDER BY real_name"]
                    <OPTION VALUE="[s-param int_id]"[calc]'[value salesrep]' eq '[s-param int_id]' ? 'selected' : '';[/calc]>[s-param disp_name]
                    [/query]

                </SELECT>
            </td>    

            <INPUT TYPE=hidden NAME="salesperson" VALUE="[s-param username]">
            [perl values scratch]
            $Scratch->{salesperson} = q{[s-param username]};
            [perl]   

        <tr>
            <td colspan="2">
            Provide us with the product you are looking for, or the brand and product type
            of interest and we will inform you if we find a match.
            </td>
            <td></td>
        </tr>

        <tr>
            <td><b>Product:</b></td>
            <td>
                <input type="text" id="wish_product" name="wish_product" size="40" value="">
            </td>
        </tr>
        <tr>
            <td>Item Description:</td>
            <td>
            <textarea name="wish_descrip" rows="2" cols="40"></textarea>
            </td>
        </tr>

        <tr>

        <tr>
            <td>Brand/Manufacturer Preference:</td>
            <td><input type="text" name="wish_man" size="40"></td>
        </tr>
        <tr>
            <td>Product Category :</td>
            <td>
            <select name="wish_cat">
            <option value="" selected>Any Category</option>
            [include pages/CATLIST.html]
            </select>
            </td>
        </tr>
        <tr>
            <td>Is this for a business?:</td>
            <td>
            <input type="radio" name="option" value="Yes"> Yes
            <input type="radio" name="option" value="No"> No<br>
            </td>
        </tr>
        <tr>
            <td>&nbsp;</td>
            <td>&nbsp;</td>
        </tr>
        <tr>
            <td colspan="2"><font size="0px">
            We do not sell, rent or otherwise share your information with anyone.<br/>
            </font>
            </td>
            <td></td>
        </tr>
        <tr>
            <td>&nbsp;</td>
            <td>
            <input type="submit" name="Submit" value="Submit" class="button">
            </td>
        </tr>

    </table>

</form>

Answer 1

使您的 JavaScript 有效（删除带有多个破折号的部分）并使其返回 false 以避免发送表单。

Answer 2

很难判断到底是什么导致了问题，因为您的代码中有几个错误。

一些提示：

• 您正在引用 document.outofstock_form，而表单的名称为 frm。
• 不是有效的 HTML。删除它或将其替换为 HTML 注释（例如：）。
• 更常见的是使用正则表达式（在此处搜索 SO）来验证表单输入。
• 如果您要将输入发送到服务器，请也在服务器上执行验证。永远不要相信来自浏览器的输入。
• 您从未打开 （可能是复制粘贴错误）。

最后，但也是最重要的一点：

• 始终首先查看浏览器的 JavaScript 错误控制台。这必须是您调试 JavaScript 代码时的起点。它可以帮助您找到问题，如果没有，它可以帮助其他人帮助您。
• 阅读如何提问。

Answer 3

你调试过你的代码吗？

使用 Firebug 等工具来调试脚本。在验证函数的第一行放置断点，然后逐步调试。您最终会遇到导致问题的线路。

关于如何改进验证器的建议

但除了调试之外，我还会以不同的方式进行验证。我不想检查各个字段的每个方面，而是只是向需要验证的输入添加自定义属性，然后检查它们是否匹配。如果任何失败，通知您的用户有关无效字段的信息...

<input name="SomeName"
       validation-expression=".+"
       display-name="Required text field"
       type="text" />

在处理此类数据时，使用像 jQuery 这样的库会更有帮助，因为枚举这些元素并处理它们的数据会更容易...我当然会强烈建议您无论如何都使用 jQuery，因为它会让您的代码更加跨浏览器。这是一个简单的库，学习曲线短，但好处巨大。

但是使用这些特殊属性将使您的验证器函数具有通用性，因此它可以与任何元素和任何页面一起使用。您所要做的就是将特定的验证属性添加到您的元素中。

当然只是一个建议。