В хелпер вынесу

вот и решение

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

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

hoc + props

имхо

Как?

Ну просто пуш заменить на concat и присваивать в то же свойство. Обычная иммутабельность. Конкат же не только массив, но и одно значение может принимать

Не понимаю где иммутабельность

Не понимаю где иммутабельность

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

linkedUsersIds = linkedUsersIds.concat(item.responsibleUser.Id);

я бы на варианте с 4 мапами остановился хотя бы из того, что он явный + массивы в const

linkedUsersIds = linkedUsersIds.concat(item.responsibleUser.Id);

И где иммутабельность?

Мы же переприсваиваем linkedUserIds лол

И где иммутабельность?

Как это где? Конкат не мутирует массив, он создает новый.

И чо?

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

Мы же переприсваиваем linkedUserIds лол

Это и есть иммутабельность если ты не знал

ну, скажем, ты эти массивы из props получил (из стора)

ты их копируешь - не мутируешь

а мутабельность внутреннего свойства функции - отдельный разговор

Ты в immutable.js как делаешь? myMap = myMap.set(key, prop); или это тоже мутабельность?

Ну да блядь ты же переприсываешь переменную

МУТИРУЕШЬ её

а как же композиция функций...

а как же композиция функций...

?

МУТИРУЕШЬ её

Сейчас она у тебя X, через 3 строчки Y

Перезаписывать новое значение в переменную не значит ее мутировать

а мутабельность внутреннего свойства функции - отдельный разговор

конечно же я имел в виду внутреннюю переменную скоупа функции, а не то, что вы подумали

@evgenyrodionov

R.pipe(R.map(...), R.filter(...), R.uniq, R.filter(...))(items)

Сейчас она у тебя X, через 3 строчки Y

по определению мутация данных про изменение данных по ссылке, а не смену ссылки в контексте

https://ru.wikipedia.org/wiki/%D0%98%D0%B7%D0%BC%D0%B5%D0%BD%D1%8F%D0%B5%D0%BC%D1%8B%D0%B9_%D1%82%D0%B8%D0%BF

но const предпочтительно, это уже отдельный разговор

Да, ссылка заменяется на новое значение и переменная не мутирует.

R.pipe(R.map(...), R.filter(...), R.uniq, R.filter(...))(items)

поправь меня, если я не прав, но это же ни на что не влияет, кроме формата записи?

R.pipe(R.map(...), R.filter(...), R.uniq, R.filter(...))(items)

Не использую рамбду

ни на производительность, ни на оптимизацию компилятором (опять производительность)

выглядит не так страшно зато

Если мне нужно что посложнее, тот мне проще мап использовать

R.pipe(R.map(...), R.filter(...), R.uniq, R.filter(...))(items)

Вкусовщина как по мне. Проблему с четырьмя вызовами это все равно не решает

R.pipe(R.map(...), R.filter(...), R.uniq, R.filter(...))(items)

не работал с рамдой, в чем отличие этого от lodash/flow?

Вкусовщина как по мне. Проблему с четырьмя вызовами это все равно не решает

добавляет гибкости при написании тех же хелперов

то есть ты можешь допередать, например, функций в лоток

не работал с рамдой, в чем отличие этого от lodash/flow?

В рамде больше всяких функциональных инструментов, чем в _. А flow ты flowtype имеешь ввиду? Это же вообще про другое, это типы.

окей, функция flow из лодэш

В рамде больше всяких функциональных инструментов, чем в _. А flow ты flowtype имеешь ввиду? Это же вообще про другое, это типы.

lodash/fp же еще есть

import flow from 'lodash/flow'

и тд

добавляет гибкости при написании тех же хелперов

Но в данном случае-то один хрен

Ладно, другой вопрос

Но в данном случае-то один хрен

мало ли чего потом еще приспичит добавить. через запятую проще чем через точку

Как с этими ебучими переносами разобраться нормально

ты про аттрибуты или про теги?

или не про кодстайл речь?

Про кодстайл да

если про аттрибуты, имхо это правильный вид

Как с этими ебучими переносами разобраться нормально

Поставь линтер с конфигом от airbnb

Не

А потом поднастрой под свой вкус

Если поставлю, то полпроекта придётся переписывать

Во треш

У меня уже есть свой линтер

Хочу его выкинуть, но не сейчас

Если поставлю, то полпроекта придётся переписывать

Да, у меня тоже с одним проектом такая шняга, но его все равно рефакторить надо. Новые проекты я пишу со своим конфигом (airbnb + строчек 30 моих) и радуюсь жизни. Все четко и красиво)

Есть такой вариант, но чё-т хз

ERROR: S client not available

У меня есть в соседнем файле

Вот у меня примерно так

И вот это абсолютный беспросветный пиздос

Каждое свойство на новой строке в смысле

А чё делать с теми компонентами, где чилдрен есть

А на том проекте примерно так))) поэтому его и надо рефакторить

Ну типа если делать так, то говно же

Но и в line-width 80 нужно вписываться, не хочу повторения этого

Можно чилдрен еще вправо сдвинуть

Тут пока к сожалению идеального кодстайла не придумали

Можно чилдрен еще вправо сдвинуть

это не очень однородно будет вообще проблема времен флеш/флекс

Но и в line-width 80 нужно вписываться, не хочу повторения этого

Я наоборот рад, что меня линтер заставляет это делать, только у меня по-моему 100, не помню точно

Ну так и я про это же)

не буду говорить о том, где я все это уже раньше видел, но там мы решали это таким образом

Не хочу повторения каши

чилды на обычном отступе, аттрибуты - по первому аттрибуту

не буду говорить о том, где я все это уже раньше видел, но там мы решали это таким образом

В пхп?)))

не, флеш/флекс ж говорю же

компонентный веб откуда есть пошел

От Delphi)))

переносы - по длине строки

Взяли и в веб перетянули идею, да и все

rich web начался с флеша, все, что я щас вижу нового в вебе, было во флеше

во флексе точнее

прям жду какого-нибудь очередного rpc типа amr

переносы - по длине строки

Не читабельно

компромисс между длинной кода и кашей

Сделал так

Сойдёт