Meilleur refactor pour gérer plusieurs validations de formulaire de champ de courrier électronique jQuery

StackOverflow https://stackoverflow.com/questions/69107

Question

Quel est le meilleur moyen de refactoriser le code joint pour l'adapter à plusieurs adresses e-mail?

Le fichier HTML / jQuery joint est complet et fonctionne pour la première adresse électronique. Je peux configurer les deux autres en copiant / collant et en modifiant le code. Mais je voudrais simplement refactoriser le code existant pour gérer plusieurs champs d'adresse e-mail. 

<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>
Était-ce utile?

La solution

Au lieu d'utiliser des identifiants pour vos champs de courrier électronique, vous pouvez leur attribuer chacun une classe: 

<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>

Ensuite, au lieu de sélectionner $ (& #; # Email_Address_Status_Icon_1 "), vous pouvez sélectionner $ (" input.email"), ce qui vous donnerait un ensemble enveloppé jQuery de tous les éléments en entrée de la classe email.

Enfin, au lieu de faire explicitement référence à l’icône de statut avec un identifiant, vous pouvez simplement dire: 

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

'this' serait le champ email, donc 'this.next ()' vous donnerait son prochain frère. Nous appliquons le " span " sélecteur en plus de cela pour être sûr que nous obtenons ce que nous avons l'intention de. $ (this) .next () fonctionnerait de la même manière.

De cette manière, vous vous référez de manière relative à l'icône d'état.

J'espère que ça aide!

Autres conseils

Merci! Voici le refactor complet avec vos modifications suggérées. 

<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>

je ferais: 

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

Ajoutez ensuite class = 'validateEmail' à toutes vos entrées de courrier électronique.

Vous pouvez également consulter le plug-in de validation de formulaire . et il est très flexible et agréable à utiliser. Vous évite de réinventer ...

Licencié sous: CC-BY-SA avec attribution
Non affilié à StackOverflow
scroll top