3

以下に示すコードは正常に動作していますが、私の意見では少し素人っぽく見えます (自分で作成しました :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();

よろしくお願いします。

4

4 に答える 4

5

removeClass次のように一度だけ使用できます

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

次のように2回使用する代わりに

$('.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();

それ以外の

$('#fancybox-wrap').hide();
$('#fancybox-overlay').hide();
$('#panel_login').hide();
$('#panel_all').hide();
于 2012-04-20T22:24:43.830 に答える
2

セレクターをコンマで区切ることができます。元：

$('#fancybox-wrap, #fancybox-overlay, #panel_login, #panel_all').hide();
于 2012-04-20T22:23:25.210 に答える
2

ステートメントを組み合わせるには、.hide()4 つの要素すべてに適用されるセレクターが必要です。あなたはこれを行うことができます：

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

または、これらの要素に共通のクラスを与えることもできます:

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

.removeClass()呼び出しを組み合わせることで、最初の行を簡素化できることにも注意してください。

$('.login_inputbox').removeClass("register_inputbox_error login_inputbox_error register_inputbox_ok login_inputbox_ok");
于 2012-04-20T22:23:37.963 に答える
1

と友達に共通のクラスを与え#fancybox-wrap、それを と呼ぶ場合new-class、次のことができます。

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

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

于 2012-04-20T22:23:39.123 に答える