Question

我如何将这两个陈述放在一个单一的陈述中。他们像镜子一样工作。我试图让代码看起来更好，更简单。非常感谢任何帮助。

           $('#q_login_dialog #menu-register').click(function() {

                    $('#q_login_dialog #menu-login').removeClass('q_dialog_panel_item_active');
                    $('#q_login_dialog #menu-register').addClass('q_dialog_panel_item_active');
                    $('#q_dialog_login input[type!=hidden]').val('');
                    $('#q_dialog_reset input[type!=hidden]').val('');
                    $('#q_dialog_login').hide();    
                    $('#q_dialog_reset').hide();                                    
                    $('#q_dialog_register').show();
                    $('#q_dialog_register input[name=email]').focus();
                    $('#q_dialog_login input').blur();
                    $('#q_dialog_login span').html('');
                    what = 'register';                              
                });

            $('#q_login_dialog #menu-login').click(function() {

                    $('#q_login_dialog #menu-register').removeClass('q_dialog_panel_item_active');
                    $('#q_login_dialog #menu-login').addClass('q_dialog_panel_item_active');
                    $('#q_dialog_register input[type!=hidden]').val('');
                    $('#q_dialog_reset input[type!=hidden]').val('');   
                    $('#q_dialog_register').hide();
                    $('#q_dialog_reset').hide();                
                    $('#q_dialog_login').show();
                    $('#q_dialog_login input[name=email]').focus();
                    $('#q_dialog_register input').blur();
                    $('#q_dialog_register span').html('');  
                    what = 'login';                     
                });

Answer 1

两者之间的唯一区别似乎是register与login的使用，这应该相当容易参数化：

function makeClickHandler(type, subtype) {
    return function() {
        $('#q_login_dialog #menu-' + subtype)
                .removeClass('q_dialog_panel_item_active');
        $(this).addClass('q_dialog_panel_item_active');
        $('#q_dialog_' + subtype + ' input[type!=hidden]').val('');
        $('#q_dialog_reset input[type!=hidden]').val('');
        $('#q_dialog_' + subtype).hide();   
        $('#q_dialog_reset').hide();                                   
        $('#q_dialog_' + type).show();
        $('#q_dialog_' + type + ' input[name=email]').focus();
        $('#q_dialog_' + subtype + 'input').blur();
        $('#q_dialog_' + subtype + 'span').html('');
        what = type;
    }
}

$('#q_login_dialog #menu-register').click(makeClickHandler("register", "login"));
$('#q_login_dialog #menu-login').click(makeClickHandler("login", "register"));

我的眼睛可能会错过另一个区别，但我认为就是这样（如果没有，你应该看到一般的想法）。

Answer 2

你必须使用变量函数和闭包：

var my_fct = function (action1, action2) {
    // here comes the closure
    return function() {
        $('#q_login_dialog #menu-'+action2).removeClass('q_dialog_panel_item_active');
        $('#q_login_dialog #menu-'+action1).addClass('q_dialog_panel_item_active');
        $('#q_dialog_'+action2+' input[type!=hidden]').val('');
        $('#q_dialog_reset input[type!=hidden]').val('');   
        $('#q_dialog_'+action2).hide();
        $('#q_dialog_reset').hide();                
        $('#q_dialog_'+action1).show();
        $('#q_dialog_'+action1+' input[name=email]').focus();
        $('#q_dialog_'+action2+' input').blur();
        $('#q_dialog_'+action2+' span').html('');  
        what = action1;                     
    };
};

$('#q_login_dialog #menu-register').click( my_fct('register', 'login') );
$('#q_login_dialog #menu-login').click( my_fct('login', 'register') );

我怎样才能将这两个陈述放在一个陈述中？

2 个答案: