и добавь pre-commit hook👀

а, ну и самое любимое: <Search {...{ search, someProp, handleChange: e => handleChange(e), }} />

а, ну и самое любимое: <Search {...{ search, someProp, handleChange: e => handleChange(e), }} />

👮🏿‍♂️

У всех разные стандарты, у нас за двойные стреляют в упор

За двойные в аттрибутах?

да, но есть только у меня лол

Беги оттуда

Беги оттуда

уже

завтра

Красава

🙈🙊🌚

Здесь есть 3 ошибки, сможете найти все?

Здесь есть 3 ошибки, сможете найти все?

Тут достаточно наследования от реализации

Первая

Здесь есть 3 ошибки, сможете найти все?

Наследование, object, вызов setState с тем же state,

Наследование, object, вызов setState с тем же state,

нет

А, туплю. Это ж переопределенный метод

Горячо

Нет коллбэка

Неплохо, хороший поинт, но он не входил в 3 ошибки

Пусть это будет экстра

Пусть это будет экстра

Переопределение метода, а потом вызов этого метода у родителя

state может быть не только объектом, но и функцией

Не в этом проблема, вызов метода родителя нормально для ООП, вы же наследуете логику.

Переопределение метода, а потом вызов этого метода у родителя

Это как раз норма при переопределении

🌚а использование ооп норма? :trollface

🌚а использование ооп норма? :trollface

Да

state может быть не только объектом, но и функцией

И тогда expanded всегда будет undefined

🌚а использование ооп норма? :trollface

Это другая тема, холеварная, я скажу что не норма для Реакта вообще

🙈🙊🌚

1. Наследование другого компонента 2. ? 3. ? Extra: не поддерживает весь интерфейс метода setState.

Здесь есть 3 ошибки, сможете найти все?

Вызов конструктора родителя?

Вызов конструктора родителя?

Зачем?

Дикая херня творится со стейтом в методе setState

а нужно вообще setState менять?

А нужно ли вообще об этом говорить?

а Object почему в список ошибок не включил?

Автора этого кода лишить права писать код и дело с концом

Поцчему там Object, а не типы, определённые для стейта/контекста?

а Object почему в список ошибок не включил?

Это flow, и он на этом проекте никогда не работал, так что можно опустить.

Этот код работает но выкидывает 1 ворнинг.

в смысле, это вообще не должно компилиться

Object это конструктор объекта, а не ts-овский тип

При чем тут ts?

а хз. привычка.

Это другая тема, холеварная, я скажу что не норма для Реакта вообще

Судя по тому, что PropTypes в реакте, хуками там даже не пахло, так что ваиантов особо не было

Судя по тому, что PropTypes в реакте, хуками там даже не пахло, так что ваиантов особо не было

Это 15 версия, но дело далеко не в этом, и хуки тут не при чем

Это 15 версия, но дело далеко не в этом, и хуки тут не при чем

Это я в тему ооп

Да согласен в 15 версии без классов тяжело, но наследование нужно забыть навсегда здесь.

1. Наследование другого компонента 2. ? 3. ? Extra: не поддерживает весь интерфейс метода setState.

Ошибки синтаксиса или проектирования?

Ошибки синтаксиса или проектирования?

проектирования

Я не уверена что правильно в setState вызывать функцию из props

Как-то это странно выглядит

Второй раз спрашиваю а вообще переопределять setState нормально?

Этого ещё не было?)

Не думаю что оно вообще заведётся если переопределить

Почему не заведется?

Второй раз спрашиваю а вообще переопределять setState нормально?

За такое убивают

Второй раз спрашиваю а вообще переопределять setState нормально?

что тут нормального? если никто так не делает

По крайней мере перерендера внутри этого компонента можно не ждать

🙈🙊🌚

Это с продакшена или че

Второй раз спрашиваю а вообще переопределять setState нормально?

если возникают такие вопросы значит что-то пошло не так )

Это с продакшена или че

Да

propTypes не определены.

За такое убивают

Почему убивают? Какие это проблемы за собой влечет?

@pizdec_js

Тут автор это чата)

Это надо поглубже знать react) расскажешь сам? я могу предположить что так как изменение состояния через собственный setState вызывает перерендер, то тут его не будет. Не будут отслеживаться ссылки. Но тут и нет рендера никакого. Кстати, третья ошибка, наверное) рендер же обязательный

Это надо поглубже знать react) расскажешь сам? я могу предположить что так как изменение состояния через собственный setState вызывает перерендер, то тут его не будет. Не будут отслеживаться ссылки. Но тут и нет рендера никакого. Кстати, третья ошибка, наверное) рендер же обязательный

рендер у родителя

Или наследодателя? Как оно называется?))

рендер у родителя

А он же обязательный у всех классовых компонентов, не?

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

А он же обязательный у всех классовых компонентов, не?

Да, но он может быть определен у Collapsible в этом примере

🙈🙊🌚

То что тут происходит называется inheritance inversion в реакте (почти)

Да, но он может быть определен у Collapsible в этом примере

А, логично) он же тоже может быть наследован. Туплю

То что тут происходит называется inheritance inversion в реакте (почти)

И это норм решение только для определенных кейсах

То что тут происходит называется inheritance inversion в реакте (почти)

Расскажи

Это надо поглубже знать react) расскажешь сам? я могу предположить что так как изменение состояния через собственный setState вызывает перерендер, то тут его не будет. Не будут отслеживаться ссылки. Но тут и нет рендера никакого. Кстати, третья ошибка, наверное) рендер же обязательный

Тут на самом деле реакт глубоко знать не обязательно, достаточно знать что setState это метод API React и его модификация является манки-патчингом, Должен ли CollapsibleScrollable знать о реализации класса Scrollable, определенно что нет, но он влияет на его работу. Так же как и класс Scrollable не знает о том, что кто-то его унаследует и будет модифицировать API реакта, которому он доверяет. У CollapsibleScrollable есть интерфейс как с ним взаимодействовать и это то, что он ожидает и контролирует. Если вы вдруг пришли из ООП и захотели унаследовать компонент, значит вам нужна композиция. https://ru.reactjs.org/docs/composition-vs-inheritance.html

А что такое манки-патчинг?

Подмена методов

Хах, в профункторе как раз в тему смехуёчек

А что такое манки-патчинг?

Лучше прочитать где-нибудь в википедии