Jquery |相同功能内的功能。这是不好的做法？

我使用此功能容易，因为我将使用fadeTo很多：Jquery |相同功能内的功能。这是不好的做法？

function fade_to(div, speed, opacity, after_fade) { 
    $(div).fadeTo(speed, opacity, after_fade); 
}

然后我呼吁after_fade参数相同的功能：

fade_to('#div', 3000, 1, function() { fade_to('#another_div', 3000, 1)});

是一件坏事去做？我会有速度/平滑问题吗？只使用jQuery的默认fadeTo函数会更好吗？

谢谢！

来源

2012-04-09 m.spyratos

它给你带来了什么？ – zetlen 2012-04-09 16:19:44

似乎有点没有意义 – nodrog 2012-04-09 16:21:39

它没有什么不对，表现会一样，但我没有看到好处。如果你跳过一些总是相同的参数，或者可能使用$（'＃'+ div），只是传递元素的ID – ericosg 2012-04-09 16:23:24

用你的方法没有任何好处。另外你正在使用jQuery fadeTo函数。你做了什么没有错，只是没有收获。你可以保存工作，这种技术例如，如果你在你的自定义函数具有较少的参数：

function fade_to(div, after_fade) { 
    $(div).fadeTo(3000, 1, after_fade); 
} 

fade_to('#div', function(){ fade_to('#another_div', $.noop); });

这实际上节省您阻止您不必输入速度和不透明度参数工作。你也可以讨好像这样

function Fade_to(speed, opacity){ 
    return function(div, callback){ 
      $(div).fadeTo(speed, opacity, callback); 
    } 
}

然后你可以把参数保存功能对飞象

var fade_to_foo = Fade_to(3000, 1); 
fade_to_foo('#div', function(){ fade_to_foo('#another_div'); });

否则没有理由不只是写的jQuery的方式

$('#div').fadeTo(3000, 1, function(){ $('#another_div').fadeTo(3000, 1); });

来源

2012-04-09 16:29:59 Fresheyeball

+1我喜欢这个答案，除了我会为某些或所有参数设置默认功能。它可能需要一些鸭子打字，但真的可以减少代码。例如，我将'$ .noop'放入函数中，并执行'after_fade ||之类的操作$ .noop'。 – 2012-04-09 16:34:46

是的，这是真的。 – Fresheyeball 2012-04-09 16:36:23

...啊，我看到你更新了一个咖啡的例子。这是另一个好主意。 – 2012-04-09 16:36:54

您的方法很方便。我怀疑它会减缓褪色效应。在我的脑海里，这种方法没有任何问题。如果您确定淡入淡出效果在淡入淡出之前应该脉冲或闪烁，则只需将您的功能修改为脉冲/眨眼，然后淡入淡出的所有呼叫即可运行新的例程。在这种情况下它是有意义的，因为它减少了代码并提高了可维护性。 Ericosg的确提出了一个有效的观点，但为什么不减少参数，如果他们将是相同的。

来源

2012-04-09 16:25:18 Namphibian

这是一个不好的做法，因为如果不将它们应用于每个实例，就不能调用你的函数。由于您可以在jQuery中链接修饰符，并且大多数人会同意这样做是一个有用的功能，所以您正在为自己或其他任何处理此代码体的人禁用该有用功能。

如果你想添加任何额外的动画或样式，这次你必须再次通过常规的jQuery选择对象。额外的工作，额外的电话，没有真正的好处。

来源

2012-04-09 16:25:59 Thomasdotnet

+1关于jQuery链接的重点。 – 2012-04-09 16:27:35

'return'语句会处理这个问题。 – 2012-04-09 16:29:48

这种做法违背了将jQuery用作可链接的短语法库的目标。然而，如果这个特定的功能对于一个项目是有用的，可以是有效的。

来源

2012-04-09 16:30:25 undefined