Nettoyage/Refactoring Jquery liaison d'événements pour éviter la duplication possible

Question

Il existe une autre façon d'écrire ce code? Je pense que c'est un peu professionnel et désordonné

Fondamentalement, le code soumettre un formulaire par "Entrée clé" et la souris "Cliquez" donc il y a une autre façon?

$("#form-signup-login input#login-submit").click(function() 
{ 
    doLogin(); 
}); 
$('input#login-request-username-email, input#login-request-password').keyup(function(e){ 
    if(e.keyCode == 13) 
     doLogin(); 
}); 

Merci pour votre temps!

Answer 1

$("#login-submit").click(doLogin); 

$('#login-request-password').keyup(function(e){ 
    if(e.keyCode === 13) 
     doLogin(); 
}); 

Explication:

1. Pas besoin de préfixer sélecteurs d'identification avec des conteneurs car ils sont uniques sur la page et vous fallback la méthode getElementById native qui est plus rapide que le moteur de grésillement.
2. Vous pouvez simplement utiliser le pointeur de fonction dans le premier cas, pas besoin de construire une autre fonction de rappel
3. Je voudrais simplement lier l'événement keyup sur le champ de mot de passe plutôt que le champ nom d'utilisateur car il y a un flux natual il

forme alternative bind soumettre qui prend soin de tout cela par défaut pour vous ..

$("#form-signup-login").submit(doLogin); 

Answer 2

vous cherchez le gestionnaire d'événements submit par hasard? http://api.jquery.com/submit/

Answer 3

Il semble que vous voulez faire sur soumettre: pourquoi ne pas utiliser:

$("#form-signup-login").on("submit", function() { 
    doLogin(); 
    return false; 
}); 

Lorsque l'id est l'ID de la forme, et non l'une des entrées.

Answer 4

$("#form-signup-login").submit(function (ev){ 
    // Do not let the browser handle the submit action 
    ev.preventDefault(); 
    // Execute your custom submit function 
    doLogin(); 
}); 

Voila!