Могу ли я улучшить свою проверку?

Question

Итак, я хочу проверить свою форму с помощью jQuery, вот как я это себе представляю, и это работает, но это лучший способ сделать это, и, возможно, я могу сделать это более эффективно с меньшим количеством кода?

<script type="text/javascript">
    $(document).ready(function()
    {
        $("#frmCatAdd").submit(function()
        {
            var name = $("#edtName").val();

            if (name == "")
                $("#edtName").css("border", "1px solid red")
            else
                $("#frmCatAdd").submit();

            return false;
        });
    });
</script>

Answer 1

Попробуйте взглянуть на jValidate , чтобы упростить и ускорить выполнение вашей задачи.Я использую его так же регулярно, как и при создании новых форм.

Однако я настоятельно рекомендую проверять и на стороне сервера.Все, что пользователь должен сделать, это отключить Javascript, чтобы победить ваши идеальные правила проверки.

Answer 2

Одна вещь, которую вы могли бы улучшить, это избавиться от бесконечного цикла.Позвонив .submit() из .submit(), вы сделали цикл.

В этом случае вы можете использовать собственный submit.

$("#frmCatAdd").submit(function() {
    var name = $("#edtName").val();

    if (name == "")
        $("#edtName").css("border", "1px solid red")
    else
        this.submit(); // call native submit method instead

    return false;
});

Хотя @jПредложение Энди избавиться от else и переместить return false в if() лучше.Вызов submit из submit важнее, если вы выполняете асинхронный код в обработчике.

Answer 3

Предложение:

$(document).ready(function() {
    $("#frmCatAdd").submit(function() {
        var $edtname = $("#edtName"),
            name     = $edtname.val();

        if (!$.trim(name).length) {
            $edtName.css("border", "1px solid red")
            return false;
        }
    });
});

Answer 4

Да, есть - НЕ полагайтесь только на валидацию JavaScript.Вам также необходимо выполнить проверку на стороне сервера !!Это чрезвычайно важно.Если я отключу JavaScript, я обойду всю вашу проверку.

Answer 5

Очевидный ответ:

<form id=frmCatAdd>
<input id=edtName required type=text>
</form>

Конечно, я должен повторить мнение, что вам также потребуется проверка на стороне сервера.

Answer 6

$("#frmCatAdd").submit(function() {
    $e = $(this).find("#edtName");
    if (!$.trim($e.val()).length) {
        $e.css("border", "1px solid red")
        return false;
    }
});