2017-05-25 8 views
0

私はjQueryをチェックしていますが、コードがどのように動作している間に、短縮できるかどうか疑問です。 私はそれを自分でやろうとしましたが、できませんでした。私はこれについて何か助けていただければ幸いです。このjQueryコードを短縮するにはどうすればよいですか?

このコードを短縮するにはどうすればよいですか?ここで

JQUERY: 

    $(function() { 
     $("#accordion").accordion({ 
      collapsible: true, 
      active: 'none' 
     }); 
    }); 

    $(function() { 
     var icons = { 
      header: "ui-icon ui-icon-plus", 
      activeHeader: "ui-icon ui-icon-minus" 
     }; 
     $("#accordion").accordion({ 
      icons: icons 
     }); 
     $("#toggle").button().on("click", function() { 
      if ($("#accordion").accordion("option", "icons")) { 
       $("#accordion").accordion("option", "icons", null); 
      } else { 
       $("#accordion").accordion("option", "icons", icons); 
      } 
     }); 
    }); 

    $("#tabs").tabs({ 
     disabled: [2, 3] 
    }); 

    $("#tabs").tabs({ 
     hide: { 
      effect: "drop", 
      duration: 750 
     } 
    }); 

    $("#tabs").tabs({ 
     show: { 
      effect: "drop", 
      duration: 750 
     } 
    }); 

    $("#tabs").tabs({ 
     heightStyle: "auto" 
    }); 
+0

あなたはhtmlコードとコードの機能を追加できますか?バイブルの例があるのは良いこと –

+0

あなたのコードを '$(function(){});'でラップするだけです。だから1つのことがあります。 – Twisty

+0

'.tabs()'にオプションを割り当てると、同じセレクタを使用しているので、同じ要素をすべて1つにまとめてください。あなたが3つの異なるタブセットを作るつもりでない限り。 – Twisty

答えて

0

は、すべて私がコメントで提案されている項目:JavaScriptの

https://jsfiddle.net/Twisty/ywx0yrub/

$(function() { 
    var icons = { 
    header: "ui-icon ui-icon-plus", 
    activeHeader: "ui-icon ui-icon-minus" 
    }; 
    $("#accordion").accordion({ 
    collapsible: true, 
    active: 'none', 
    icons: icons 
    }); 
    $("#toggle").button().click(function() { 
    if ($("#accordion").accordion("option", "icons")) { 
     $("#accordion").accordion("option", "icons", null); 
    } else { 
     $("#accordion").accordion("option", "icons", icons); 
    } 
    }); 
    $("#tabs").tabs({ 
    disabled: [2, 3], 
    hide: { 
     effect: "drop", 
     duration: 750 
    }, 
    show: { 
     effect: "drop", 
     duration: 750 
    }, 
    heightStyle: "auto" 
    }); 
}); 
  1. ラップすべてのコード内の1つの
  2. 初期化のすべての部分を1つのステートメントに定義する
  3. ボタンは静的なので、.on()を使用する必要はありません。これは間違っているわけではありませんが、.click()を使用するだけで短くすることができます。

希望があれば。

+0

ありがとう、これはとても役に立ちました – Zevothan

関連する問題