Javascript 嵌套方法、最佳实践

Javascript 嵌套方法、最佳实践,javascript,jquery,json,Javascript,Jquery,Json,我目前正在摆弄jQuery的ajax方法,从JSON提要获取一些数据,并根据数据设置一些条的动画 由于我正在进行黑客攻击,我真的不知道我的方法是否正确 $.ajax({ url: "/echo/json/", type: "POST", data: data, success: function(data){ $("#value1-bar .usage-bar-fill").animate({ width: data.Value1

我目前正在摆弄jQuery的ajax方法,从JSON提要获取一些数据,并根据数据设置一些条的动画

由于我正在进行黑客攻击,我真的不知道我的方法是否正确

$.ajax({

    url: "/echo/json/",
    type: "POST",
    data: data,
    success: function(data){


    $("#value1-bar .usage-bar-fill").animate({
        width: data.Value1
        }, 2500, function() {
        // Animation complete.
    });

    $("#value2-bar .usage-bar-fill").animate({
        width: data.Value2
        }, 2500, function() {
        // Animation complete.
    });

    },
    dataType: "json"
});
从ajax方法内部设置div动画可以吗,或者有更正确的方法来实现这一点——格式化、内存等等

如何从ajax方法返回一些变量并从ajax方法外部执行动画方法是否更好

在这里查看JSFIDLE示例:


我知道这是一个相当广泛的问题,但作为一名平面设计师,我需要自学成才的网页设计师——我从来都不知道我所做的是否是良好的实践。

我喜欢这个问题,我很想看到一些更详细的反馈,但我认为这看起来不错。您已经在ajax方法中调用了一个嵌入式success函数,因此担心进一步的嵌入式函数的内存/性能几乎可以忽略不计

如果您想用success函数中使用的相同/类似动画为
用法栏填充更多的动画,那么我可能会创建单独的函数并调用它们……但我相信您已经知道了;)


问得好

这几乎是一个品味问题,因为这似乎是一个非常小的应用程序。当您的应用程序增长时,总体设计和原则将成为流程中更重要的部分

但也就是说,我不喜欢这种方法。它很容易产生不必要地重复自身的代码——除非这些是非常特定于此用例的严格的一次性情况,但大多数情况下,您可以进行泛化以实现代码重用

就我个人而言,我会在其他地方声明回调函数,可能是在一个已经知道应该使用什么(即选择器)并且知道如何使用它的对象上。这样,代码几乎自行构造,只在需要的地方保留非常局部相关的位

在这种特定的情况下,它将是一个图形对象的线条,当设置一个值时,它会适当地更新自己的显示/视图


希望我的闲话有意义。我可能有点太累了,无法给出意见和专家建议,所以我就到此为止。

你能重写他的例子吗?我通常和@timkl做的一样,我也一直在寻找更好的方法。现在我太累了,但可能会晚一点。