developit / developit/decko

Don't think @memorize fit any case.

Aperta
#13 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
JavaScript
Stelle
1k
Fork
34
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

I saw you use first argument as key of cache.
What if the first argument is an object, like this:

```js
const a = {};
const b = {c: {}}
String(a) //"[object Object]"
String(b) //"[object Object]"
```
Besides, can you provide any benchmark tests?
I don't think it fit any case.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Start at the @memorize decorator and reproduce the object-key examples from the issue. Check the existing test setup, then add benchmark coverage for those cases; done means the collision behavior and performance trade-off are addressed or clearly documented.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
javascript
Ambito
tooling
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
30/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.