2017-09-14 37 views
-3

哪个编写代码更好?下面我有两种方法来删除一个项目及其父项。编写一个函数然后在单击垃圾桶图标(First Way)时调用该函数或直接删除它而不写入任何新函数(第二种方法)会更好。哪个代码更好?为什么?

可能有两种情况:

  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

我想你自己回答了这个问题。如果您打算重新使用代码,则创建一个方法。否则,在事件本身写入一次没有问题 – Verthosa

+0

这会影响代码的执行速度吗?假设我只想执行一次这个动作,并有两个选项来编写如上所示的代码 –

+1

如果您想多次使用该函数,那么我会使用第一种方法。如果你只使用一次函数,那么我会用第二种方法。我总是尝试使用[KISS principe](https://en.wikipedia.org/wiki/KISS_principle)和[DNRY principe](https://en.wikipedia.org/wiki/Don%27t_repeat_yourself) – LinkinTED

回答

0

有一个在这个特定的例子中,没有具体的 “更好” 的方式,因为基本功能仅仅是包装的东西该事件正在使用。

但是,在这种情况下,仍然是第一个代码片段很好,因为修改可能仅限于一个地方并且仅限于一处。 - 我猜 !!!

相关问题