Так что, пожалуйста, не "работайте с настройками".
То, что вы делаете там (в вашем первом примере), это взаимодействие с глобальной переменной.Никогда не делай этого, если можешь помочь.Это анти-паттерн.Вы не хотите глобалов в вашем коде.Это делает код сложнее для понимания / труднее для тестирования.Вместо этого, всякий раз, когда вы создаете экземпляр класса, читайте ваши глобальные настройки и добавляйте их в поля вашего класса (желательно в конструкторе).Сам класс не должен знать о вашем хранилище настроек.
Кроме того, НЕ ИСПОЛЬЗУЙТЕ onDestroy (WTF?).Вот примечание от ссылка :
Примечание: не рассчитывайте, что этот метод вызывается как место для сохранения данных!
Вместо этогопоместите эту логику в onPause ().