@react_js

Страница 2083 из 5115
Max
04.09.2017
08:17:30
если появляется желание сделать this.renderSmth(type)

то для меня это сигнал что компонент делает дохуя

и надо сделать сабкомпоненты

Sergey
04.09.2017
08:19:25
и надо сделать сабкомпоненты
Там просто switch, который пихает нужную вещь

Google
Max
04.09.2017
08:21:05
звучит как контейнер для сабкомпонентов ?

Sergey
04.09.2017
08:21:32
Ну есть в этом правда

Max
04.09.2017
08:21:50
а кстати

серега

слыш

Stepan
04.09.2017
08:21:53
степ, давай без хейта
Извини, просто пригорело с постановки. "Ребята, не вызывается, помогите" "Да дело не в этом"

Max
04.09.2017
08:22:11
возьми рекомпоуз

Sergey
04.09.2017
08:22:16
Если подробнее, то это график, и у него есть пропа type, с которой он рендерит либо линейный, либо столибки

Max
04.09.2017
08:22:20
вооо

Emir
04.09.2017
08:23:37


Sergey
04.09.2017
08:25:03
Так вот тебе и два компонента
Слишком жирно, там только меняется между <Bar/> и <Group><Line/><Scatter/></Group>

Stepan
04.09.2017
08:26:49
Никогда не видел ничего плохого в бОльшей декомпозиции

Google
Emir
04.09.2017
08:26:54
не актуально, проблема была в jest'e, он отваливается из-за axios в окружении --env=jsdom

Sergey
04.09.2017
08:29:28
Никогда не видел ничего плохого в бОльшей декомпозиции
Там помимо этого есть еще вещи, которые делают свое дело. Два компонента с повторением кода, ну такое...

Max
04.09.2017
08:40:43
сергей

https://github.com/acdlite/recompose/blob/master/docs/API.md#branch

Sergey
04.09.2017
08:45:03
https://github.com/acdlite/recompose/blob/master/docs/API.md#branch
Угу, уже сижу курю маны. Тэнкс. Только исходный вопрос не совсем исчерпан: если есть функция которой передаются пропс, их лучше архументом или в ней их тягать?

Max
04.09.2017
08:45:17
убирай классы ?

вопрос пропадет

Sergey
04.09.2017
08:46:11
Чет невъехал о чем ты...

Max
04.09.2017
08:46:46
функциональные компоненты юзай

у них нет this

Ilya
04.09.2017
08:47:33
не используй аргументы

пусть функция сама к this.props обращается и всему что ей нужно

Sergey
04.09.2017
08:48:47
функциональные компоненты юзай
Стараюсь в полной мере, если это stateless компонент

Max
04.09.2017
08:49:45
я бы делал через аргументы

хотя

похеру

Sergey
04.09.2017
08:51:07
Ну вот так и у меня было: "так, а хотя нет, вот так.... хотя вообще похеру"

Ilya
04.09.2017
08:51:15
чтобы не засорят render()

плюс сегодня один аргумент, завтра два

Google
Ilya
04.09.2017
08:51:40
ну и вынести потом в отдельный компонент проще

Sergey
04.09.2017
08:52:07
плюс сегодня один аргумент, завтра два
Ну а разве с пропсом не тоже самое?)

Max
04.09.2017
08:52:08
Ilya
04.09.2017
08:52:41
Ну а разве с пропсом не тоже самое?)
в случае с аргументами тебе нужно будет сначала вызов поправить, а потом в функции аргумент добавить

сложна!

Max
04.09.2017
08:53:21
хотя не

все-таки через аргументы лучше

в отдельный компонент если выносишь - первоый строчкой декомпозицию из пропсов добавляешь

Ilya
04.09.2017
08:53:54
ой, все.

Max
04.09.2017
08:54:05
прозрачнее

Sergey
04.09.2017
08:54:31
Я просто исходил из того, что это чем-то похоже на React.createElement

И я пихаю туда "пропсы"

