@react_js
Oleg ?16.05.2017
08:56:35
08:56:35
https://github.com/georgeOsdDev/react-web-notification
Dreamerinnoise16.05.2017
08:58:21
08:58:21
Не знаю, зачем для такой херни нужна была либа, ну да ладно...
Dmitrii16.05.2017
08:59:48
08:59:48
Всем привет. Вопрос по redux.
Мне нужно удалить состояние в редуксере.
Я делаю так:
delete state[action.id];
return {
...state
};
Это нормально? Какие могут быть подводные камни. Не лучше ли делать сначала создать новый объект, а потом у него удалить свойство?
Спасибо
Если я все правильно понял, то ты мутируешь стейт, чего делать нельзя идеологически
Ruslan16.05.2017
09:01:20
09:01:20
Всем привет. Вопрос по redux.
Мне нужно удалить состояние в редуксере.
Я делаю так:
delete state[action.id];
return {
...state
};
Это нормально? Какие могут быть подводные камни. Не лучше ли делать сначала создать новый объект, а потом у него удалить свойство?
Спасибо
Делай копию стейта, ее и мутируй
GoogleRuslan16.05.2017
09:03:05
09:03:05
или части стейта
Dmitrii16.05.2017
09:03:21
09:03:21
const newState = {...state}; delete newState[id]; return newState;
Max16.05.2017
09:05:21
09:05:21
Всем привет. Вопрос по redux.
Мне нужно удалить состояние в редуксере.
Я делаю так:
delete state[action.id];
return {
...state
};
Это нормально? Какие могут быть подводные камни. Не лучше ли делать сначала создать новый объект, а потом у него удалить свойство?
Спасибо
return {
…state,
[id]: undefined
}
Сергей16.05.2017
09:06:36
09:06:36
Max16.05.2017
09:06:51
09:06:51
вот давайте рамдой обмазываться, ага )
Сергей16.05.2017
09:07:19
09:07:19
whynot?
Max16.05.2017
09:07:42
09:07:42
whyyes?
Dmitrii16.05.2017
09:07:43
09:07:43
return {
…state,
[id]: undefined
}
Тогда в hasOwnProperty попадет. Если это не важно, то можно так, да
? ethorzMax16.05.2017
09:08:54
09:08:54
мусье знает толк
Viktor16.05.2017
09:22:01
09:22:01
Вообще Дэн рекомендует soft delete
GoogleMax16.05.2017
09:22:33
09:22:33
где там стикер про абрамова?
Viktor16.05.2017
09:22:47
09:22:47
*дэн.жпег*
https://github.com/paularmstrong/normalizr/issues/21
К слову о flow и лени описывать тип.
Сделал так https://habrahabr.ru/company/docsvision/blog/326602/#comment_10204750
Кто мне скажет - не херню ли я сделал?
https://pastebin.com/G8spvkPZ
Oleg ?16.05.2017
09:27:14
09:27:14
Мне вот интересно, вам в кайф описывать эти длинные типы?
Viktor16.05.2017
09:27:38
09:27:38
Описывать не в кайф
Oleg ?16.05.2017
09:27:49
09:27:49
А зачем вы вообще это делаете?
Max16.05.2017
09:28:01
09:28:01
чтобы ошибок меньше было
Oleg ?16.05.2017
09:28:02
09:28:02
Типо меньше ошибок с промахом типов?
Max16.05.2017
09:28:07
09:28:07
угу
Viktor16.05.2017
09:28:10
09:28:10
Затем что команда большая и проект большой
Oleg ?16.05.2017
09:28:14
09:28:14
Для этого существуют юнит тесты не?
Viktor16.05.2017
09:28:32
09:28:32
вам в кайф писать юнит тесты?
Сергей16.05.2017
09:28:37
09:28:37
А положа руку на сердце, часто такая ошибка возникает?
Oleg ?16.05.2017
09:28:41
09:28:41
Нам да
Viktor16.05.2017
09:28:48
09:28:48
на банальные вещи
Max16.05.2017
09:28:49
09:28:49
export class Message {
type = this.constructor.name;
}
GoogleOleg ?16.05.2017
09:28:50
09:28:50
Потому что ошибки сразу замечаются
Max16.05.2017
09:28:52
09:28:52
забавно
Oleg ?16.05.2017
09:29:15
09:29:15
Чем раньше вы обнаружите баги, тем лучше
Oleg ?16.05.2017
09:29:58
09:29:58
У тебя строки длинные становятся
Max16.05.2017
09:30:14
09:30:14
ты не видел мои строки в ERP на питоне
Oleg ?16.05.2017
09:30:17
09:30:17
И это не очень, каждый раз описывать тип
code4aman16.05.2017
09:30:26
09:30:26
> Для этого существуют юнит тесты не?
> Чем раньше вы обнаружите баги, тем лучше
чет не понял))
Oleg ?16.05.2017
09:31:02
09:31:02
Юнит тесты способствуют обнаружению простых логических ошибок и промахов типов
code4aman16.05.2017
09:31:06
09:31:06
тесты раньше интерпретатора типа фидбек дадут?)
Oleg ?16.05.2017
09:31:16
09:31:16
Типо того
Dreamerinnoise16.05.2017
09:32:17
09:32:17
Честно не понимаю людей, которые любят динамическую типизацию в js
Dreamerinnoise16.05.2017
09:32:32
09:32:32
как они живут? тесты весь день пишут?
Oleg ?16.05.2017
09:32:40
09:32:40
Что IDE?
Oleg ?16.05.2017
09:33:03
09:33:03
Нет, не раньше
Googlecode4aman16.05.2017
09:33:30
09:33:30
Max16.05.2017
09:33:46
09:33:46
короче надо reason
Oleg ?16.05.2017
09:33:49
09:33:49
Короче, пишите свои флоу типы, вас никто не заставляет перейти на юнит тесты)
Max16.05.2017
09:33:51
09:33:51
чтобы допилили
Сергей16.05.2017
09:34:01
09:34:01
Max16.05.2017
09:34:03
09:34:03
тебе как пример привели )
а не говорят что тесты не нужны )
AdminERROR: S client not available
Dreamerinnoise16.05.2017
09:34:19
09:34:19
Короче, пишите свои флоу типы, вас никто не заставляет перейти на юнит тесты)
статик тайпинг не исключает написание юнит тестов...Сергей16.05.2017
09:34:27
09:34:27
Короче, пишите свои флоу типы, вас никто не заставляет перейти на юнит тесты)
ты не шаришь))
юнит тесты должны тестировать только то, что сложно понять из кода
Алексей16.05.2017
09:34:28
09:34:28
Короче, пишите свои флоу типы, вас никто не заставляет перейти на юнит тесты)
поддерживаю, люди живут в утопии что типы спасут ихcode4aman16.05.2017
09:34:33
09:34:33
как раз поинт в том что это не взаимоисключающие вещи
Сергей16.05.2017
09:34:45
09:34:45
Сергей16.05.2017
09:34:51
09:34:51
плот в наводнении не спасет)
Oleg ?16.05.2017
09:34:51
09:34:51
Просто вы написали единственное приемущество строгой типизации - меньше ошибок
Max16.05.2017
09:35:03
09:35:03
Oleg ?16.05.2017
09:35:09
09:35:09
Я подумал, что юнит тесты тоже справляются с этой задачей
Max16.05.2017
09:35:10
09:35:10
подсказки там
GoogleСергей16.05.2017
09:35:23
09:35:23
Просто вы написали единственное приемущество строгой типизации - меньше ошибок
так оно для этого и придумано!
Но ты меня не читал: подсказки ИДЕ, вывод типов, большее пониманиеАлексей16.05.2017
09:35:26
09:35:26
не спасут, но помогут в спасении
ну по мне так корректность типов не равна корректности программы, вывод тебе все равно писать тестыMax16.05.2017
09:35:27
09:35:27
Сергей16.05.2017
09:35:40
09:35:40
ну по мне так корректность типов не равна корректности программы, вывод тебе все равно писать тесты
но не проверять тестами типы аргументов)Oleg ?16.05.2017
09:35:51
09:35:51
На данный момент 46% покрытие
Сергей16.05.2017
09:35:56
09:35:56
лол
Oleg ?16.05.2017
09:36:23
09:36:23
Причём тут у меня, я просто предлагаю варианты, чтобы узнать больше в этой теме)
Oleg ?16.05.2017
09:37:01
09:37:01
Хороший холивар, можно понять зачем что делается и какие у этого плюсы
Dreamerinnoise16.05.2017
09:37:07
09:37:07
http://fsharpforfunandprofit.com/posts/ten-reasons-not-to-use-a-functional-programming-language/
<сарказм>
Oleg ?16.05.2017
09:38:06
09:38:06
Хорошо, закрыли тему)
Алексей16.05.2017
09:38:42
09:38:42
ну например с тем что понятнее программа с типами, тоже не факт, особенно дженерики когда
Открыть в Telegram