@react_js

Страница 4626 из 5115
John
16.08.2018
09:07:06
231 строка закомментирована - в массиве числа. Раскомментирована - могут быть и строки

John
16.08.2018
09:08:22
цикл for чего стоит
Чего же он стоит?

Сергей
16.08.2018
09:09:20
Чего же он стоит?
ощущение, что про методы .reduce, .map, .filter ты вообще не в курсе

Google
John
16.08.2018
09:10:28
ощущение, что про методы .reduce, .map, .filter ты вообще не в курсе
В курсе и считаю, что тут они неуместны

Сергей
16.08.2018
09:10:38
ну тогда страдай в одиночестве



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

John
16.08.2018
09:12:10
В чем ад, если число может прийти и как "123456", и как "123 456", и как "123'456", и как "123 456,23", и как "123 456.23" и еще миллион всяких вариантов

Сергей
16.08.2018
09:12:34
В курсе и считаю, что тут они неуместны
вообще-то здесь как раз уместен .map

Читай книгу Мартина “Чистый код”

John
16.08.2018
09:13:03
такое на кодревью должно быть завернуто и создана встреча с автором, чтобы поговорить о чистом коде
Предложи чистый код для решения этой задачи, мне в голову не приходит, как это сделать проще

Сергей
16.08.2018
09:13:10
И весь этот код, может быть заменен на 5-7 качественного кода

John
16.08.2018
09:13:55
затем, что такое надо выносить в отдельную функцию и тестировать
Отдельная функция работает отлично, проблема тут не в конвертации числа

Сергей
16.08.2018
09:14:20
Отдельная функция работает отлично, проблема тут не в конвертации числа
если бы ты написал эту часть нормально, то искать проблемы было бы легче.

Google
Сергей
16.08.2018
09:14:34
будет весело, если окажется, что проблема вообще не в этом куске кода

Alexey
16.08.2018
09:14:47
Друзья, кто подскажет, как называется компонент карточки на флипяарте, с возможностью указывать стрелки от 1 карточки к другой?

John
16.08.2018
09:14:53
она уже не в нём, лодаш как-то мутирует

Alexey
16.08.2018
09:16:18
Нет, наподобие chatfuel доски для состояний

Сергей
16.08.2018
09:17:06
Нет, наподобие chatfuel доски для состояний
нормально опиши че тебе надо

Игорь
16.08.2018
09:17:15
http://joxi.ru/823oZxjCJq3JP2 Сыч подскажи, мне кажеться что я делаю что то не то когда смотрю на этот метод...

Alexey
16.08.2018
09:17:23
Если видели такое, вроде майнд мэп, но с карточками трелло, если смог описать нормально

pepsi ¯\_(ツ)_/¯
16.08.2018
09:17:49
Alexey
16.08.2018
09:18:23
нормально опиши че тебе надо
Нужно сделать дерево диалога, фраза - одна карточка

Сергей
16.08.2018
09:18:31
пишется на раз

Alexey
16.08.2018
09:19:19
пишется на раз
За основу что-то можно взять?

Сергей
16.08.2018
09:19:28
John
16.08.2018
09:19:53
И весь этот код, может быть заменен на 5-7 качественного кода
И так, и сяк посмотрел, не может это быть написано на map из-за того, что у меня итератор используется для создания идентификатора

Roman
16.08.2018
09:19:53
?

Google
Игорь
16.08.2018
09:21:00
учи javascript
Сыч подскажи, у меня все норм со стилем, просто мне не нравиться этот метод.

Игорь
16.08.2018
09:22:07
Сергей
16.08.2018
09:22:27
Игорь
16.08.2018
09:23:59
открой био
Сергей ? просто сова не бывает мужского пола, я думал между сычем, и филином )

Игорь
16.08.2018
09:25:41
У меня имя Сергей, фамилия Сова
ахахах не ловко получилось ) сори ) Можешь глянуть можешь глянуть мой кусочек кода. Что думаешь о моем стиле ?

Roman
16.08.2018
09:27:47
Roman
16.08.2018
09:28:05
нет единой ответственности

Игорь
16.08.2018
09:28:49
абсолютно не понятно назначение этого кода
странно звучит, но название метода имеет смысл в контексте задачи. Есть список меток, и их можно выбрать не корректно

