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

Size: a a a

☄️ effector

2020 March 23

в эффекторе граф нод визуализируется в тестах, чтобы убедиться что мы не наплодили сами себе кучу проблем в этой области https://github.com/zerobias/effector/blob/master/src/effector/__tests__/__snapshots__/index.test.js.snap

там кстати такой визуализатор тоже есть, просто выключен

источник

03:41пожаловаться #1

Valeriy Kobzar in ☄️ effector

la gente está muy loca

код визуально вполне делится на две секции

не понятно зачем там return result.map.....

источник

03:58пожаловаться #2

Valeriy Kobzar in ☄️ effector

можно же сразу в result пихать нужное

источник

03:58пожаловаться #3

Valeriy Kobzar in ☄️ effector

и избежать лишнего прохода по массиву

источник

03:59пожаловаться #4

la gente está muy loca in ☄️ effector

придётся вызывать getGraph в трёх местах в коде, по одному на каждый пуш

источник

03:59пожаловаться #5

la gente está muy loca in ☄️ effector

но наблюдение хорошее, возможно так даже лучше 👍

источник

04:02пожаловаться #6

la gente está muy loca in ☄️ effector

Valeriy Kobzar

можно же сразу в result пихать нужное

да, короче сейчас так и сделаю) спс

источник

04:03пожаловаться #7

la gente está muy loca in ☄️ effector

Valeriy Kobzar

не понятно зачем там return result.map.....

а, блин, тогда тут всё равно мап появится

источник

04:05пожаловаться #8

la gente está muy loca in ☄️ effector

но на том скрине этот фрагмент кода не уместился, да)

источник

04:05пожаловаться #9

Valeriy Kobzar in ☄️ effector

la gente está muy loca

а, блин, тогда тут всё равно мап появится

есть ощущение, что там можно применить рекурсивный вызов это й функции

источник

04:06пожаловаться #10

Valeriy Kobzar in ☄️ effector

но я пока не уверен

источник

04:06пожаловаться #11

la gente está muy loca in ☄️ effector

Valeriy Kobzar

есть ощущение, что там можно применить рекурсивный вызов это й функции

да, скорее всего

источник

04:06пожаловаться #12

Valeriy Kobzar in ☄️ effector

const arrifyNodes = (list: Graphite | Graphite[] = [], result = []): Graph[] => {
if (Array.isArray(list)) {
for (let i = 0; i < list.length; i++) {
if (Array.isArray(list[i]))
arrifyNodes(list[i], result)
else
result.push(getGraph(list[i]))
}
} else {
result.push(getGraph(list))
}
return result
}

источник

04:10пожаловаться #13

Valeriy Kobzar in ☄️ effector

не оно?

источник

04:10пожаловаться #14

Valeriy Kobzar in ☄️ effector

хотя хз, если это на производительность не влияет то возможно лучше map оставить

источник

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

Valeriy Kobzar in ☄️ effector

в конце концов map понятнее, а списки вряд ли будут настолько огромными, чтобы думать об этом