Код ревью telegrambot'a

@ff5h · Регистрация: 08.02.2022

Студворк — интернет-сервис помощи студентам

Пишу телеграм бота, создается впечатление, что писать код таким образом не корректно, прошу обратить внимание на клас Selection.cs Что вы увидели подозрительного?
Ссылка на проект: https://github.com/ff5h/Bot

@sau · 14.03.2022, 23:39

ну повсеместное нарушение принципов solid

@Usaga · 15.03.2022, 05:36

ff5h, ещё добавлю, что "капитанские" комментарии:

C#
1
2
3
4
5
6
7
8
        /// <summary>
        /// Підключення до бази даних
        /// </summary>
        private MySqlConnection Сonnection;
        /// <summary>
        /// Конструктор для підключення бази даних
        /// </summary>
        public DataBase(string host, uint port, string user, string password, string table)

... не несут никакой практической пользы, а только захламляют код. Ну и да: в вашем коде попраны все возможные практики разработки)

@ff5h · 15.03.2022, 10:52 **[ТС]**

Это я уже понял, что с этим делать?

Добавлено через 1 минуту
Что делать?

@sau · 15.03.2022, 12:50

Сообщение от ff5h

Это я уже понял, что с этим делать?

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

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

Сообщение от ff5h

создается впечатление, что писать код таким образом не корректно

а что именно вас смущает ? , некорректно по отношению к чему ?

@ff5h · 21.03.2022, 21:05 **[ТС]**

Сообщение от sau

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

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

а что именно вас смущает ? , некорректно по отношению к чему ?

Мне не нравится что мои классы делают все и сразу, но после прочитаного, я понял что не стоит заморачиваться при разработки маленького проекта. Спасибо

@deNoi5e · 01.04.2022, 14:53

Ну и как данность, я бы не стал светить в паблик репе гитхаба свой ТГ токен %)

@Luca Brasi · 01.04.2022, 15:25

ff5h, BotToken
Реализуй подгрузку с внешего конфигруационного файла. А тот файл уже непосредственно в гит игнор.

Новые блоги и статьи Все статьи Все блоги /
сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .
Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .	Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и источниками (напряжения, ЭДС и тока). Найти токи и напряжения во всех элементах. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и. . .

@ff5h 0 / 0 / 0 Регистрация: 08.02.2022 Сообщений: 5

	Код ревью telegrambot'a 14.03.2022, 18:07. Показов 2041. Ответов 7 Метки .net 6, code review, telegram bot api (Все метки) Пишу телеграм бота, создается впечатление, что писать код таким образом не корректно, прошу обратить внимание на клас Selection.cs Что вы увидели подозрительного? Ссылка на проект: https://github.com/ff5h/Bot 0

@sau 2773 / 2073 / 386 Регистрация: 22.07.2011 Сообщений: 7,820
	14.03.2022, 23:39
	ну повсеместное нарушение принципов solid 0

@ff5h 0 / 0 / 0 Регистрация: 08.02.2022 Сообщений: 5
	15.03.2022, 10:52 [ТС]
	Это я уже понял, что с этим делать? Добавлено через 1 минуту Что делать? 0

@deNoi5e 0 / 0 / 0 Регистрация: 14.06.2016 Сообщений: 3
	01.04.2022, 14:53
	Ну и как данность, я бы не стал светить в паблик репе гитхаба свой ТГ токен %) 0

@Luca Brasi 892 / 204 / 55 Регистрация: 06.11.2015 Сообщений: 2,288 Записей в блоге: 2
	01.04.2022, 15:25
	ff5h, BotToken Реализуй подгрузку с внешего конфигруационного файла. А тот файл уже непосредственно в гит игнор. 0

Код ревью telegrambot'a

Решение