Javascript сглаживающий метод цепочки

Question

У меня есть веб-сервиса, который может принимать определенные параметры, как таковой,? Наверху = 5 & OrderBy = столбец --- и т.д ...

Я хочу, чтобы иметь возможность выполнить свой объект, как, например :

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

Задача состоит в том, чтобы только последний порядок запуска запускал метод execute(). Вот мой код. Пожалуйста, дайте мне знать, если у вас есть идеи! В настоящее время он задерживается на 25 мс, когда заканчивается каждая функция, и останавливает таймер при запуске следующей функции. Является ли это правильным/приемлемым?

---

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: {} 
}; 

Answer 1

Update: Вы знаете, что? Я собираюсь смягчить свою позицию на этом (оригинальный ответ ниже). Вы должны быть в порядке, учитывая, что обратный вызов, который вы передаете в setTimeout, никогда не может быть запущен до того, как ваша цепочка методов будет «завершена» с помощью однопоточного цикла событий JavaScript. (И в самом деле, это также означает, что вы должны быть безопасным прохождением 0 к setTimeout вместо 25.)

Я все еще думает, что вы гайку придумывая эту конструкцию (и если это код, который несколько разработчиков будут трогательными, Я бы сказал, что вам лучше с более простым дизайном, чтобы уменьшить риск путаницы в команде из-за чрезмерной сложности); но если вы настаиваете на том, чтобы пройти этот путь, вы на самом деле не должны сталкиваться с любыми странными гейзенбергами.

Но да, я согласен с моим первоначальным советом о необходимости прямого вызова execute.

---

О, человек. Вы сумасшедший, чтобы даже подумать об этом! Я имею в виду, что часть меня действительно вас любит (я big fan of horrifying hacks); но факт заключается в том, что, принимая этот подход, в то время как он может в конечном итоге работать, приведет вас в орехи, если/когда он идет haywire.

Основная причина, я бы сильно обескуражить его в том, что альтернативой является очень легко и, что более важно, на самом деле надежным: просто установить правило, execute метод, который фактически отправляет запрос, и поэтому любой прикован вызов метода должен заканчиваться, что:

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

Если вы серьезно влюблена в эту идею таймера на основе, что-то подсказывает мне, что вы действительно никогда не страдал от Heisenbug до (или, как я изначально догадывался, ты это просто из вашего ума).

Answer 2

Интересная идея. Хотя, почему бы не сделать что-то вроде этого, вместо:

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

Затем обработать каждый аргумент, обернув через объект внутри call реализации, а затем сделать запрос на обслуживание.

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

Это простота вещей.