esamattis / esamattis/underscore.string

.sprintf and .numberFormat should work on numeric strings.

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

Descrizione

The current behavior of .sprintf and numberFormat is briefly demonstrated below. Providing numeric strings throws an error.

```
> ._str.sprintf("%.2f", 0.5);
"0.50"
> ._str.sprintf("%.2f", "0.5");
Error: [_.sprintf] expecting number but found string
```

With respect to keeping the utility functions as simple as possible, this behavior is considered 'correct'.
Since JS is loose-typed, at least having an option to run these would be beneficial to developers. I'd prefer not to cast all inputs with parseFloat or parseInt before using the utility, when it does it already just after it's own type check.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Individua le implementazioni e i test di sprintf e numberFormat, quindi riproduci gli esempi di stringhe numeriche dell’issue. Chiarisci il comportamento di opt-in previsto e l’API prima di modificare entrambe le utility; il lavoro è completo quando le stringhe numeriche funzionano come concordato senza compromettere il comportamento esistente del controllo dei tipi, con una copertura per i casi dimostrati.

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

Valutazione

Stack tecnologico
javascript
Ambito
tooling
Tipo di issue
Funzionalità
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.