Patch 3

[Jacotsu]

non credo che il silent fail con valore di default sia una buona idea, rischi di incasinare la classificazione delle transazioni non mappate

[Alfystar]

Ho pensato di inserirla poiché, in caso di modifiche/aggiunte future lato intesa, almeno lo script continua a funzionare, e poi, usando il risultato, quando si trova un record con un tipo diverso da quello che ci si aspettava, guardando la descrizione (che ora comprende anche il valore nella mappa) possiamo facilmente migliorare lo script senza compromettere la sua funzionalità.

Però hai ragione, completamente silenzioso potrebbe non essere una buona idea, che ne dici di fargli fare un print su terminale con un warning ⚠?

[Jacotsu]

Ho pensato di inserirla poiché, in caso di modifiche/aggiunte future lato intesa, almeno lo script continua a funzionare, e poi, usando il risultato, quando si trova un record con un tipo diverso da quello che ci si aspettava, guardando la descrizione (che ora comprende anche il valore nella mappa) possiamo facilmente migliorare lo script senza compromettere la sua funzionalità.

Però hai ragione, completamente silenzioso potrebbe non essere una buona idea, che ne dici di fargli fare un print su terminale con un warning warning?

si, la print è una buona idea. Magari scrivendo di aprire una issue su questo repo

[Alfystar]

Buona idea 💡! Appena ho un attimo mi ci metto a farlo, magari stessa cosa per la valuta?

Mettiamo euro di default e facciamo printare il warning ⚠

Così rendiamo il codice più future proof 😁

[Jacotsu]

Buona idea bulb! Appena ho un attimo mi ci metto a farlo, magari stessa cosa per la valuta?

Mettiamo euro di default e facciamo printare il warning warning

Così rendiamo il codice più future proof grin

no, per la valuta non andrebbe bene così. Immagina fare una transazione di 4000 yen e mettere euro come valuta di default, romperesti tutto il bilancio.

[Alfystar]

Ti do ragione, per la valuta meglio che fallisca