Оцените тестовое задание (junior)?
const keys = ["name", "my-city-id", "family-status",
    "Number", "e-mail"
];
for (let id of keys) {
    // выставил
    let input = document.getElementById(id);
    input.value = localStorage.getItem(id);
    console.log("сохранено");
    // закинул
    let fun = function(id, input) {
        input.addEventListener("change", function() {
            localStorage.setItem(id, input.value);
        });
    };
    fun(id, input);
}[
  'e-mail',
  'family-status',
  'my-city-id',
  'name',
  'Number',
].forEach((id) => {
  /** @type {HTMLInputElement} */
  const inputElement = document.getElementById(id);

  inputElement.value = localStorage.getItem(id);
  inputElement.addEventListener(
    'change',
    () => localStorage.setItem(id, inputElement.value),
  );
});
Разница понятна?
Я бы еще это оформил в виде функции с названием, объясняющим суть происходящего.
Я бы советовал webp использовать, добавит оптимизации вам и советовал бы не подключать из вне скрипт, лучше себе их скачать.<script src="https://cdn.jsdelivr.net/npm/simplebar@latest/dist/simplebar.min.js "></script>