Как корректно превратить Значения из LocalStorage в Date?

Question

[laravel_creative_3103]

Имеется код, в котором одна из строк выводит ошибку:

get token(): string | null {
    const expDate = new Date(localStorage.getItem('fb-token-exp')); //как корректно взять значения из localStorage, и
//сравнить его со временем так как он здесь выводит о 
    if (new Date() > expDate) {
      this.logout();
      return null;
    }
    return localStorage.getItem('fb-token') as string;
  }

Мой вариант:

const expDate = new Date(localStorage.getItem('fb-token-exp') as string) // через as;
    if (new Date() > expDate) {
      this.logout();
      return null;
    }

Корректно ли так будет писать?

Comments

[WbICHA]

То есть ты хочешь сказать, что проверять на наличие значения ненадо?

[StepsOnes]

Да, ваш вариант корректный. Вы используете оператор `as` для приведения типа значения из `localStorage` к строке, которую принимает конструктор `Date()`, а затем создаете новый объект `Date` с полученным значением. Таким образом, вы сможете сравнить дату из `localStorage` со временем выполнения кода.

[laravel_creative_3103]

WbICHA, то есть может есть другие варианты вытаскивания из localStorage помимо as

[laravel_creative_3103]

StepsOnes, Спасибо!

[WbICHA]

StepsOnes, бред. Его решение небезопасно. Да, оно работает, но это хрень.

[WbICHA]

laravel_creative_3103, для начала хранить токен и экспаер в одном ключе стореджа в виде джейсона.
Потом проверить, что ключ не пустой и при парсинге уже можно заассертить тип объекта (по хорошему это тоже омжно провалидировать, но это излишне). И там уже проверять на экспаер.

Answer 1

[Алексей Повольнов]

type Token = {
  exp?: string;
  token?: string;
}

get token(): string | null {
  const logout = () => {
    localStorage.removeItem('fb-token')
    this.logout();
    return null
  }

  try {
    const token: Token = JSON.parse(localStorage.getItem('fb-token') || "{}")

    if(token?.exp && token?.token) {
      const expDate = new Date(token.exp); //как корректно взять значения из localStorage, и
//сравнить его со временем так как он здесь выводит о
      if (new Date() > expDate) {
        return logout()
      }
      return token.token;
    }
  }
  catch (_) {
    return logout()
  }

  return logout()
}

Comments

[WbICHA]

const token: Token = localStorage.getItem('fb-token')

if(token?.exp && token?.token) {

И давно сторедж у нас объекты возвращает?

[Алексей Повольнов]

WbICHA, виноват очепятался, не удобно писать в здешней "IDE"))