You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
... da lahko enostavno do vseh dostopaš na enak način - brez pisanja dodatnih pogojev
(spodnja koda je JS)
for (let i in stevila){
console.log(stevila[i].rodilnik.z)
}
namesto
for (let i in stevila){
if(Object.keys(stevila[i].rodilnik).length>1){
console.log(stevila[i].rodilnik.z)
}
else{
console.log(stevila[i].rodilnik.vsi)
}
}
O tem sem razmišljal, veliko pravzaprav, in se odločil za trenutno stanje ker:
je iskanje napak lažje, če je manj teksta (sploh, če tekst berejo netehnični ljudje, npr. slavisti), odprava napak pa je nekako cel point tega repozitorija,
pri veliki večini primerov ni posebne oblike za moški, ženski in srednji spol,
celotna datoteka velika_stevila.json ne potrebuje spolov,
ker se bo celotna koda uporabljala v testih, tam pa imamo ponavadi (no, vsaj jaz 😄) tako ali tako en kup pomožnih metod, ki pomagajo, da so dejanski testi bolj jasni (npr. function getZapis(stevilo, sklon, spol) { .. } - v JS bi bilo morda dovolj dobro že stevila[i].rodilnik.vsi || stevila[i].rodilnik.z?)
Razmišljal sem o tem, da bi dal v repozitorij skripto, ki bi spremenila trenutni JSON v neko kanonično obliko ali pa, da bi to naredil na svoji mašini in zacommital/publishal samo transformirano verzijo (bonus pike, če bi to delal kak CI service avtomatsko). Lahko bi tudi kasneje, ko bomo (recimo) našli vse napake, spremenil strukturo datoteke, ker je trenutno gotovo še nihče ne uporablja in s tem nikogar ne bi prizadanel, učinek na iskanje napak pa bi bil minimalen.
Sem odprt za nadaljno debato in zato puščam tudi issue odprt, ampak zaenkrat je status WONTFIX :)
... da lahko enostavno do vseh dostopaš na enak način - brez pisanja dodatnih pogojev
(spodnja koda je JS)
namesto
Testni primer
The text was updated successfully, but these errors were encountered: