javascript - 简化语法，如何避免重复 "if"？

Question

我想在字段出错时检查表单并酌情禁用提交按钮。我找到了一个可行的解决方案，但语法很丑陋。 :)

$(document).ready(function() {
    $("#send").attr("disabled", "disabled");
    $("#validatemail").keyup(function()
    {
        var email = $("#validatemail").val();
        if (email != '')
        {
            if(isvalidmail(email))
            {
                // do stuff
            }
            else
            {
                // do stuff
            }
        }
        else
        {
                // do stuff
        }
    });
    $("#validatetitle").keyup(function()
    {
        var subject = $("#validatetitle").val();
        if (subject != '')
        {
            if ((subject.length < 3) || (subject.length > 30))
            {
                // do stuff
            }
            else
            {
                // do stuff
            }
        }
        else
        {
                // do stuff
        }
    });
    $("#form_message").live('keyup', function()
    {
        // Duplicate is not smart!!!
            var email = $("#validatemail").val(); // Duplicate
            var subject = $("#validatetitle").val(); // Duplicate
            if (
            (isvalidmail(email)) // Duplicate
            &&
            (!((subject.length < 3) || (subject.length > 30))) // Duplicate
            )
            {       
                $("#send").removeAttr("disabled");  
            }
            else
            {
                $("#send").attr("disabled", "disabled");                
            }
    });
});

那么如何简化代码呢？

我花了几个小时尝试，但没有成功。

非常感谢。

问候，

文森特

Answer 1

试试这个。

---

$(document).ready(function() {
    var isEmailValid = false;
    var isSubjectValid = false;
    $("#send").attr("disabled", "disabled");
    $("#validatemail").keyup(function()
    {
        var email = $("#validatemail").val();
        if (email != '')
        {
            if(isvalidmail(email))
            {
                // do stuff
                isEmailValid = true;
            }
            else
            {
                // do stuff
                isEmailValid = false;
            }
        }
        else
        {
                // do stuff
        }
    });
    $("#validatetitle").keyup(function()
    {
        var subject = $("#validatetitle").val();
        if (subject != '')
        {
            if ((subject.length < 3) || (subject.length > 30))
            {
                // do stuff
                isSubjectValid = true;
            }
            else
            {
                // do stuff
                isSubjectValid = false;
            }
        }
        else
        {
                // do stuff
        }
    });
    $("#form_message").live('keyup', function()
    {
            // now just get all the set values here
            $("#send").attr("disabled",(isEmailValid && isSubjectValid) ? "" : "disabled");
    });
});