문제가 있냐는 말씀이 MVVM 관점에서 MVVM에 위배되냐는 의미라면, 엄밀히 말해 MVVM 관점에서는 문제가 있는 방식이라고 생각합니다. 테마는 단순히 사용자에게 어떻게 보여줄지에만 집중하는 Presentation Logic이니까요. ViewModel이 아니라 View에 있어야 할 내용이죠.
테마를 변경하는 방법이 문제가 있냐는 의미라면, 문제 없다고 생각됩니다. 일반적으로 실시간으로 테마를 변경할 때 첨부하신 방법으로 구현하는 것으로 알고 있습니다.