Как правильно организовать код?

Question

[IseeCollapse]

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

Comments

[Василий Банников]

Почему бы не сделать экземпляр 1 раз и закжшировать его где-нибудь?
Если функция чистая и полиморфизма нет, то можно и статическим сделать

[IseeCollapse]

Василий Банников, вот я и интересуюсь у людей с коммерческим опытом как делать лучше, стараясь избегать говнокод

[Василий Банников]

IseeCollapse, как я написал - есть по сути два варианта.
Конкретнее можно предложить, если код покажете.

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

Ну и получение экземпляра обычно разруливается через IoC-контейнер

[IseeCollapse]

Василий Банников, что имеется ввиду под закешировать? Полиморфизма никакого нет, функция просто возвращает экземпляр класса MySqlConnection к бд

[Василий Банников]

IseeCollapse, закешировать = сохранить где-то (в переменную в Main например и прокидывать везде)

А функция создаёт новое подключение или отдаёт уже существующее?

[IseeCollapse]

Василий Банников, создаёт новое, постоянное подключение в приложении мне не нужно, только при добавлении новых данных в бд или при их извлечении

[Василий Банников]

IseeCollapse, фатальная ошибка.
Создание соединения сильно дороже, чем держать его постоянно открытым.

+ Если сразу во много потоков будет открыто много соединений без пула - можно выжрать лимит подключений и убить базу

[Василий Банников]

IseeCollapse, на чём у тебя сервер сделан?

[IseeCollapse]

Василий Банников, на чем основано соединение с пользователем? На TCP протоколе. Каким образом выжрется лимит если соединения просто так не открываются? Плюс они сразу же закрываются после отработки методов отправки/извлеченич

[Василий Банников]

IseeCollapse, то есть сервер какой-то кастомный, а не http на aspnet или типа того? И только 1 клиент и он за раз только 1 запрос делает?

[IseeCollapse]

Василий Банников, да сервер кастомный, изучаю разные технологии на практике. Клиентов по TCP может подключаться до 100 пользователей(ограничение установлено ручками, пальцем в небо) запросы идут выполняются в порядке очереди

[Василий Банников]

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

Либо пусть так и остаётся как сейчас - фабрики это ок, но создавайте экземпляр фабрики только 1 раз и передавайте его в конструкторы тех классов, которым нужно соединение.

[IseeCollapse]

Василий Банников, подскажите пожалуйста, как IoC контейнер работает под капотом, он так же будет каждый раз создавать экземпляр класс? Только он сделает весь мой код менее зависимым?

[Василий Банников]

IseeCollapse,

он так же будет каждый раз создавать экземпляр

как настроишь - так и будет