dice-roller/rpg-dice-roller

Support ternary operator

Aperta

#240 aperta il 17 gen 2022

Vedi su GitHubGuida contributor
 (4 commenti) (0 reazioni) (0 assegnatari)JavaScript (68 fork)auto 404
enhancementgood first issue

Metriche repository

Star
 (279 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Is your feature request related to a problem? Please describe. In the tool I'm writing currently I use mustache for user side templating and this lib to "evaluate" the values. For some expression it would be easier and cleaner to write and use them if this lib would support ternary operations.

Describe the solution you'd like Support writing a ? b : c which returns b if a is non-zero and c if a is zero. With this the comparison operators should also work with just naked numbers so that the following expression would also work 1>0 ? 100 : 0.

Describe alternatives you've considered I can always write my own parser on top of this lib and I might have to do that at some point anyways. 😅

Guida contributor

Direzione di ricerca
Esamina il parser di espressioni corrente per capire come vengono gestiti gli operatori binari e estendilo per supportare gli operatori ternari (a ? b : c). Assicurati che anche gli operatori di confronto funzionino con numeri nudi come 1>0.
Tech stack
javascript
Dominio
tooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptExpression parsing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60