История изменений

Твой код, допустим, будет подключаться в некий готовый сайт и фронтендерам может понадобится заюзать этот атрибут для своих целей, твоя «либа» лишает их этой возможности потому, что ты заюзал id для хранения данных инициализации.

После инициализации id элемента будет иметь значение указанное в id JSON’a для инициализзации. Фронтендеры могут использовать его как обычно.

как ты вообще представляешь себе использование ups_gauges? через нее даже нельзя получить доступ к отдельному индикатору. Они там массивом в порядке инициализации.

Если очень надо будет - то несложно пройтись по этому массиву и найти нужный именно по id. Иметь 100500 индикаторов на странице - идиотизм. Больше 20 уже глупость. Поэтому накладные расходы на этот поиск среди 20 элементов ничтожны. Логика «масштабирования» тут не работает. Число индикаторов на страничке не масштабируется, по объективной причине ограниченности человеческого восприятия. Если тебе требуется 100500 индикаторов на одной страничке, то у тебя проблемы вовсе не с жабоскриптом.

Нужна ли вообще в new Gauges( ‘ups’ ) логика поиска элементов

Да, нужна. Чтобы создавать индикаторы чисто декларативно. Если нужна будет, например, температура рядом с UPS с другого источника данных, то достаточно просто добавить элемент с нужным классом и настройками в id.

а если у человека в его JS уже найдены все нужные элементы и он мог бы просто их передать в Gauges но твой код этого не позволяет

Что мешает переписать Gauges так, как нужно? Чтобы позволяло? Там же 70 строчек кода всего.

Продолжать можно очень долго, такое вот у меня отсутствие аргументации.

Аргументация у тебя уровня «почему без шапки».

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

• нет реализации удаления индикатора
• нет остановки таймера если удалены все индикаторы с одним источником
• разные группы индикаторов с одинаковыми источниками будут дёргать один источник несколько раз и т.д.

Это не мешает использованию и работоспособности в предполагаемом варианте использования, но по-уму такие штуки было бы логично сделать. Но ничего этого ты не увидел, и даже не подумал о них. Всё твоё «ревью» исключительно про то, как этот высер не соответствует твоим представлениям о «стандартах написания кода специалистами по написанию кода на Javascript за деньги». Вот и всё. И именно поэтому - веб катится в страное говно. Никого не интересует что делает код и насколько хорошо он делает то, для чего он собственно и нужен. «Всех» интересует как он написан.

Соответственно, ты и я из совершенно разных миров, где ценятся совершенно разные вещи.

Твой код, допустим, будет подключаться в некий готовый сайт и фронтендерам может понадобится заюзать этот атрибут для своих целей, твоя «либа» лишает их этой возможности потому, что ты заюзал id для хранения данных инициализации.

После инициализации id элемента будет иметь значение указанное в id JSON’a для инициализзации. Фронтендеры могут использовать его как обычно.

как ты вообще представляешь себе использование ups_gauges? через нее даже нельзя получить доступ к отдельному индикатору. Они там массивом в порядке инициализации.

Если очень надо будет - то несложно пройтись по этому массиву и найти нужный именно по id. Иметь 100500 индикаторов на странице - идиотизм. Больше 20 уже глупость. Поэтому накладные расходы на этот поиск среди 20 элементов ничтожны. Логика «масштабирования» тут не работает. Число индикаторов на страничке не масштабируются. Если тебе требуется 100500 индикаторов на одной страничке, то у тебя проблемы вовсе не с жабоскриптом.

Нужна ли вообще в new Gauges( ‘ups’ ) логика поиска элементов

Да, нужна. Чтобы создавать индикаторы чисто декларативно. Если нужна будет, например, температура рядом с UPS с другого источника данных, то достаточно просто добавить элемент с нужным классом и настройками в id.

а если у человека в его JS уже найдены все нужные элементы и он мог бы просто их передать в Gauges но твой код этого не позволяет

Что мешает переписать Gauges так, как нужно? Чтобы позволяло? Там же 70 строчек кода всего.

Продолжать можно очень долго, такое вот у меня отсутствие аргументации.

Аргументация у тебя уровня «почему без шапки».

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

• нет реализации удаления индикатора
• нет остановки таймера если удалены все индикаторы с одним источником
• разные группы индикаторов с одинаковыми источниками будут дёргать один источник несколько раз и т.д.

Это не мешает использованию и работоспособности в предполагаемом варианте использования, но по-уму такие штуки было бы логично сделать. Но ничего этого ты не увидел, и даже не подумал о них. Всё твоё «ревью» исключительно про то, как этот высер не соответствует твоим представлениям о «стандартах написания кода специалистами по написанию кода на Javascript за деньги». Вот и всё. И именно поэтому - веб катится в страное говно. Никого не интересует что делает код и насколько хорошо он делает то, для чего он собственно и нужен. «Всех» интересует как он написан.

Соответственно, ты и я из совершенно разных миров, где ценятся совершенно разные вещи.