МГ
Кто-то забыл _
забыл)))
спасибо!)
спасибо!)
Size: a a a
2020 August 26
МГ
Но тогда непонятен момент с "был такой код, даже тесты под него написаны, сейчас открыл и уже ошибка"
тоже не знаю, это вещь на тестах работала
типо тесты под это делце написал и они проходили
ну ,видимо, я невнимательный и где-то что-то исправил)
типо тесты под это делце написал и они проходили
ну ,видимо, я невнимательный и где-то что-то исправил)
ДБ
ДБ
или как то так?
https://i.imgur.com/6W9hzkq.png
https://i.imgur.com/6W9hzkq.png
ДБ
и ещё
я написал простой MailReader
но в самом mailBox получается какая-то каша, как можно улучшить что стоит вынести?
https://gist.github.com/Git-Drive/13e40144da0925a415164ed54a380c83#file-fs-L61
Мне кажется что getClient не стоит выносить потому как он нужен только в обработке Receive и наоборот кажется что его лучше вынести чтоб не мешался в коде обработки
когда делаю модуль каша только растет, а как закинуть логгер в тот же mailBox не понятно, статья с FP DI сбила с толку
я написал простой MailReader
но в самом mailBox получается какая-то каша, как можно улучшить что стоит вынести?
https://gist.github.com/Git-Drive/13e40144da0925a415164ed54a380c83#file-fs-L61
Мне кажется что getClient не стоит выносить потому как он нужен только в обработке Receive и наоборот кажется что его лучше вынести чтоб не мешался в коде обработки
когда делаю модуль каша только растет, а как закинуть логгер в тот же mailBox не понятно, статья с FP DI сбила с толку
ДБ
ещё и костыль с авто диспоузом
Dv
Вот этот вариант выглядит лучше, если нет требований к его обобщению.
Отмечу, что такое использование
Отмечу, что такое использование
Dictionary выглядит подозрительно. Уж не ConcurrentDictionary ли ты хотел?ДБ
Вот этот вариант выглядит лучше, если нет требований к его обобщению.
Отмечу, что такое использование
Отмечу, что такое использование
Dictionary выглядит подозрительно. Уж не ConcurrentDictionary ли ты хотел?хотел бы его, но подумал что я в MailBox могу не бояться
ДБ
сделал ревизию - выпилил MailBoxProcessor, заюзал ConcurrentDictionary и стало все проще
Dv
Вообще, я бы вынес какой-то другой примитив для словаря, типа
getOrCreate (key) (factory)ДБ
странно git gist потерял мою первую версию
ДБ
можешь глянуть новую?
ДБ
новая версия влезла в 20 строк
ДБ
из за Async.AwaitEvent
ДБ
правильно ли я подменил Cancel Token?
ДБ
чтоб отмена с верху меня не бахнула
Dv
правильно ли я подменил Cancel Token?
Посмотри
Async.StartChild, там встроенная поддержка таймаутов есть.ДБ
Он unit возвращает
ДБ
Вроде смотрел