[python] Kritika, zhodnocení...

[Jirka Vejrazka]

Ahoj,

  par malych drobnosti (nejsem expert, tak to ber s rezervou :)

    """Tato funkce vypočítává juliánské nedělní písmeno,
    parametr je sluneční kruh """

  pri viceradkovych komentarich byva zvykem ty posledni uvozovky dat na
samostatny radek, tj.

    """Tato funkce vypočítává juliánské nedělní písmeno,
    parametr je sluneční kruh
    """

            if rok <= 1699:
            return NEDELNI_PISMENO_G_1699_MAP[nedelni_p_j]
        elif rok <= 1799:
            return NEDELNI_PISMENO_G_1799_MAP[nedelni_p_j]


  "elif" je zbytecne, protoze je pred nim "return". Samotne "if" by stacilo.


  elif rok > 1700 and rok < 1800:

  lepe:

  elif 1700 < rok < 1800:


  Jirka


2013/10/15 Honza Javorek <mail at honzajavorek.cz>

> :-)
>
> Tak ony velikonoce jsou dělány přes ten dateutil, takže tam záleží, jak
> moc do minulosti jde ten. Jinak platí to, co je v README:
>
> Aim of this library is to simplify work with Czech public holidays in
> current applications, thus *it does not provide any historical data*. For
> example, *Restoration Day of the Independent Czech State* is celebrated
> since 2000, but the library returns it also for, let's say, 1978.
>
> Čili - ne, historicky to tam není. Dalo by se to dodělat přes
> https://cs.wikipedia.org/wiki/%C4%8Cesk%C3%BD_st%C3%A1tn%C3%AD_sv%C3%A1teka jiné zdroje, ale nemám pro to využití. Pokud by se ti chtělo knihovničku
> obohatit o znalost historie, budu jen rád :-)
>
> Honza
>
>
> 2013/10/15 Marcus Scalpere <xdlabacekm at seznam.cz>
>
>>  A jak se zdá, dokonce sám autor:-) Cvičení to není, archiváři ani
>> tesaři taková cvičení bohužel nemají:-) Svátky se hodí, něco takového
>> přesně potřebuji. Jen se zeptám - jsou tam nějaká omezení co se týče roků?
>> Snažím se z toho vykutit pomůcku pro archiváře, tudíž by to mělo zvládat
>> třeba i 14. století.
>>
>> Dne 15.10.2013 19:25, Honza Javorek napsal(a):
>>
>>   Ahoj,
>>
>>  jen doplním, že pokud šlo jen o to vypočítat velikonoční neděli/pondělí
>> (a není to jen nějaké cvičení), nástroje na to už existují hotové:
>>
>> -
>> http://labix.org/python-dateutil#head-8863c4fc47132b106fcb00b9153e3ac0ab486a0d(python-dateutil)
>>  - případně https://pypi.python.org/pypi/czech-holidays/
>>
>>  H
>>
>>
>>
>> 2013/10/15 starenka . <starenka0 at gmail.com>
>>
>>> Ahoj,
>>>
>>> cet sem jen zbezne..
>>> - napriklad mi prijde dobry, kdyby to u tech funkcni, kdy to vraci
>>> (napr. ty velikonoce apod) mesic a den, vratilo radsi datetime.date,
>>> nez string (kde je jeste navic nahardkodenej ceskej mesic).
>>> - obdobne nedelni_pismeno_g, by IMO melo vratit None misto "neni"
>>> - spoustu tech if/else returnu muzes vratit na jedny radce a stejne
>>> citelne pomoci (napr. u zlate_cislo):  return cislo if cislo else 0
>>>
>>> Ten dotaz ohledne pramenu nechapu (vubec moc nechapu, co to vubec
>>> dela), ale rozhodne bych do toho zadny xml netahal :D. Kdyby neco,
>>> ubirat a pridavat dny muzes jednoduse pomoci datetime.timedelta
>>>
>>> slunce v dusi
>>>    starenka
>>> ---
>>> In Perl you shoot yourself in the foot, but nobody can understand how
>>> you did it. Six months later, neither can you. | print
>>> 'aknerats'[::-1]
>>>
>>>
>>> 2013/10/15 Marcus Scalpere <xdlabacekm at seznam.cz>:
>>>  > Pěkný den vinšuji, rád bych si nechal zkritizovat přiložený skript,
>>> > popřípadě bych uvítal jakékoli připomínky atd. Mimo jiné bych se rád
>>> optal
>>> > na nejlepší řešení následujícího problému - v některých historických
>>> > pramenech je datování typu např. 2 dny před sv. Václavem, nebo ve
>>> středu po
>>> > sv. Václavu, tudíž mě napadlo vytvořit xml, kde budou svatí + datum
>>> jejich
>>> > svátku a pak ve skriptu bych přidával či ubíral dny (pro zjištění
>>> přesného
>>> > data). Jen první nápad.
>>> >
>>> > http://pastebin.com/4SKWA3ba
>>> > _______________________________________________
>>> > Python mailing list
>>> > Python at py.cz
>>> > http://www.py.cz/mailman/listinfo/python
>>> _______________________________________________
>>> Python mailing list
>>> Python at py.cz
>>> http://www.py.cz/mailman/listinfo/python
>>>
>>
>>
>>
>> _______________________________________________
>> Python mailing listPython at py.czhttp://www.py.cz/mailman/listinfo/python
>>
>>
>>
>> _______________________________________________
>> Python mailing list
>> Python at py.cz
>> http://www.py.cz/mailman/listinfo/python
>>
>
>
> _______________________________________________
> Python mailing list
> Python at py.cz
> http://www.py.cz/mailman/listinfo/python
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.py.cz/pipermail/python/attachments/20131016/a8968be1/attachment.html>