
Vladimir
08.01.2017
22:00:37
и не за feature bloat
just a disclaimer

Vasiliy
08.01.2017
22:04:31
тут чел кстати упоминает jsx-control-statements (: https://github.com/facebook/jsx/issues/65#issuecomment-255319680
https://github.com/facebook/jsx/issues/65#issuecomment-254073018
кстати https://github.com/facebook/jsx/issues/65#issuecomment-254077743

Roman
08.01.2017
22:05:51
один handleOnChange в компоненте это нормально если не надо там ничего менять в значении поля?

Google

Adel
08.01.2017
22:14:05
можно вообще в отдельный файлик выкинуть сразу, никакого html-in-js сразу
сразу сразу

Vladimir
08.01.2017
22:17:33
Что я скинул

Adel
08.01.2017
22:18:25
он немного про другое, как я понял
и все равно остается html-in-js
react-templates вроде больше в духе ng-if
меньше тегов, больше аргументов
кстати, всем привет

Sergey
08.01.2017
22:21:55
hi

Google

Adel
08.01.2017
22:23:45

Дмитрий
08.01.2017
22:29:23

Roman
08.01.2017
22:42:02

Дмитрий
08.01.2017
22:42:57
Вот так всегда
Выкинь этот мануал)

from
08.01.2017
22:57:27
да норм мануал
onClick={() => this.setState({ value: 'X' })} — наглядно
не у всех ререндер приложения происходит по 50 раз в секунду

Дмитрий
08.01.2017
22:58:36
А потом у них реакт медленный и без scu ничего не робит

from
08.01.2017
23:01:00
громкое заявление, сомнительно, что новые функции в рендере могут заметно замедлить приложение в одиночку
вот условно такое же мы пишем:
renderButton({ color, text, className }), а это создание нового объекта при каждом рендере между прочим)

Таймураз
08.01.2017
23:02:05
Насколько раз подобную вещь оставить в коде, а потом при росте проекта лаги начинаются

from
08.01.2017
23:02:26
или ещё хуже!
<span>${user.name} ${user.lastname}</span>
конкатенация строк при каждом рендере!

Таймураз
08.01.2017
23:03:30
Вот говорят, экономия на спичках
А спустя время этих спичек уже коробок

Дмитрий
08.01.2017
23:03:47
++

from
08.01.2017
23:03:51
я тоже в общем-то привязанные функции всегда в конструкторе создаю, и никогда в рендере новые не создаю
Но говорить, что плох мануал, который использовал такой синтаксис — неправильно

Дмитрий
08.01.2017
23:04:07
Ну вот видишь

Таймураз
08.01.2017
23:04:33

Дмитрий
08.01.2017
23:04:37
Мануал который обучает антипаттернам — не плох?
Ну ок
А потом тут сеньеры со стрелками в рендере

Google

from
08.01.2017
23:05:05
ну я давно об этом задумывался
должна ли каждая строчка в туториалах быть production-ready?
в какой-то момент хотелось злобно написать статью о том, что должна

Дмитрий
08.01.2017
23:05:38
Мануал не должен обучать антипаттернам просто так
Без оговаривания этого
Люди тебе верят, повторяют твой код

Таймураз
08.01.2017
23:06:32

from
08.01.2017
23:06:32
Оговаривать надо, я согласен.
Но если захламлять код непонятными edge-кейсами — плохой туториал выйдет

Таймураз
08.01.2017
23:07:23

from
08.01.2017
23:07:43
почему?
об этом

Таймураз
08.01.2017
23:07:47

Таймураз
08.01.2017
23:08:06
Пускай эти точки будут правильно расставлены

Дмитрий
08.01.2017
23:08:12

from
08.01.2017
23:09:08
Короче
— Антипаттерны в туториалах — плохо
— Оговаривать non-production код — надо
— onClick={() => this.doSomething()} — не однозначно плохо
я спать)