Sergey
04.09.2017
08:54:52
Ну типо пропсы

Max
04.09.2017
08:54:57
а чем компонент не?

вот прямо в том же файле положи стейтлесс

вместо renderHui <Hui>

у тебя код основного компонента будет сильно проще

и не на 20 экранов

Сергей
04.09.2017
08:56:17
парни, свежий вопрос

Sergey
04.09.2017
08:56:29
Если в тот же файл, примерно тоже по строкам

Google
Сергей
04.09.2017
08:56:30
впадлу ли юзать isFunction/isArray/isString из lodash?

Sergey
04.09.2017
08:56:37
Хотя потом можно будет вынести, если вдруг вырастет

Сергей
04.09.2017
08:56:53
или для феншуя лучше typeof() === 'function'?

Max
04.09.2017
08:57:04
Если в тот же файл, примерно тоже по строкам
зато они будут ниже основного класса

и не надо продираться через 100500 renderX

Sergey
04.09.2017
08:57:18
Если у тебя lodash есть, так почему бы не юзать?

и не надо продираться через 100500 renderX
Ок, согласен... ДЕКЛАРАТИВНОСТЬ ЕП ВАШУ МАТЬ!

Ilya
04.09.2017
08:57:43
с массивами дрочиться долго, он и объект и массив одновременно

так что лучше _.isArray

Admin
ERROR: S client not available

Сергей
04.09.2017
08:58:07
Если у тебя lodash есть, так почему бы не юзать?
мне кажется это выглядит несколько по-ламерски

Andrey
04.09.2017
08:58:14
А ещё лучше импортить отдельно функции

Сергей
04.09.2017
08:58:29
так что лучше _.isArray
есть же Array.isArray

Sergey
04.09.2017
08:58:31
мне кажется это выглядит несколько по-ламерски
Мноха бойлерплейта при проверках каждый раз

Сергей
04.09.2017
08:58:45
для string/function просто typeof === 'string/function'

дрочиться нужно только с plainobject

Ilya
04.09.2017
08:58:54
_.isArray быстрее писать

Sergey
04.09.2017
08:59:02
А, и да, это читабельнее

Andrey
04.09.2017
08:59:16
Как же бесит это слово

Google
Andrey
04.09.2017
08:59:19
читабельнее.

Sergey
04.09.2017
08:59:24
А как надо?

Andrey
04.09.2017
08:59:29
Есть русское слово читаемее

Max
04.09.2017
08:59:29
нормально

Сергей
04.09.2017
08:59:30
_.isArray быстрее писать
тебе сперва еще надо это заикнлудить

Andrey
04.09.2017
08:59:42
читабельный, блин

Max
04.09.2017
08:59:46
читаемее это не то

Ilya
04.09.2017
08:59:46
один раз в начале файла

Max
04.09.2017
08:59:53
читаемее это количественное

Алексей
04.09.2017
08:59:55
Max
04.09.2017
09:00:04
читабельнее - качественное

Andrey
04.09.2017
09:00:07
Sergey
04.09.2017
09:00:12
читабельный, блин
https://ru.wiktionary.org/wiki/%D1%87%D0%B8%D1%82%D0%B0%D0%B1%D0%B5%D0%BB%D1%8C%D0%BD%D1%8B%D0%B9

Экзактли))0))0)0))

Max
04.09.2017
09:00:38
> Сейчас завелось хорошее слово: читабельный. Л. Я. Гинзбург, «Записные книжки. Воспоминания. Эссе», 1920–1943

хы

Сергей
04.09.2017
09:00:42
читабельнее - качественное
думаешь лучше лодаш? ну так-то да, читабельнее

Max
04.09.2017
09:00:54
какой лодаш

мы тут про грамманаци

Andrey
04.09.2017
09:01:06
Ок

Сергей
04.09.2017
09:01:07
isFunction/isArray

Andrey
04.09.2017
09:01:08
Ушёл гореть

Страница 2083 из 5115