Roman Mejtesдля того, чтоб следовать принципу единственной ответственности, поступайте следующим образом. попробуйте описать необходимый для реализации класс 1 предложением:
Класс, который реализует всю функциональность приложения

Я вам не Димон.В общем всё работает, но вопрос в том - нормальная ли это практика, что объект (экземпляр класса) загружает сам себя из файла (и изменяется при этом) ?
Если хорошо решает стоящие в этом месте задачи и не вызывает проблем - это адекватное решение. Если плохо решает стоящие в этом месте задачи или вызывает проблемы - это неадекватное решение. "Нормальной практики", вообще говоря, не существует, это просто сокращённые правила для тупых уровня "не используй goto". Такое решение есть куда улучшать. А вот нужно ли улучшать - зависит от того, где и как применяется.