Телеграмм чат группы projs

Привет, это Vue, но это не важно, graph это вычисляемое свойство для графика. Перепробовав варианты с forEach, я перешёл на map и оставил пока такой вариант.

Как я могу улучшить этот код, сделать рефакторинг, может кто заметит сразу более подходящее решение? Спасибо!

источник

15:21пожаловаться #7

PV

Peter V in pro.js

Artemy S

Привет, это Vue, но это не важно, graph это вычисляемое свойство для графика. Перепробовав варианты с forEach, я перешёл на map и оставил пока такой вариант.

Как я могу улучшить этот код, сделать рефакторинг, может кто заметит сразу более подходящее решение? Спасибо!

Ну как бы рефакторить 20 строчек это такое себе. Если тебе нужно по оформлению совет то я бы сохранил длинное дублирующееся выражение до map в отдельную переменную

источник

15:34пожаловаться #8

AS

Artemy S in pro.js

Спасибо

источник

15:35пожаловаться #9

PV

Peter V in pro.js

Ну и непонятно зачем писать данные для вебнастера и не вебнастера в разные проперти

источник

15:36пожаловаться #10

PV

Peter V in pro.js

У тебя же не может быть сценария когда ты одновременно и тот и тот

источник

15:36пожаловаться #11

PV

Peter V in pro.js

Artemy S

Привет, это Vue, но это не важно, graph это вычисляемое свойство для графика. Перепробовав варианты с forEach, я перешёл на map и оставил пока такой вариант.

Как я могу улучшить этот код, сделать рефакторинг, может кто заметит сразу более подходящее решение? Спасибо!

А ты класть в объект graph что то еще будешь?

источник

15:42пожаловаться #12

AS

Artemy S in pro.js

Это зависит от роли пользователя, разная роль = разные данные

источник

15:43пожаловаться #13

AS

Artemy S in pro.js

Могу скинуть, из какого объекта я делаю и что

источник

15:43пожаловаться #14

PV

Peter V in pro.js

Artemy S

Это зависит от роли пользователя, разная роль = разные данные

Имхо не имеет смысла держать 2 разные проперти т к одна из них всегда будет пустая, тебе в любом случае надо будет обращаться к роли пользователя чтобы понять какую использовать

источник

15:45пожаловаться #15

PV

Peter V in pro.js

Artemy S

Могу скинуть, из какого объекта я делаю и что

скинь код лучше текстом

источник

15:45пожаловаться #16

AS

Artemy S in pro.js

Это не два разных map