Underscore: Vorschlag: _.debounce und _.throttle nehmen zusätzliche Parameter zum Kombinieren von Argumenten

Meinen eigenen Vorschlag kommentierend, sollte der Aufruf von Combine() daher den gleichen Kontext wie die Payload-Funktion angeben

allargs = combine.apply(this, [allargs, slice.call(arguments,0)])

falls die Argumente auf das Kontextobjekt zugreifen müssen....

Sollte jetzt auf Master behoben sein. throttle sollte das korrekte Verhalten zeigen, wo es immer die neueste Kopie Ihrer Argumente verwendet, sofort einmal und danach alle N Sekunden ausgelöst wird ... und sich N Sekunden danach selbst zurücksetzt der letzte nachlaufende Trigger ist aufgetreten.

Ich denke, Ihr knapper Kommentar bezieht sich auf ein anderes Problem (wahrscheinlich Nr. 170), da das durch diese Anfrage aufgeworfene Problem immer noch für den Master gilt.
Es gibt immer noch keine einfache Möglichkeit, Argumente aus den kombinierten Aufrufen zu entprellen oder zu drosseln, und ich denke immer noch, dass es eine nützliche optionale Ergänzung ist, die das Standardverhalten unverändert lässt, wenn das optionale Kombinationsargument nicht angegeben wird.

Ah, du hast recht. Das Sammeln von Argumenten liegt außerhalb des Bereichs von Underscore - Sie können Ihre gesammelten Daten gerne an einem guten Ort außerhalb der Funktionen _.throttle und _.debounce verstauen.

Schade, ich betrachte Debounce als eine Art Fold-Left (Reduzieren) über mehrere Anrufe mit Timeout, daher der Akkumulator ... aber es ist dein Anruf :)

OK, um nicht weiter zu hämmern, aber falls sich jemand das irgendwann später ansieht und sich fragt, wie man dasselbe macht, dachte ich, dass dies der sauberste Weg ist, ohne die Entprellung selbst zu ändern (ich füge es dem _-Objekt hinzu, andere ziehen es vielleicht vor, es nicht zu tun zu)

_.mixin({
  debounceReduce: function(func, wait, combine) {
    var allargs,
        context,
        wrapper = _.debounce(function() {
            var args = allargs;
            allargs = undefined;
            func.apply(context, args);
        }, wait);
        return function() {
            context = this;
            allargs = combine.apply(context, [allargs,  Array.prototype.slice.call(arguments,0)]);
            wrapper();
        };
    }
})

Dies ergibt eine entprellte Funktion, deren Argumente durch die Combine-Funktion reduziert werden, also zum Beispiel

  delayLog = _.debounceReduce(function() { console.log(arguments); }, 5000, 
                              function(acc,args) { return (acc || []).concat(args); });
  delayLog(3,4);
  delayLog(7,8,9);

wird einige Sekunden später console.log mit dem Array [3,4,7,8,9] aufrufen

@schmerg — das sieht enorm nützlich aus. Wären Sie bereit, diesen Code unter der MIT-Lizenz zu lizenzieren? (Ein „Ja“ genügt!)

@markjaquith Klare Sache - ja. Gerne...

Wenn jemand vorbeikommt und eine aktualisierte/kommentierte moderne js-Version des oben Genannten haben möchte:

_.mixin({
  debounceReduce(func, wait, combine) {
    let allArgs; // accumulator for args across calls

    // normally-debounced fn that we will call later with the accumulated args
    const wrapper = _.debounce(() => func(allArgs), wait);

    // what we actually return is this function which will really just add the new args to
    // allArgs using the combine fn
    return (...args) => {
      allArgs = combine(allArgs,  [...args]);
      wrapper();
    };
  },
});

@kmannislands Hey, deine Version setzt allArgs innerhalb wrapper() nicht zurück, daher erhalten nachfolgende Aufrufe von func() historische Stapel von Argumenten sowie den aktuellen Stapel.

Sollte es nicht heißen:

const wrapper = _.debounce(() => {
    const args = allArgs;
    allArgs = undefined;
    func(args);
}, wait);

@markjaquith +1

Auch func( args ) unterscheidet sich von der Originalversion, die func.apply(context, args); verwendet.
Für ersteres wird args wie im Ziel func() verwendet, wohingegen Sie im späteren _(ursprünglichen Code)_ entweder arguments in einer normalen Funktion verwenden müssen oder ( ...args ) in einer es6-fetten Pfeilfunktion.