Плохое ли подключение в конструторе бд?
Лучше создать отдельный класс, который будет хранить в себе объекты (экземпляр подключения к БД и т.д.)SingletonService LocatorRepository
Тем плохо, что кол-во экземпляров этих классов будет равно кол-ву открытых соединений с БД. Плюсом экземпляры класса могут умереть неожиданно, а соединения будут висеть не закрытыми, что не очень хорошо.
Нужно стремиться к тому, чтобы было минимальное кол-во соединений с бд.
Denis Derepko  дал правильное направление тебе.