Сейчас не модно что ли писать onClick={this.props.onClick} а модно так. Что-то я

Question

Сейчас не модно что ли писать onClick={this.props.onClick} а модно так. Что-то я

пропустил?
onClick={() => this.props.onClick()}
https://cdb.reacttraining.com/react-inline-functions-and-performance-bdff784f5578

#frontend #javascript #programming #react.js #russian

0

23.05.2018

6 ответов

19 просмотров

Sergey S

ты каждый раз кидаешь новую пропсу во втором варианте

0

23.05.2018

Андрей Звёздочка

Без типизации лучше второе, потому что кривые аргументы могут прийти.

0

23.05.2018

Sergey Ganin

При каждом рендере создает новый контекст для функции и это плохая практика. Но это не точно, может ктото уточнить?

0

23.05.2018

from earth

что значит модно/не модно сам же привёл статью, где всё аргументировано хорошо

0

23.05.2018

Sergey Sova

пара советов есть на эту тему но все они сводятся к контексту использоания () => this.handleClick() юзается если ты описал хендлер как метод, а не как проперти ещё юзается, когда порядок аргументов не устраивает. например есть некоторые библиотеки которые в onClick бросают value, props, event и тебе хочется поменять порядок onClick={(value, props, event) => props.someClick(event, value)} но всё это полностью убивает PureComponent, так как на каждый рендер будет новая функция. И не дай бог ты в замыкании хендлера прокинул какие-то важные данные, например список. А в sCU компонента этот пропс с функцией не проверяешь, то будут ОЧЕНЬ странные эффекты. кейс onClick={props.handlerClick} лучше использовать в большинстве возможных случаев, он легко проверяется в Pure и sCU. Не городит новых функций. Но в нем сложнее отследить пробрасываемые параметры. У себя в проекте я решил, что в хендлерах всегда должен пробрасывать первым аргументом event

0

23.05.2018

🦜 · Accepted Answer

🦜

не надо писать так onClick={() => this.props.onClick()}

0

23.05.2018

169 похожих чатов

Сейчас не модно что ли писать onClick={this.props.onClick} а модно так. Что-то я

6 ответов

Похожие вопросы