questo codice dovrebbe essere strutturato in un evento di cattura e un metodo o inserire la funzionalità nell'evento che cattura

Ho adottato un codice jQuery che ha come 20 acquisizioni di eventi che assomigliano a questo

  // used on menu#edit
  $('.show-headers-only-toggle').on('click',function(){
    arc_eh.menu_edit.show_headers_only_toggle.call(this);
  });

che chiama un metodo che fa la funzionalità. Questo è l'unico posto in cui viene chiamato questo metodo. Voglio mettere la funzionalità dopo l'evento in questo modo poiché penso che non ci sia motivo per avere questo altro metodo. L'argomento per averlo in un metodo è che sto inquinando questo spazio e la gestione degli eventi è come un router e questo è un controller. Penso che questo sia il ciclismo e non stiamo usando un framework, quindi è più semplice avere tutto a posto. Chi ti sembra sbagliato in questo? Sento che stanno cambiando questa cosa.

  // used on menu#edit
  $('.show-headers-only-toggle').on('click',function(){
    var action=$(this).attr('arc-action');
    // this would just be what is in the function above
    if(action=='hide-items'){
      $('.menu-item-admin').hide();
      $(this).text('show items');
      $(this).attr('arc-action','show-items');
    }else{
      $(this).text('show headers only');
      $('.menu-item-admin').show();
      $(this).attr('arc-action','hide-items');
    }
  });