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