Да, вы делаете правильно, разделяя функционал приложения на отдельные классы и используя внедрение зависимостей для их взаимодействия. Это позволяет сделать ваш код более модульным, улучшить его читаемость, поддерживаемость и тестируемость.

То, что вы выносите все, кроме работы с БД, в отдельные классы, также является хорошей практикой. Это поможет разделить заботы вашего приложения на различные слои и упростит поддержку кода.

Что касается конкретного функционала, который стоит выносить в отдельные классы, это зависит от конкретных потребностей вашего приложения. Обычно хорошей практикой является выносить в отдельные классы такие аспекты как работа с файлами, сетью, обработка данных и другие подобные операции.

В целом, следуя принципам SOLID и разделяя функционал на независимые компоненты, вы улучшаете качество вашего кода и делаете его более гибким и масштабируемым.