bobicnina / opb-projekt

0 stars 2 forks source link

Koda v `pregled` je grozna #4

Closed andrejbauer closed 10 years ago

andrejbauer commented 10 years ago

Koda v funkciji pregled je grozna. Za začetek popravite, da bo datum integer, ne pa string (namesto str se uporabi int, pa zakaj na roke delate split na datumu, ki ga vrne baza, če pa bottle zna sam delati z datumi -- preberite si dokumentacijo). Dalje, tej spremenljivki se ne reče datum ampak mesec, ker je ful ful nevljudno imeti zavajajoča imena spremenljivk.

Potem, če hočete preslikati številko meseca v ime meseca, se to ne dela s špageti iz elif, ampak se naredi tabelo in se indeksira v njo:

imena_mesecev = ['Januar', 'Februar', ..., 'December'] mesec = 5 ime = imena_mesecev[mesec - 1] # odštejemo ena, ker se tabele začnejo indeksirati z 0

A ste iz finančne matematike?

shroomba commented 10 years ago

Split naredim, da dobim iz datuma mesec in leto posebej (sem googlala in nisem našla ničesar uporabnega), vse ostalo je pa popravljeno.

andrejbauer commented 10 years ago

Ok, potem lahko zapreš ta issue s "Close issue".