@react_js

Страница 3170 из 5115
Виталий
15.02.2018
09:17:39
а в чем проблема?
выглядит как лишние операции, хотя я хз че у тебя там ф-ция делает

Max
15.02.2018
09:18:03
ну во-первых не у меня

во-вторых он уже выше писал что это простой селектор из стейта

там просто лежит где-то по пути id

Google
Nikolay
15.02.2018
09:20:10
а нафига делать вызов replyFormId если потом его рендер зависит от сравнения с пропсом парент, у тебя там под капотом битки не фармятся случаем?
parentId - id комментария в котором лежит форма, он статичный для каждоый формы replyFormId - лежит в стейте, он динамичный, дальше mapStateToProps он никуда не девается

Max
15.02.2018
09:20:39
неколай

Nikolay
15.02.2018
09:20:45
НЕ!

Max
15.02.2018
09:20:47
три раза спросили

Nikolay
15.02.2018
09:20:53
м?

Max
15.02.2018
09:20:55
юзаешь ли ты pure

в CommentsForm

или shouldComponentUpdate

или где там у тебя ререндер идет

Виталий
15.02.2018
09:22:07
можно сжедать хок компонент который будет показывать когда прошел рендер

Виталий
15.02.2018
09:22:39
export const log = (BaseComponent) => (props) => { console.log(`Rendering ${BaseComponent.name}`); return <BaseComponent {...props} />; };

и обернуть посмотреть что у тебя рендерится

Google
Andrey
15.02.2018
09:22:59
Виталий
15.02.2018
09:22:59
потом ответить себе на вопрос

а оно должно

а я как понял у чела лишние рендеры там где они ему не нужны

Andrey
15.02.2018
09:23:35
1) У тебя код неверный.

Nikolay
15.02.2018
09:23:56
юзаешь ли ты pure
юзаю PureComponent

Виталий
15.02.2018
09:23:57
ну да неверный и работающий

Vitaly
15.02.2018
09:24:08
бинго

Andrey
15.02.2018
09:24:22
ну да неверный и работающий
И как же ты это проверил?

Max
15.02.2018
09:25:15
юзаю PureComponent
значит какие-то пропсы меняются, если ты прямо уверен что ререндер везде идет

Nikolay
15.02.2018
09:25:44
ну я console.log залепил внутри render, и вместо двух строк выводит все 500+

Kelin
15.02.2018
09:25:56
можно сжедать хок компонент который будет показывать когда прошел рендер
Боже, хок для лога в консоль, давайте еще хок для массажа ног сделаем

Виталий
15.02.2018
09:26:07
И как же ты это проверил?
меняются пропсы, идет ререндер log

че тут еще может быть

Andrey
15.02.2018
09:26:26
меняются пропсы, идет ререндер log
А если не меняются пропсы, то что?

Виталий
15.02.2018
09:27:09
а если не меняются пропсы, которые замапаны на редакс, то ничего не будет

у Николая мапстейттупропс как бэ

Andrey
15.02.2018
09:27:23
А если свой sCU написан?

Nikolay
15.02.2018
09:27:53
Google
Vitaly
15.02.2018
09:28:05
А в чем проблема с хоком для лога удобно же?

Max
15.02.2018
09:28:54
Виталий
15.02.2018
09:28:56
ребят вы прям помогли пацану

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

Andrey
15.02.2018
09:29:30
Я помню, что был плагин к бабелю, который лишние перерендеры показывал. Жаль, что сломался(

Vitaly
15.02.2018
09:29:39
Нахрена?
что бы смотреть что происходит между хоками, а не писать console.log в рендере

Виталий
15.02.2018
09:29:49
а рекомпоз кто юзает?

Max
15.02.2018
09:29:51
ребят вы прям помогли пацану
я за пассивную агрессию люблю выдавать RO

у нас добрый и позитивный чатик

на будущее

Виталий
15.02.2018
09:30:12
мнение

Max
15.02.2018
09:30:20
ну либа как либа

Виталий
15.02.2018
09:30:23
стоит ли

Max
15.02.2018
09:30:26
иногда полезна, иногда нет

да, мне зашла

Andrey
15.02.2018
09:30:32
стоит ли
Стоит.

Google
Andrey
15.02.2018
09:30:44
Выкидывай логику из компонентов.

Max
15.02.2018
09:30:50
часто позволяет вынести логику из компонентов

да

андрей меня понимает

Виталий
15.02.2018
09:31:36
основную идею мнения понял, спс

Andrey
15.02.2018
09:31:42
Да, withProps великая вещь.

Виталий
15.02.2018
09:32:00
про пассивный агр никакого агра не было, просто я обратил внимание что тема из ответа человеку перешла в некоторый спор

Max
15.02.2018
09:32:29
ну так бы и написал

Виталий
15.02.2018
09:33:07
а я вроде так и написал. видимо не совсем прямо

кстати по теме древа комментариев

Vitaly
15.02.2018
09:34:10
Да, withProps великая вещь.
главное не упарываться, сложные мутации лучше переносить в mapStateToProps(если используется redux)

Виталий
15.02.2018
09:34:11
как я понял там редакс

может где-то при обновлении редбюссором стейта происходит обновление лишнего элемента

родителя или еще чего и дальше реакт срабатывает в рендер

Andrey
15.02.2018
09:34:50
Мне вообще не особо нравится редакс, но лучшего пока не придумал.

Виталий
15.02.2018
09:35:17
да в целом он понятен

лишний код можно на горячие вставки делать

но вот мутация сложного объекта с каскадом ...{ ...{ ...[ не всегда радует

Vitaly
15.02.2018
09:36:37
Не могу тебе ответить. Всё индивидуально.
я про распределение вычислений между тиками, mapStateToProps просиходит до рендера, а не в том же стеке

Google
Vasiliy
15.02.2018
09:47:18
Подскажите есть страница новости, я нажимаю на новость срабатывыет экшен и я получаю Id новости в саге обращаюсь к апи передаю id, получаю данные новости, эти данные толкаю в редюсер и пушем перехожу на детальную новость. Коннектом получаю данные из редюсера и вывожу на экран. Правильно ли так делать? и как быть если на прямую зайти на детальную новость

Nikolay
15.02.2018
09:48:07
пропсы в cwrp приходят те же, и в willUpdate те же

Andrew
15.02.2018
09:52:36
Roman
15.02.2018
09:52:39
Напишите пожалуйста топ 10 либ по вашему мнению, без которых Реакт приложение будет считаться несовременным.

Max
15.02.2018
09:53:23
"современным"

лал

Рома, тебе надо работающее приложение

Max
15.02.2018
09:53:46
а не "современное"

Roman
15.02.2018
09:53:57
Рома, тебе надо работающее приложение
ну типо топ либы для реакт приложения 2к!8

Max
15.02.2018
09:54:12
не сильно с прошлым годом поменялось

Denis
15.02.2018
09:54:18
react-router-2o18

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