Почему статические методы в js это плохо?

Question

[Сергей Котов]

Пишу фронт на Angular. Была задача хранить данные фильтров в localStore, для этого я создал класс с статическими методами:

export class FilterStorageUtil {
  private static storage = window.localStorage;
  private static prefix = 'StorageFilter-';

  public static setFilter<T extends object>(filter: T, key = location.pathname) {
    const itemKey = this.prefix + key;
    this.storage.setItem(itemKey, JSON.stringify(filter));
    return true;
  }

  public static getFilter(key = location.pathname) {
    const itemKey = this.prefix + key;
    const jsonFilter = this.storage.getItem(itemKey);
    try {
      return JSON.parse(jsonFilter);
    } catch (e) {
      return null;
    }
  }
}

Мой коллега сказал мне что это зло и так писать нельзя якобы память жрет, и нужно использовать сервисы. На сколько это оправдано?

Comments

[Lynn «Кофеман»]

А какой смысл в этом классе?
С тем же успехом можно просто две функции написать

[Сергей Котов]

Lynn «Кофеман», возможно буду расширять код

[Антон В.]

Конкретно в это реализации, в памяти будет создана storage(ссылка на localStorage )и prefix (строка) + два метода
Как тут может быть утечка памяти?

[Антон В.]

Может быть ваш коллега имел ввиду какую ту конкретную ситуацию. Попросите его объяснить "на пальцах". возможно он сам не разбирается.

Answer 1

[Владимир]

Статические методы в JS это прекрасно и они позволяют экономить память. Статические методы в вашем коде это плохо, потому что конкретно в этом случае лучше использовать сервис.

Comments

[Сергей Котов]

а если я буду расширять код будет, какой-нибудь умный метод итд?

[Владимир]

Сергей Котов, не понял ни слова.
если вы сделаете StorageUtil сервисом, то вы можете использовать один инстанс в нескольких компонентах, вы можете регулировать какие компоненты будут шарить этот инстанс между собой, а какие будут создавать новый. Почитайте документацию для чего используются сервисы.

[Сергей Котов]

Владимир, Получается всегда создается инстанс при вызове StorageUtil.getFilter()?

[Demian Smith]

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

[Владимир]

Demian Smith, Напишите статический метод не используя тайпскрипт и классы.

[Demian Smith]

Владимир, как-то так, наверное:

function MyClass() {};
MyClass.staticMethod = function() {};

Я, признаться, не уверен как GC догадается освободить память от MyClass. Можете показать куда смотреть? Ато я туплю.

[Владимир]

Как только в коде не удет ссылок на MyClass так сразу GC его и соберет

[Demian Smith]

Владимир, я может что-то путаю, но MyClass - это разве не глобальная переменная?

В первом пункте https://javascript.info/garbage-collection#reachability как раз об этом говорятся. Или я куда-то не в ту степь полез?

[Антон В.]

Demian Smith, Зависит от того где объявлен MyClass. В корне или в контексте.

[Антон В.]

Demian Smith,

Например в таком случае GC очистит MyClass после того как станет недостежимым (App.service_one = null;)

importClass(){
function MyClass() {};
MyClass.staticMethod = function() {};
return MyClass
}

let App = {
service_one: importClass()
}

App.service_one = null;

[Demian Smith]

Антон В., что-то я об этом не подумал. Спасибо!