Сергей
16.08.2018
09:30:48
странно звучит, но название метода имеет смысл в контексте задачи. Есть список меток, и их можно выбрать не корректно
во первых его можно вынести в отдельную функцию ибо здесь зависимость только от пропс (если я не слепой) уже будет чище

John
16.08.2018
09:32:06
Roman
16.08.2018
09:33:11
я не могу смотреть на твой код, поставь себе eslint пожалуйста

Сергей
16.08.2018
09:33:50
Вынести, из компонента всмысле? А во вторых?
да. во вторых, ощущение, что можно переписать чуть более понятно. Сейчас код как регулярка, поправить может только автор. Хер пойми что там внутри .map. Не легко воспринимаемый код

Google
John
16.08.2018
09:34:05
я не могу смотреть на твой код, поставь себе eslint пожалуйста
Стоит. А что не так? Фигурные скобки на одном уровне не любишь?

Сергей
16.08.2018
09:35:09
Стоит. А что не так? Фигурные скобки на одном уровне не любишь?
нормальный поставь. это выглядит максимально меркзо. бля, хотя бы преттиер, если не в состоянии нормальный стиль кода выбрать

John
16.08.2018
09:35:20
ты не на C пишешь
У заказчика нет требования к google code style, а значит можно использовать более удобную стилистику

John
16.08.2018
09:35:38
Admin
ERROR: S client not available

Roman
16.08.2018
09:35:43
а потом говорят, почему js разрабы такие токсик

Eugene
16.08.2018
09:36:02
google code style?

Сергей
16.08.2018
09:36:18
Если подправить его без стрелочной в мапе то код раздувается строк на 10
возможно, есть смысл вынести стрелки в функции с говорящим названием (мб и нет) также может разбить сравнения внутри на именованные константы

Roman
16.08.2018
09:36:18
Сергей
16.08.2018
09:36:51
У заказчика нет требования к google code style, а значит можно использовать более удобную стилистику
удали нахуй свой стиль кода и поставь prettier.io и срать я хотел на то, что тебе не нравится ты либо пишешь как человек в большом сообществе либо не обращаешься к сообществу

Roman
16.08.2018
09:36:54
google code style?
https://github.com/google/eslint-config-google

Eugene
16.08.2018
09:38:03
?airbnb уже не в моде?

Roman
16.08.2018
09:39:13
Andrey
16.08.2018
09:39:35
?airbnb уже не в моде?
Проще самому собрать, чем их правила терпеть или переписывать.

John
16.08.2018
09:39:49
удали нахуй свой стиль кода и поставь prettier.io и срать я хотел на то, что тебе не нравится ты либо пишешь как человек в большом сообществе либо не обращаешься к сообществу
Код пишется или как удобнее программисту, или под требования заказчика. Очень много людей пишут в такой стилистике, я один из них

Google
Roman
16.08.2018
09:40:10
Проще самому собрать, чем их правила терпеть или переписывать.
конечно проще потратить несколько дней своей жизни читая доку eslint

Andrey
16.08.2018
09:40:29
За пару часов спокойно.

Roman
16.08.2018
09:40:36
John
16.08.2018
09:40:38
Eugene
16.08.2018
09:41:23
можно сказать, что чел пишет код так, чтобы написать и забыть про него и никогда не возвращаться к нему и плевать на дальнейшее развитие проекта ?

Andrey
16.08.2018
09:41:25
Roman
16.08.2018
09:42:15
Andrey
16.08.2018
09:42:24
Roman
16.08.2018
09:42:34
команда смотрит на стайлгайды сообщества и пишет по нему

у меня в проекте airbnb и никто не против

Andrey
16.08.2018
09:42:52
команда смотрит на стайлгайды сообщества и пишет по нему
Нет, команда строит стайлгайд как ей удобно.

Eugene
16.08.2018
09:42:55
?в команде может быть свой стайлгайд

Arthur
16.08.2018
09:42:58
Вы доебались из-за скобочек, серьёзно ?

Roman
16.08.2018
09:43:18
?в команде может быть свой стайлгайд
может, но за основу берется стайлгайд от сообщества

Andrey
16.08.2018
09:43:32
Мне и команде удобнее табы. Взяли в проект табы. И пусть сообщество со своим мнением идёт лесом.

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