2017-09-14 10 views
-3

コードを書くのに適した方法はどれですか?以下では、親と一緒にアイテムを削除する2つの方法があります。関数を記述し、その関数をゴミ箱アイコン(First Way)をクリックして呼び出すか、新しい関数を書くことなく直接削除する方が良いですか(Second Way)。これはどちらが良いコード方法ですか?なぜ?

2つのシナリオが考えられます:

  1. 私は再び特定のアクションを繰り返したいと再び
  2. 私は一度だけ、このアクションをやりたいです。

この例では、これは一度だけ必要です。

最初の方法:

function deleteTodoItem(e, item) { 
    e.preventDefault(); 
    $(item).parent().fadeOut('slow', function() { 
    $(item).parent().remove(); 
    }); 
} 
$(".middlepanel").on('click', '.fa.fa-trash', function(e){ 
    var item = this; 
    deleteTodoItem(e, item) 
}) 

第二の方法:

$('.middlepanel').on('click','.fa.fa-trash',function(){ 
    $(this).parent().fadeOut('slow',function(){ 
     $(this).parent().remove(); 
    } 
}) 
+1

あなたはあなた自身で質問に答えたと思います。コードを再利用する予定の場合は、メソッドを作成してください。それ以外の場合は、イベント自体に1度書き込むだけで問題ありません – Verthosa

+0

これはコードの実行速度に影響しますか?上記のようにコードを書くための2つのオプションがあるとします。 –

+1

この関数を複数回使用する場合は、最初の方法を使用するよりも2回以上使用する必要があります。一度しか関数を使用しない場合は、2番目の方法を使用します。私はいつも[KISS principe](https://en.wikipedia.org/wiki/KISS_principle)と[DNRY principe](https://en.wikipedia.org/wiki/Don%27t_repeat_yourself)を使用しようとしています – LinkinTED

答えて

0

基本的機能は、単に何かをラップすることですので、特段 "より良い" の方法は、この特定の例ではありませんイベントが使用しています。

ただし、変更が1箇所の変更に限定されるため、最初のコードスニペットはこのコンテキストでは良好です。 - 私は推測する !!!

関連する問題