5

Например у меня есть сервисы ScannerService и UsersService. Первый отвечает за управление сканером на устройстве, второй получает данные из базы данных и содержит в себе объект DbContext (реализация UoW) для работы с базой данных. На разных формах используются разные сервисы, но эти два почти на всех. Стоит ли создавать новые объекта в каждой форме, или лучше использовать Singleton?

Можно ли объект, реализующий Unit of work, делать Singleton? Он вроде должен быть уничтожен после выполнения работы сервиса, в котором используется, но для меня не очевидно зачем его создавать по сто раз для каждого сервиса, если можно создать один раз и уничтожить после выполнения работы приложения.

Есть ли причины не делать сервисы статическими, если мне не нужно создавать их объекты, а нужно лишь использовать их функции, которые можно сделать статическими?

5
  • Нет, не нужно, а если сервис содержит в себе DbContext - то не просто "не нужно" а "запрещено".
    – user177221
    14 сен 2017 в 12:01
  • @PashaPash скажите пожалуйста почему?
    – Kunoichi
    14 сен 2017 в 12:06
  • 3
    Метка c++ тут для каких целей присутствует?
    – αλεχολυτ
    14 сен 2017 в 12:30
  • @alexolut я написал языки, на который сейчас пишу
    – Kunoichi
    14 сен 2017 в 12:42
  • у вас первая половина вопроса (DataContext) специфична для .NET. Не стоит ставить на вопрос метку любой-язык, если к нему не применимо решение на каком-нибудь brainfuck :)
    – user177221
    14 сен 2017 в 13:04

1 ответ 1

5

Объект DbContext кэширует в себе все выбранные через него объекты (т.е. реализует не только Repository, UoW, но еще и Identity Map).

Соответственно, если вы косвенно сделаете его синглтоном, то ваше приложение

  • закэширует в себе все выбираемые через себя данные (всю базу)
  • перестанет видеть изменения, вносимые в базу извне
  • не сможет восстановиться после первой же ошибки валидации / сохранения (упал один SaveChanges - упадут и все следующие)
  • будет дико тупить при сохранении из-за необходимости обнаруживать изменения в большом количестве объектов

Нет никаких причин делать сервисы или вообще любые другие классы статическими, если того явно не требует логика работы конкретных классов. Т.е. статическим синглтоном может быть какой-то глобальный кэш, который по определению должен быть один на один инстанс приложения. Или, например, класс, оборачивающий в себе доступ к стороннему сервису, и управляющий соединениями к нему (пул соединений SQL, мультиплексор StackExchange Redis и т.п.). И даже в этом случае единственность экземпляра должен обеспечивать контейнер, а не прямая статика.

Все остальное должно быть нестатическим, или, по крайней мере, не должно предполагать статичности/единственности зависимостей в своем коде.

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.