Menší problémky

jan commented

2023-07-10 22:44:22 +02:00

Member

špatné anotace typů
unifikace snake_case
slovnik.get('klíč', default)
deklarování atributů u tříd

- [ ] špatné anotace typů - [ ] unifikace snake_case - [ ] `slovnik.get('klíč', default)` - [x] [deklarování atributů u tříd](https://gitea.ks.matfyz.cz/KSP/formatitko/issues/14#issuecomment-2309)

jan commented

2023-07-19 16:09:48 +02:00

Author

Member

K unifikaci na snake_case, z letmého prohlížení různých pythonových knihoven i letmého pročtení PEP8 jsem se dočetl, že u tříd pythonisté preferují CapWords, u proměnných a funkcí snake_case. Jediné, kde používám camelCase jsou instance tříd, kde mi dává smysl jen změnit první písmeno na malé. Teď v posledních pár commitech jsem se snažil všechno psát snake_case, ale v některých případech to vypadá fakt nepěkně (Output_processor), takže bych se docela rád vrátil k tomu, jak jsem to dělal původně, tedy více méně jak to má PEP8 + instance tříd jsou camelCase.

Přijde ti to takhle smysluplné @mj?

K unifikaci na snake_case, z letmého prohlížení různých pythonových knihoven i letmého pročtení [PEP8](https://peps.python.org/pep-0008/) jsem se dočetl, že u tříd pythonisté preferují CapWords, u proměnných a funkcí snake_case. Jediné, kde používám camelCase jsou instance tříd, kde mi dává smysl jen změnit první písmeno na malé. Teď v posledních pár commitech jsem se snažil všechno psát snake_case, ale v některých případech to vypadá fakt nepěkně (Output_processor), takže bych se docela rád vrátil k tomu, jak jsem to dělal původně, tedy více méně jak to má PEP8 + instance tříd jsou camelCase. Přijde ti to takhle smysluplné @mj?

jan referenced this issue from a commit

2023-07-19 16:16:04 +02:00

Unifikace stylu názvů proměnných, viz můj komentář v #14.

jan referenced this issue from a commit

2023-07-19 16:18:14 +02:00

Unifikace stylu názvů proměnných, viz můj komentář v #14.

mj commented

2023-07-19 19:34:00 +02:00

Owner

Proč není instance třídy prostě output_processor?

Proč není instance třídy prostě `output_processor`?

jan commented

2023-07-19 22:04:02 +02:00

Author

Member

Na tom instancování tříd trvám asi nejméně, ale přijde mi, že

katexClient = KatexClient()

vypadá lépe a lépe to sedí k sobě než

katex_client = KatexClient()

Na tom instancování tříd trvám asi nejméně, ale přijde mi, že ```python katexClient = KatexClient() ``` vypadá lépe a lépe to sedí k sobě než ```python katex_client = KatexClient() ```

mj commented

2023-07-19 22:21:34 +02:00

Owner

Vidíš, mně naopak přijde mnohem lepší to druhé, protože je zřetelněji odlišitelné, co je typ a co instance. Naopak mi přijde matoucí používat dlouhé identifikátory, které se liší jenom velikostí jednoho písmenka (nota bene "K", které je vizuálně dost podobné "k") :-)

mj commented

2023-07-20 18:49:11 +02:00

Owner

Ještě jedna věc ke stylu: Bylo by hezke, kdyby v třídách byly deklarované typy atributů, nebo aspoň na začátku vyjmenované, jaké atributy existují. Takhle občas nevycházím při čtení metod z údivu, co také objekt má za atributy :-)

jan commented

2023-07-20 19:20:30 +02:00

Author

Member

Přidáno do todo listu.

jan referenced this issue from a commit

2023-08-20 13:52:02 +02:00

Deklarovány atributy u tříd. #14

Menší problémky #14