Я хотел бы рассмотреть возможность избежать динамического объекта, а лучше создать класс хранения данных (SettingsModel?) С одним методом хранения и одним методом для получения данных. Поэтому вместо использования:
configurationSettings.randomSetting = value;
... вы бы написали:
configurationSettings.store("randomSetting", value);
Вы можете хранить именованные настройки внутри словаря и выполнять любую магию, необходимую для метода store()
.
Основным преимуществом здесь является удобочитаемость: когда вы видите вызываемый метод, вы знаете, что за ним стоит функциональность. По той же причине я склонен не любить функции получения / установки.
В конце концов, это дело вкуса, поэтому, если ваш код не сломан, зачем это исправлять. :)