Хранение данных в localstorage?

Question

[prolina]

const logOut = () => {
    const user = JSON.parse(localStorage.getItem("User"));
    users.push(user);
    localStorage.setItem("Users", JSON.stringify(users));
    console.log(users.length);
    window.location = './form.html';
};

const logOutBtn = document.getElementById('log-out');
logOutBtn.addEventListener('click', logOut);

const password = document.getElementById('password');
const username = document.getElementById('username');

const loginUser = (username, password) => {
    const user = {
        userName: username.value,
        password: password.value
    };
    localStorage.setItem("User", JSON.stringify(user));
};
const submitBtn = document.getElementById('submit-btn');
submitBtn.addEventListener('click', loginUser.bind(null, username, password));

В данный момент массив получается из одного юзера. При logout в массив перезаписывается новый юзер каждый раз. Как сделать, чтобы при нажатии на logout, в массив добавлялся новый пользователь и тот, кто был ранее?

Comments

[dollar]

Ну, изменить скрипт так, чтобы так был массив юзеров.
Как бы очевидно. Но это задание, а не вопрос. И уж точно не вопрос по самому языку javascript. Вы хотите, чтобы кто-то для вас накодил за бесплатно?

[Cr2ed]

Так вычитайте на логауте из localstorage "Users", добавте туда текущего и сохрание это значение в localstorage.

Answer 1

[Сергей Соколов]

Способом, как у вас 2-я строка, получите сохранённый ранее массив users, и в него пушьте.

Рекомендации:

1. Названия ключей LS лучше держать в константах, а не писать строкой каждый раз.
   
2. Предусмотрите вариант, когда в LS ничего нет – с этим точно столкнётесь впервые с массивом users
   
3. Предусмотрите вариант, что LS в принципе недоступно - например в режиме Private Browsing.
   

Comments

[prolina]

Сергей Соколов, Сделала вот так

const logOut = () => {
    const user = JSON.parse(localStorage.getItem("User"));
    if(JSON.parse(localStorage.getItem("Users")=== null)) {
        const users = [];
        users.push(user);
        localStorage.setItem('Users', JSON.stringify(users));
    } else {
        const users = JSON.parse(localStorage.getItem("Users"));
        users.push(user);
        localStorage.setItem("Users", JSON.stringify(users));
    }
    window.location = './form.html';
};

[Сергей Соколов]

prolina, почти хорошо. Дублируются две строчки кода – попробуйте этого избежать. "Users" и "User" лучше однажды назначить в константы и потом использовать их:

const keyUsers = "Users";
// ...
localStorage.getItem(keyUsers)

Эти две строки можно сократить:

const users = [];
users.push(user);
// то же, что:
const users = [user];

[prolina]

Сергей Соколов, спаибо за рекомендации и за помощь)