Хорошим тоном как по мне в конструкторе писать код связанный только с конструированием обьекта и проверкой предусловий. Т.е. в вашем случае всё хорошо. Однако я думаю не стоит в конструкторе писать основную логику класса для которого он создавался. То есть если это например класс Order, не следует в конструкторе тянуть список товаров из базы, заполнять корзину и т.д.