stepa1000 / echo-bot-template

0 stars 0 forks source link

Имена #9

Closed pavelzarubin closed 1 year ago

pavelzarubin commented 1 year ago

Что за типы Welcome9, Welcome10 и т. п.? Дай им нормальные имена. В коде много где присутствуют очень странные имена.

pavelzarubin commented 1 year ago

https://github.com/stepa1000/echo-bot-template/blob/f897342adce6b543f17b4fc110e8c2f95040f88d/src/FrontEnd/Telegram/Data/PollMessage.hs#L14

почему все-таки Welcome

stepa1000 commented 1 year ago

утилита так сгенерировала.

pavelzarubin commented 1 year ago

Ну в общем, имена должны отражать суть функций или типов. Так что называй нормально. Слишком короткие имена тоже плохо, ну и длинные. Пройдись по всему коду и назови так, чтобы человек с первого взгляда мог понять, что где происходит.

pavelzarubin commented 1 year ago

утилита так сгенерировала.

что за утилита?

stepa1000 commented 1 year ago

https://jsonformatter.org/json-to-haskell

pavelzarubin commented 1 year ago

Ну в проде таким лучше не заниматься.

stepa1000 commented 1 year ago

мне переменовать в Response?

pavelzarubin commented 1 year ago

Ну да. Все Welcome нужно убрать.

pavelzarubin commented 1 year ago

Вообще это не только Welcome касается, у тебя очень много имен состоящих из одной буквы. Все это нужно назвать развернуто.

stepa1000 commented 1 year ago

у меня много типов состоящих из одного слова, нужен болие развёрнутое название?

pavelzarubin commented 1 year ago

Зависит от того, что они отражают. Если одного слова достаточно, то исправлять не нужно.

stepa1000 commented 1 year ago

можно один пример где исправить.

pavelzarubin commented 1 year ago

https://github.com/stepa1000/echo-bot-template/blob/4f4e3004ededa482ea95a19530044011dfbafe3e/src/FrontEnd/Telegram/Data/GetUpdate.hs#L61-L66

Непонятно что за опция.

stepa1000 commented 1 year ago

22

stepa1000 commented 1 year ago

можно закрывать?