jQuery onclick関数のクリーンアップ

Question

以下に示すコードは、自分の意見に少しアマチュアリなように見えますが（自分で作ったもの：P）、それ以外はうまくいきます。私はこれがはるかにクリーンかもしれないと確信しています。だから私の質問は、それらの '隠れ'をどのように組み合わせるのか？その他の提案はありますか？あなたの助けを事前に

$('.login_inputbox').removeClass("register_inputbox_error login_inputbox_error").removeClass("register_inputbox_ok login_inputbox_ok"); 
validator.resetForm(); 
$(".btn-slide_login").removeClass("active_login"); 
$(".btn-slide_all").removeClass("active_all"); 
$('#fancybox-wrap').hide(); 
$('#fancybox-overlay').hide(); 
$('#panel_login').hide(); 
$('#panel_all').hide(); 

ありがとう！

Answer 1

あなたはすべての4つの要素に適用されるセレクタを必要とするだろう.hide()文を組み合わせること。あなたの最初の行は.removeClass()呼び出し組み合わせることにより、簡素化することができることも

$('.someClass').hide(); 

注：

$('#fancybox-wrap,#fancybox-overlay,#panel_login,#panel_all').hide(); 

それとも、それらの要素に共通のクラスを与えることができる：あなたはこれを行うことができます

$('.login_inputbox').removeClass("register_inputbox_error login_inputbox_error register_inputbox_ok login_inputbox_ok"); 

を

Answer 2

セレクタをカンマで区切ることができます。例：

$('#fancybox-wrap, #fancybox-overlay, #panel_login, #panel_all').hide(); 

Answer 3

#fancybox-wrapと友だちに共通のクラスを指定する場合は、new-classとすると、次のようになります。

$('.newclass').hide(); 

すべてを一度に取得します。

$('.login_inputbox').removeClass("register_inputbox_error login_inputbox_error").removeClass("register_inputbox_ok login_inputbox_ok"); 

とも

$('#fancybox-wrap, #fancybox-overlay, #panel_login, #panel_all').hide(); 

の代わりに次のように

Answer 4

次のことが可能USS removeClassは1回2回

$('.login_inputbox').removeClass("register_inputbox_error login_inputbox_error register_inputbox_ok login_inputbox_ok"); 

代わりにそれを使用しての、以下のように

$('#fancybox-wrap').hide(); 
$('#fancybox-overlay').hide(); 
$('#panel_login').hide(); 
$('#panel_all').hide();