2

我可以在这里看到警报消息。即使返回 false，我也可以看到我的表单发布它。代码哪里坏了？

function ValidateForm(){
    var productName = document.addProduct.product_name;
    var partNumber = document.addProduct.part_number;
    var description = document.addProduct.description;
    var price = document.addProduct.price;

    var formelements = [productName, partNumber, description, price];
    formelements.forEach(function(obj) {
        if(obj.value=="") {
           obj.style.borderColor = "#FF0000"; 
           alert(obj);
           return false;
        }
    });

我的 HTML 代码

<form action="product_formhandler.php" name="addProduct" id="addProduct" onsubmit="return ValidateForm();" method="post">
    <table>
        <tr>
            <td>
                Product Name:
            </td>
            <td>
                <input type="text" id="product_name" name="product_name" />
            </td>
        </tr>
        <tr>
            <td>
                Part number:
            </td>
            <td>
                <input type="text" id="part_number" name="part_number" />
            </td>
        </tr>
        <tr>
            <td>
                Description:
            </td>
            <td>
                <textarea id="description" name="description" rows="8" col="25"></textarea>
            </td>
        </tr>
        <tr>
            <td>
                Price:
            </td>
            <td>
                <input type="text" id="price" name="price" />
            </td>
        </tr>
        <tr>
            <td colspan="2" style="text-align: center;">

                <input type="reset" id="reset" value="Reset" />
                <input type= "hidden" name="addproduct" value="1" />
                <input type="submit" id="addProductSubmit" name="action" value="Add" />
            </td>
        </tr>
    </table>
</form>
4

3 回答 3

2

该函数ValidateForm没有return声明。您唯一拥有的就是您传递给的匿名函数forEach

  • 在调用之前定义一个变量forEach
  • 给它一个默认值（可能true）
  • 在循环中更改它（具体逻辑将取决于您要查找的内容，但可能if (condition) { retVal = false; }没有 no else）
  • 最后归还

这样的：

var retVal = true;
formelements.forEach(function(obj) {
    if(obj.value=="") {
    obj.style.borderColor="#FF0000"; 
    alert(obj);
    retVal = false;
});
return retVal;

或者，使用传统的 for 循环：

for (var i = 0; i < formelements.length; i++) {
   var obj = formelements[i];
   if (obj.value == "") {
       obj.style.borderColor="#FF0000"; 
       alert(obj);
       return false;
   }
}

尽管一旦发现单个故障，这种方法就会停止。

于 2013-10-31T09:34:38.570 回答
0

你是false从你的forEach循环中返回的，而不是你的验证函数。您需要执行以下操作：

/* Declare variable outside of loop which is set to true by default */
var forEachResult = true;
formelements.forEach(function(obj) {
    if(obj.value=="") {
    obj.style.borderColor="#FF0000"; 
    alert(obj);
    /* Rather than returning false, set the variable to false instead. */
    forEachResult = false;
    /* Break the loop. */
    return;
}
/* Return the variable which will either be true or false. */
return forEachResult;
于 2013-10-31T09:36:41.430 回答
-1 
function ValidateForm(){
    var productName=document.addProduct.product_name;
    var partNumber=document.addProduct.part_number;
    var description=document.addProduct.description;
    var price=document.addProduct.price;

    var formelements =[productName, partNumber, description, price];

    var isValid = true;
    formelements.forEach(function(obj) {
        if(obj.value=="") {
            obj.style.borderColor="#FF0000"; 
            alert(obj);
            isValid = false;
            return false;
        }
    });

    return isValid;
}
于 2013-10-31T09:37:04.660 回答