更好的方法來優化IF語句

有沒有更好的方式來優化jQuery中的以下IF語句？更好的方法來優化IF語句

我不得不編輯下面的代碼幾次，這將是很好，如果我可以插在必要的值在一個不錯的陣列或東西...

$("#pmselect").change(function() { 
     if ($("#pmselect option:selected").attr("class") == "null") { 
      $('.avatar').hide(); 
      $('.pmpass').hide(); 
      $('.pmlogin').hide(); 
      $('#nullavatar').show(); 
     } 
     if ($("#pmselect option:selected").attr("id") == "pm") { 
      $('.avatar').hide(); 
      $('.pmpass').show(); 
      $('.pmlogin').show(); 
      $('#pmavatar').show(); 
     } 
     if ($("#pmselect option:selected").attr("id") == "as") { 
      $('.avatar').hide(); 
      $('.pmpass').show(); 
      $('.pmlogin').show(); 
      $('#asavatar').show(); 
     } 
     if ($("#pmselect option:selected").attr("id") == "pn") { 
      $('.avatar').hide(); 
      $('.pmpass').show(); 
      $('.pmlogin').show(); 
      $('#pnavatar').show(); 
     } 
     if ($("#pmselect option:selected").attr("id") == "jm") { 
      $('.avatar').hide(); 
      $('.pmpass').show(); 
      $('.pmlogin').show(); 
      $('#jmavatar').show(); 
     } 
     if ($("#pmselect option:selected").attr("id") == "mh") { 
      $('.avatar').hide(); 
      $('.pmpass').show(); 
      $('.pmlogin').show(); 
      $('#mhavatar').show(); 
     } 
     if ($("#pmselect option:selected").attr("id") == "rh") { 
      $('.avatar').hide(); 
      $('.pmpass').show(); 
      $('.pmlogin').show(); 
      $('#rhavatar').show(); 
     } 
     if ($("#rmselect option:selected").attr("id") == "rm") { 
      $('.avatar').hide(); 
      $('.pmpass').show(); 
      $('.pmlogin').show(); 
      $('#rmavatar').show(); 
     } 
    });

來源

2013-03-02 theshizy

switch/case語句？ – ChuckE 2013-03-02 12:32:45

除了第一個「if」之外，您還根據元素的「id」進行選擇。爲此使用'switch'更清潔。 https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Statements/switch。還要考慮是否所有顯示/隱藏邏輯都可以移動到單獨的函數中，以便您可以將它與開關/ if語句分開 – thaJeztah 2013-03-02 12:34:26

改進已編寫和正在工作的代碼 - > http://meta.codereview.stackexchange.com/ – 2013-03-02 12:34:50

你可以使用開關/箱，這樣你可以鞏固有些東西在多個案件進行...

switch($("#pmselect option:selected").attr("id")){ 
    case 'pm': 
    //do stuff 
    break; 
    case 'xyz': 
    case 'rh': 
    //do stuff for xyz and rh 
    case 'mh': 
    // do stuff for mh only 
    break; 
}

來源

2013-03-02 12:33:04

'switch（$（「＃pmselect option：selected」）。attr（'id'））{':) – Johan 2013-03-02 12:34:44

除了在原代碼中，最後的'if'測試'＃rmselect'而不是'＃pmselect'。 – 2013-03-02 12:35:31

試試這個：

$("#pmselect").change(function() { 
     $('.avatar').hide(); 
     if ($("#pmselect option:selected").attr("class") == "null") { 
      $('.pmpass, .pmlogin').hide(); 
      $('#nullavatar').show(); 
     } else { 
      var id = $("#pmselect option:selected").attr("id"); 
      $('.pmpass, .pmlogin').show(); 
      $('.pmlogin').show(); 
      $('#' + id + 'avatar').show(); 
     } 
    });

來源

2013-03-02 12:38:47 AlecTMH

在過去的假設$("#rmselect option:selected")如果表達式是一個錯字（以信譽明察秋毫@RaymondChen），我想你會發現代碼簡化爲這樣：

$("#pmselect").change(function() { 
    var opt = this[this.selectedIndex], 
     isNull = $(opt).hasClass('null'), 
     id = isNull ? 'null' : opt.id; 
    $('.avatar').hide(); 
    $('.pmpass, .pmlogin')[isNull ? 'hide' : 'show'](); 
    $('#' + id + 'avatar').show(); 
});

沒有在眼前的if :)

如果$("#rmselect option:selected")不是拼寫錯誤，則需要單獨處理#rmselect條件。

來源

2013-03-02 13:19:16

更好的方法來優化IF語句

回答

相關問題