Лучший рефакторинг для обработки множественных проверок формы поля электронной почты jQuery - PullRequest
Новая
       16

Лучший рефакторинг для обработки множественных проверок формы поля электронной почты jQuery

0 голосов
/ 16 сентября 2008

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

Прикрепленный HTML / jQuery завершен и работает для первого адреса электронной почты. Я могу настроить два других путем копирования / вставки и изменения кода. Но я хотел бы просто провести рефакторинг существующего кода для обработки нескольких полей адреса электронной почты. 

<html>
<head>
    <script src="includes/jquery/jquery-1.2.6.min.js" type="text/javascript"></script>
    <script language="javascript">
        $(document).ready(function() {
            var validateUsername = $('#Email_Address_Status_Icon_1');

            $('#Email_Address_1').keyup(function() {
                var t = this;
                if (this.value != this.lastValue) {
                    if (this.timer) clearTimeout(this.timer);
                    validateUsername.removeClass('error').html('Validating Email');

                    this.timer = setTimeout(function() {
                        if (IsEmail(t.value)) {
                            validateUsername.html('Valid Email');
                        } else {
                            validateUsername.html('Not a valid Email');
                        };
                    }, 200);

                    this.lastValue = this.value;
                }
            });
        });

        function IsEmail(email) {
            var regex = /^([a-zA-Z0-9_\.\-\+])+\@(([a-zA-Z0-9\-])+\.)+([a-zA-Z0-9]{2,4})+$/;
            if (regex.test(email)) return true;
            else return false;
        }  
    </script>
</head>
<body>
    <div>
        <label for="Email_Address_1">Friend #1</label></div>
        <input type="text" ID="Email_Address_1">
        <span id="Email_Address_Status_Icon_1"></span>
    </div>
    <div>
        <label for="Email_Address_2">Friend #2</label></div>
    <input type="text" id="Email_Address_2">
        <span id="Email_Address_Status_Icon_2"></span>
    </div>
    <div>
        <label for="Email_Address_3">Friend #3</label></div>
    <input type="text" id="Email_Address_3">
        <span id="Email_Address_Status_Icon_3"></span>
    </div>
    </form>
</body>
</html>

Ответы [ 3 ]

2 голосов
/ 16 сентября 2008

Вместо того, чтобы использовать идентификаторы для своих полей электронной почты, вы можете назначить им класс: 

<div>
    <label for="Email_Address_1">Friend #1</label></div>
    <input type="text" class="email">
    <span></span>
</div>
<div>
    <label for="Email_Address_2">Friend #2</label></div>
    <input type="text" class="email">
    <span></span>
</div>
<div>
    <label for="Email_Address_3">Friend #3</label></div>
    <input type="text" class="email">
    <span></span>
</div>

Затем, вместо выбора $ ("# Email_Address_Status_Icon_1"), вы можете выбрать $ ("input.email"), что даст вам завернутый в jQuery набор всех элементов ввода класса email.

Наконец, вместо того, чтобы явно ссылаться на значок состояния с идентификатором, вы можете просто сказать: 

$(this).next("span").removeClass('error').html('Validating Email');

'this' будет полем электронной почты, поэтому this.next () даст вам следующий брат. Мы применяем селектор «span» поверх этого, просто чтобы быть уверенным, что мы получаем то, что намереваемся. $ (this) .next () будет работать так же.

Таким образом, вы относитесь к значку состояния относительным образом.

Надеюсь, это поможет!

0 голосов
/ 16 сентября 2008

Я бы сделал: 

$(document).ready(function() {
        $('.validateEmail').keyup(function(){Update_Email_Validate_Status(this)});            
    });

Затем добавьте class = 'validateEmail' ко всем вашим адресам электронной почты.

В качестве альтернативы загляните в Плагин проверки формы Я много использовал это, и он очень гибкий и приятный в использовании. Спасает вас заново изобретать ...

0 голосов
/ 16 сентября 2008

Спасибо! Вот завершенный рефакторинг с предложенными вами изменениями. 

<script language="javascript">
        $(document).ready(function() {
            $('#Email_Address_1').keyup(function(){Update_Email_Validate_Status(this)});
            $('#Email_Address_2').keyup(function() { Update_Email_Validate_Status(this)});
            $('#Email_Address_3').keyup(function() { Update_Email_Validate_Status(this)});            
        });

        function Update_Email_Validate_Status(field) {
            var t = field;
            if (t.value != t.lastValue) {
                if (t.timer) clearTimeout(t.timer);
                $(t).next("span").removeClass('error').html('Validating Email');

                t.timer = setTimeout(function() {
                    if (IsEmail(t.value)) {
                        $(t).next("span").removeClass('error').html('Valid Email');
                    } else {
                    $(t).next("span").removeClass('error').html('Not a valid Email');
                    };
                }, 200);

                t.lastValue = t.value;
            }
        }

        function IsEmail(email) {
            var regex = /^([a-zA-Z0-9_\.\-\+])+\@(([a-zA-Z0-9\-])+\.)+([a-zA-Z0-9]{2,4})+$/;
            if (regex.test(email)) return true;
            else return false;
        }  
    </script>
...