Дмитрий
08.01.2017
23:09:42
Ну чо ты))
Вместо понятного краткого onClick={this.doSomething} какая-то фигня, к тому же являющаяся антипаттерномъ

Roman
08.01.2017
23:11:04
grabs popcorn ну вы еще подеритесь)

from
08.01.2017
23:12:18
надо уже измерить этот кейс в девтулзах наконец

Google

Roman
08.01.2017
23:13:25
мне дали официальный ман, и вы говорите что он не правильный, и как мне чувствовать себя?
я впринципе за то что бы неререндерить безтолку

Таймураз
08.01.2017
23:14:00
Изучай дальше, тебе все равно рано думать об этом

Roman
08.01.2017
23:14:21
но если фб такое написало может они там прикрутили чего чтобы неренедилось безтолку или как?

Таймураз
08.01.2017
23:14:30
Учиться сразу писать идеальный код- не всегда правильно
Точнее всегда неправильно

Roman
08.01.2017
23:15:14
ну как бы сказать - питон учит именно идеальному коду, selfexplicit так сказать. а вот php..

Таймураз
08.01.2017
23:15:54
Питон близок к этому, и все же разные по
Программисты на том же питоне по-разному решат задачу

Roman
08.01.2017
23:16:07
комюнити питона такого что потом понимать начинаешь объективно где что правильно а где нет )

Дмитрий
08.01.2017
23:16:36

Admin
ERROR: S client not available

Roman
08.01.2017
23:16:44
идти в сторону никто не запретит, но хотя бы будешь знать верный путь ) а там уж под свою отвественность

Дмитрий
08.01.2017
23:16:51
Там вроде везде где надо нормальный код

Таймураз
08.01.2017
23:16:53

Roman
08.01.2017
23:17:22
да вроде я от туда и скопировал
когда про callback'и искал там

Дмитрий
08.01.2017
23:18:29
Я кажется нашел
Там узкий edge case описывался
В единственом месте
grep callback не всегда является разумным методом поиска инфы

Google

Roman
08.01.2017
23:19:45
ну unix way
:D

Дмитрий
08.01.2017
23:19:54
If you aren't using property initializer syntax, you can use an arrow function in the callback:
А меж тем хороший способ был описан как раз до этой строчки)

Roman
08.01.2017
23:20:43
вы там из контекста вытащили я просто искал в доке как вообще callback послать

Дмитрий
08.01.2017
23:21:03
Ой всё
))

Roman
08.01.2017
23:22:32
а теперь думаю как мне гуманно вернуть ошибку в форму, всмысле с аксиоса на уровне console.log могу, а вот как вернуть так что бы я в onSubmit сходу знал ошибка или там все ок без излишних if'ов

Дмитрий
08.01.2017
23:23:24
this.setState наверн

Roman
08.01.2017
23:24:01
да видать до state еще надо понять что получил
в onSubmit вызвал это https://dpaste.de
catch + promises..?
ладно забейте
добавил к вебпаку --hot и чето ниче не изменилось: все равно страницу перезагружать надо

? Алёна
08.01.2017
23:33:17
Попробуй добавить в фаил где у тебя рут рендерится,
if (module.hot) {
module.hot.accept();
}
Ну и ещё запускай с webpack-dev-server --inline --watch --hot

Roman
08.01.2017
23:34:05
кто то советует inline другие ругают

? Алёна
08.01.2017
23:34:56
У тебя два дня. Тебе надо сделать, а разбираться как там что тру и не тру потом будешь

Roman
08.01.2017
23:35:17
?
inline после hot должен идти - забавно и причем не конфиге вебпака, а именно в командной строке (package.json)

? Алёна
08.01.2017
23:46:22
ммм. http://andrewhfarmer.com/webpack-hmr-tutorial/ я включала его по этому гайду. там несколько вариантов есть, попробуй каждый из них и посмотри что подойдёт.

Roman
08.01.2017
23:46:24
хера се как у вас тут все запущено :D