В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

GraphQL — русскоговорящее сообщество

939 membersпожаловаться на группу
2019 January 04

EM

Eugene Maltsev in GraphQL — русскоговорящее сообщество
componentWillReceiveProps  - депрекейтед
🤔да и default export нынче нежелательно юзать, https://blog.neufund.org/why-we-have-banned-default-exports-and-you-should-do-the-same-d51fdc2cf2ad
Medium
Why we have banned default exports in Javascript and you should do the same
ES2015 was the most important improvement to Javascript in years. Among many great features it brought brand new module system — Ecma…
источник
14:03пожаловаться#1

EM

Eugene Maltsev in GraphQL — русскоговорящее сообщество
странный гитхаб:(
источник
14:04пожаловаться#2

EA

El Anonimo in GraphQL — русскоговорящее сообщество
componentWillReceiveProps() уже deprecated? Думал, это с 17-й версии.
источник
14:04пожаловаться#3

EA

El Anonimo in GraphQL — русскоговорящее сообщество
Его недолго заменить на getDerivedStateFromProps().
источник
14:06пожаловаться#4

EA

El Anonimo in GraphQL — русскоговорящее сообщество
Пойнт не в этом. Чел выше указал на наличие недостатков и не указал на них самих.
источник
14:06пожаловаться#5

EA

El Anonimo in GraphQL — русскоговорящее сообщество
Просто в какой строке случаются.
источник
14:07пожаловаться#6

EA

El Anonimo in GraphQL — русскоговорящее сообщество
Eugene Maltsev
на бекенде всегда должна быть валидация:)
ну и на фронте не помешает, да и redux-form говно. Лучше уж formik/final-form
Имелось ввиду, что там на фронте её нет вовсе. Объект передаётся на сервер и валидируется там.
источник
14:16пожаловаться#7

s

sho? in GraphQL — русскоговорящее сообщество
El Anonimo
Был бы признателен за указание в репе, где случаются указанные пункты.
чутьь позже, щас с телефона просто :)
источник
14:21пожаловаться#8

s

sho? in GraphQL — русскоговорящее сообщество
https://github.com/ElAnonimo/DevConnector/blob/master/client/src/components/create-profile/CreateProfile.js#L38
Вот пример пробрасывания пропсов в стейт.
По-хорошему, это должен быть не один компонент, а контейнер без верстки и дочерний stateless component, который только получает пропсы и не имеет стейта.

https://github.com/ElAnonimo/DevConnector/blob/master/client/src/components/create-profile/CreateProfile.js#L120
Вот этого в рендер-методе вообще быть не должно. Зачем там эта мапа? почему ее не вынести в атрибут класса?


https://github.com/ElAnonimo/DevConnector/blob/master/client/src/components/create-profile/CreateProfile.js#L207
https://github.com/ElAnonimo/DevConnector/blob/master/client/src/components/post/CommentItem.js#L28
инстанциирование функций-обработчиков прямо в компоненте тоже антипаттерн.

Ну и так далее.
GitHub
ElAnonimo/DevConnector
Contribute to ElAnonimo/DevConnector development by creating an account on GitHub.
источник
14:31пожаловаться#9

EA

El Anonimo in GraphQL — русскоговорящее сообщество
Понятнее, спасибо.
Надо было вынести компонент в функциональный (presentational) и подать его родителю.
источник
14:42пожаловаться#10

EA

El Anonimo in GraphQL — русскоговорящее сообщество
С мапой тоже просто поленился.
источник
14:42пожаловаться#11

EA

El Anonimo in GraphQL — русскоговорящее сообщество
Про последнее, у меня не ту строку открыл браузер.
источник
14:50пожаловаться#12

EA

El Anonimo in GraphQL — русскоговорящее сообщество
Вы. наверно, об этом.
button onClick={this.onDeleteClick.bind(this, postId, comment._id)} className="btn btn-danger mr-1">
Вот этот bind().
источник
14:50пожаловаться#13

EA

El Anonimo in GraphQL — русскоговорящее сообщество
Прочитал на Medium, почему. Это было использовано мной, чтоб не делать конструктор.
источник
14:51пожаловаться#14

EA

El Anonimo in GraphQL — русскоговорящее сообщество
Более ленивый способ.
источник
14:51пожаловаться#15

EA

El Anonimo in GraphQL — русскоговорящее сообщество
К тому же, это CommentItem. Item, т. е. компонент, получающий пропсы от родителя.
источник
14:53пожаловаться#16

s

sho? in GraphQL — русскоговорящее сообщество
Все это могло быть (и наверное было) расценено как плохой код. Потмоу и оценка джуниор плюс :)
источник
14:53пожаловаться#17

EA

El Anonimo in GraphQL — русскоговорящее сообщество
Вероятно. Спасибо за комментарии, в любом случае.
источник
14:54пожаловаться#18

EA

El Anonimo in GraphQL — русскоговорящее сообщество
У меня там и дальше есть левоватые хаки.
источник
14:54пожаловаться#19

s

sho? in GraphQL — русскоговорящее сообщество
El Anonimo
Прочитал на Medium, почему. Это было использовано мной, чтоб не делать конструктор.
кроме того, не обязательно делать конструктор, можно использовать arrow functions as class properties
источник
14:54пожаловаться#20