Jquery自己呼び出し関数を避ける

Question

私は2つのボタンを同時に切り替えるためにJquery関数の下にありますが、私の上司が言うコードを書くのは「汚い」方法です。私は関数を呼び出す、またはJqueryパラメータに渡す必要はありません、私は全く新しいプログラミングなので、この関数を書くためのシンプルで良い方法ですか？私は、コードは多くの方法で行うことができると思いjQueryの

var startStopBtn = function() { 

    var startBtn = $('#timerStart'); 
    var stopBtn = $('#timerStop').hide(); 

    var Start = function() { 
     startBtn.hide(); 
     stopBtn.show(); 
    }; 

    var Stop = function() { 
     var remarks2 = $(".textArea-one").val(); 
     if (remarks2 !== "") { 
     startBtn.show(); 
     stopBtn.hide(); 
     } 
    }; 
    return { 
     Start: Start, 
     Stop: Stop 
    }; 
}(jQuery); 
jQuery('#timerStart').on('click', startStopBtn.Start); 
jQuery('#timerStop').on('click', startStopBtn.Stop); 

Answer 1

以下 が、これはちょうどそれらの一つです。

最初にstopBtnを非表示にして、これを行う関数を作成するように見えます。ページの読み込み時にその関数を呼び出すか、関数を作成し、ページが読み込まれたときに呼び出します。ここでは、いつでも呼び出す必要がある関数を作成します。それをしたくない場合は、その機能を削除してください。

#timerStartまたは#timerStopをクリックすると、2つの異なる機能が実行されます。

これは私のバージョンですが、私はそれを向上させることができると確信している：

function startStopBtn(){ 
     $('#timerStop').hide(); 
    }; 

    $('#timerStart').on('click', function(){ 
     $('#timerStart').hide(); 
     $('#timerStop').show(); 
    }); 

    $('#timerStop').on('click', function(){ 
     var remarks2 = $(".textArea-one").val(); 
      if (remarks2 !== "") { 
      $('#timerStart').show(); 
      $('#timerStop').hide(); 
      } 
    });