0

我有一个可以接受某些参数的网络服务，例如 ?top=5&orderby=column --- 等等...

我希望能够像这样执行我的对象：

var call = new API();
call.get().top(5).skip(15).orderby("address");

挑战在于只有最后一个 orderby 触发 execute() 方法。这是我的代码。如果您有更好的想法，请告诉我！当前每个函数结束时延迟 25ms，并在下一个函数开始时停止计时器。这是适当的/可以接受的吗？

var API = function (webservice) {
this.webservice(webservice);
return this;
};

API.prototype = {
version: function (urlFormat) {
    if (urlFormat) {
        return "v" + urlFormat.split('.').join('_');
    }
    return sessionStorage.getItem("version");
},
path: function () {
    return "../WebAPI/";
},
execute: function () {
    var path = this.path() + this.webservice() + ".svc/";
    if (this.__parameters) {
        path += "?";
    }
    var first = true;
    for (var k in this.__parameters) {
        if (k !== "type")
        path += ((first) ? (function(){first = false; return ""})() : "&") + "$" + k + "=" + this.__parameters[k];
    };
    console.log(this.__parameters.type + ": " + path);
    return this;
},
put: function () {
    this.doIt("type","put");
    return this;
},
post: function () {
    this.doIt("type","post");
    return this;
},
get: function() {
    this.doIt("type","get");
    return this;
},
delete: function() {
    this.doIt("type","delete");
    return this;
},
toString: function () {
    return "API";
},
webservice: function(webservice) {
    if (webservice) {
        this.__webservice = webservice;
    }
    else {
        return this.__webservice;
    }
},
top: function (p) {
    this.doIt("top",p);
    return this;
},
view: function (p) {
    this.doIt("view",p);
    return this;
},
orderby: function (p) {
    this.doIt("orderby",p);
    return this;
},
criteria: function (p) {
    this.doIt("criteria",p);
    return this;
},
skip: function (p) {
    this.doIt("skip",p);
    return this;
},
filter: function (p) {
    this.doIt("filter",p);
    return this;
},
doIt: function (method, parameter) {
    this.__timerStop();
    this.__parameters[method] = parameter;
    this.__timerStart();
},
__timerStop: function () {
    if (this.__timer) {
        clearTimeout(this.__timer);
    }
},
__timerStart: function (append) {
    var self = this;
    if (this.__timer) {
        this.__timerStop();
    }
    this.__timer = setTimeout(function() {
        console.log("executing.");
        console.log(JSON.stringify(self.__parameters));
        self.execute();
    }, 25);
},
__parameters: {}
};
4

2 回答 2

6

更新：你知道吗？我将在这个问题上软化我的立场（下面的原始答案）。setTimeout鉴于 JavaScript 的单线程事件循环，您实际上应该没问题，因为在您的方法链“完成”之前，您传递给的回调永远不会触发。（事实上​​，这也意味着你应该安全地传递0给setTimeout而不是25。）

我仍然认为你想出这个设计是疯了（如果这是多个开发人员会接触的代码，我会说你最好使用更简单的设计，以减少过度复杂导致团队混乱的风险） ; 但如果你坚持走这条路，你其实不应该遇到任何奇怪的海森虫。

但是，是的，我坚持我最初的建议，即execute明确要求调用。

天啊。你疯了，甚至考虑到这一点！我的意思是，我的一部分确实爱你（我是可怕的黑客的忠实粉丝）；但事实是，采用这种方法，虽然它最终可能会奏效，但如果/当它失控时会让你发疯。

我强烈反对它的主要原因是替代方案非常简单，更重要的是，实际上是可靠的：只需建立execute实际发送请求的方法的规则，因此任何链式方法调用都必须以此结束：

call.get().top(5).skip(15).orderby("address").execute();

如果你真的爱上了这个基于计时器的想法，那么有些事情告诉我你以前从未真正遭受过Heisenbug（或者，正如我最初猜测的那样，你只是疯了）。

于 2012-05-15T21:11:13.840 回答
1

有趣的想法。虽然，为什么不做这样的事情呢：

call({ type: "get", top: 5, skip: 15, orderby: "address" });

然后通过遍历实现中的对象来处理每个参数call，然后发出服务请求。

for(var arg in args) {
    if(args.hasOwnProperty(arg) && args.propertyIsEnumerable(arg)) {
        // process argument
    }
}

这使事情变得简单。

于 2012-05-15T21:44:13.410 回答