`transform` + `walk` nebo `TransformProcessor`? #32

New issue

Closed

opened 2023-08-19 13:13:05 +02:00 by jan · 1 comment

jan commented

2023-08-19 13:13:05 +02:00

Member

Z #21 (comment)👍:

Taky jsem zkusil naimplementovat ten TransformProcessor, který jsem si tu vybásnil a nepřijde mi to úplně špatné, ta overridovatelnost může být dost fajn, nehledě na nezávislost na panflutím walku a nějakém minimálním zrychlení, protože je tam míň ifů a víc se spoléhá na třídy, ale kód je téměř přesně na 3x tolik řádcích a asi to není tak přehledné. Upřímně zatím nemám preferenci ani na jednu stranu a rád uslyším nějaké další argumenty pro obě varianty.

Kontext jsem také přivázal ke Groupám a naučil OG, aby si v self.context udržoval ten aktuální. Nový TP si ho udržuje stejným způsobem, místo toho, aby se podával jako argument. Tohle teoreticky může být náchylnější k chybám, ale tak jak to je by to snad mělo fungovat.

@mj @jirikalvoda co si o tom myslíte? JK už začal používat ve svém projektu nový TP, já si stále nejsem jist, jestli je to to, co chceme, byť asi je.

Z https://gitea.ks.matfyz.cz/KSP/formatitko/issues/21#issuecomment-2358👍: > Taky jsem zkusil naimplementovat ten TransformProcessor, který jsem si tu vybásnil a nepřijde mi to úplně špatné, ta overridovatelnost může být dost fajn, nehledě na nezávislost na panflutím walku a nějakém minimálním zrychlení, protože je tam míň ifů a víc se spoléhá na třídy, ale kód je téměř přesně na 3x tolik řádcích a asi to není tak přehledné. Upřímně zatím nemám preferenci ani na jednu stranu a rád uslyším nějaké další argumenty pro obě varianty. > > Kontext jsem také přivázal ke Groupám a naučil OG, aby si v self.context udržoval ten aktuální. Nový TP si ho udržuje stejným způsobem, místo toho, aby se podával jako argument. Tohle teoreticky může být náchylnější k chybám, ale tak jak to je by to snad mělo fungovat. @mj @jirikalvoda co si o tom myslíte? JK už začal používat ve svém projektu nový TP, já si stále nejsem jist, jestli je to to, co chceme, byť asi je.

jirikalvoda commented

2023-08-19 13:36:43 +02:00

Owner

Teď jsem si obě varianty přošel o vážně se docela dost klaním k TransformProcessoru.
Zejména se mi líbí mnohem lepší možnosti v přepisování volajícím programem.

Mimochodem nešel by počet řádků snížit nějakým lepším meta-programováním?

Aktuální podoba kontextů vypadá docela hezky, ale ještě jsem ji nezkoumal tak do detailů.

Teď jsem si obě varianty přošel o vážně se docela dost klaním k TransformProcessoru. Zejména se mi líbí mnohem lepší možnosti v přepisování volajícím programem. Mimochodem nešel by počet řádků snížit nějakým lepším meta-programováním? Aktuální podoba kontextů vypadá docela hezky, ale ještě jsem ji nezkoumal tak do detailů.