Administrace komentářů

Pořád ti hapruje ta štábní kultura. Pro odsazení tam máš smíchané tabulátory, 2 mezery, 3 mezery, 12 mezer, ...

Mezery kolem operátorů máš jen v první půlce skriptu, v té druhé už to vše lepíš na sebe.

V metodě insert_text máš pořád řádek dlouhý 590 znaků. Nevím co za objekt je txtvu.buffer.text, ale většina tříd pro práci s řetězci anebo IO má definovaný operátor <<, který ti umožní to rozdělit do více řádků.

Když chceš, aby některá větev case byla použita pro více hodnot, tak máš u where [hodnota1, hodnota2, ...]. To je špatně! Ruby používá operátor ===, aby zjistilo, jestli se tou větví vydat. Uzavřením více hodnot do hranatých závorek způsobíš to, že se ti porovnává pole s nějakou hodnotou, například [2, 8, 19] === sKruh u metody nedelni_pismeno_j. Ty hranaté závorky odstraň.

Když už jsme u toho, název proměnné sKruh taky není ideální. Je konvence používat snake case pro jména proměnných, tedy například slunecni_kruh.

Zbav se toho zanořeného ternárního operátoru. Naopak bych ho použil u metod konkurenty a zlate_cislo:

def zlate_cislo(rok)
  x = (rok + 1) % 19
  x == 0 ? 19 : x
end

U metody velikonoce_g si hlídáš, aby fungovala až pro roky větší než 1582. U takto jednoduchých výrazů je v Ruby konvence dávat podmínku až za výraz: return 'neni' if x <= 1582.

Jinak u metod velikonoce_j a velikonoce_g máš jeden vstupní argument, jednou se jmenuje x, podruhé l. Tohle je matoucí, musel jsem se podívat, jak ty metody voláš z insert_text, abych zjistil, v čem je tam rozdíl. A rozdíl tam není. Oba ty parametry pojmenuje rok anebo year, abys ulehčil čtení kódu ostatním a vlastně i sobě, až se k tomu skriptu v budoucnu vrátíš a už ho nebudeš mít celý v hlavě.

K těm metodám bych měl ještě jednu poznámku. Jako vstup pro ně máš rok a výstup například "22. dubna". Snaž se dělat metody tak, aby plnily vždy jen jednu úlohu. Tady už máš smíchané dvě věci dohromady: výpočet dne velikonoc a naformátování datumu na český řetězec. Navrhoval bych, aby metody velikonoce_j a velikonoce_g vracely instanci třídy Date a dodělej si metodu, která ti naformátuje Date na české datum. Jako výhodu to má například to, že všechny tři ty metody budou snadněji znovupoužitelné v budoucnu.