sebenik / stevila-z-besedo

Zapis števil, datumov in cifer z besedo.
MIT License
1 stars 0 forks source link

Nepravilen izpis nekaterih celih števil #1

Open gregopet opened 5 years ago

gregopet commented 5 years ago

Koda nepravilno izpiše nekatera števila in nekonsistentno menja med moškim in ženskim spolom, npr.:

21 - enindvajset 200 - dvasto ena (ž) / sto en (m) dve (ž) / sto dva (m)

Testiral sem s primeri, ki smo jih zbrali v https://github.com/gregopet/zapis-slovenskih-stevil v namen, da developerji lahko testirajo zapis slovenskih števil v kateremkoli programskem jeziku (in kjer tudi sprejemamo issue-je :smiley: ).

gregopet commented 5 years ago

(testiral sem z naslednjo kodo)

var szb = require("./stevila-z-besedo")
var primer = require("./primeri")
var converter = new szb({type: "number"});

for (var a = 0; a < primer.length; a++) {
    var testCase = primer[a].stevilo;
    var expected = primer[a].imenovalnik.vsi || primer[a].imenovalnik.z;
    var actual = converter.convert(testCase);
    if (expected != actual) {
        console.log('' + testCase + ': ' + actual + ' != ' + expected);
    }
}
sebenik commented 5 years ago

Ob prvi priložnosti porpavim. Na hitro sem pognal vaše testne primere in me zanima stališče glede števila: 1803603801 (milijarda osemsto trije milijoni šeststo tri tisoč osemsto ena)

Glede na spodnji vir je oblika tri milijone veliko pogostejša in pri pišočih in govorečih prva izbira, nasprotno z oblikotrije milijoni. Vir

gregopet commented 5 years ago

Ja, se strinjam in bom popravil, hvala!