в данный момент, когда у тебя 2 типа - чем проще тем лучше

+

А что она делает тогда, прости?

фабрика не может выбирать тип

как не может?)

если бы фабрика могла выбирать тип по заголовку, то в ней были бы захаркодены правила для каждого из типов

если бы фабрика могла выбирать тип по заголовку, то в ней были бы захаркодены правила для каждого из типов

так и хотим сделать, через Dict

что то типа, if(Header.Contains("Order") return new Order();

правила в словаре?

что то не похоже на правила

так ты можешь компаратор заодно с классом инжектить

Надоело

Пойду напишу гист

ОМГ! 2 класса, нахрена огород?

что то типа, if(Header.Contains("Order") return new Order();

типа dict.Add("order", () => { return new OrderMail();})

я конечно понимаю, что через 20 итераций оно будет круто, но уже через 5 оно будет работать по другому

так ты можешь компаратор заодно с классом инжектить

это как?

отлично! а как создать элемент, когда тебе пришел текст заголовка: "Order #12229 от Васи"?

я конечно понимаю, что через 20 итераций оно будет круто, но уже через 5 оно будет работать по другому

мы рассматриваем все варианты, статиком пока самый быстрый в написании

короче на десяти классах можно и свич-кейсом сделать. и не парить мозг.

вот и я о том же

это как?

сделай словарь <Func..., Type> или лист из tupple<Func, Type>, получил мессэдж, прогнал коллекцию, когда Func вернул тру, то либо рефлексией, либо из пула дёрнул экземпляр Type и выдал запрашивающему, ну и соответственно все типы реализуют один интерфейс, чтоб не кастить

скорость выборки правд O(n)

но можн чё подумать ещё, как колдунство ускорить

походу тут нужно что то еще. я ведь хочу создавать тот производный тип Mail, который соответствует определенной теме письма в ящике. т.е. если письмо "заказ" - создаем объект класса OrderMail, если письмо с ценами - PriceMaile, как это реализовать?

оригинальный текст задачи говорит, что логика выбора сложная, так что простым словарём не обойтись и надо прогонять анализ данных пока не придёшь к успеху

оригинальный текст задачи говорит, что логика выбора сложная, так что простым словарём не обойтись и надо прогонять анализ данных пока не придёшь к успеху

логика выбора - субстринг темы письма

логика выбора - субстринг темы письма

если письмо с ценами - PriceMaile

не уверен

отлично! а как создать элемент, когда тебе пришел текст заголовка: "Order #12229 от Васи"?

не важно, что от васи, главное order

логика выбора - субстринг темы письма

ну вот и делай как я говрю, когда будешь фабрику настраивать, будешь там-же и компаратор вписывать - меньше шанс оьъебаться

не важно, что от васи, главное order

и какой будет код выбора типа, если есть словарь?

ищем вхождение ключа в теме письма. не?

foreach( mail in recieved_mails)

то есть цикл?

и тогда фабрика будет знать как происходит выбор

а это не круто совсем

https://gist.github.com/tmteam/96e702b58e7c650232ecb7fb7c139c07

Вот

foreach( key in dict) if (substring(mail.theme, key) == 1 ) exec dict[key]

ну вот и делай как я говрю, когда будешь фабрику настраивать, будешь там-же и компаратор вписывать - меньше шанс оьъебаться

Абслютно поддерживаю. Фабрика тем и прекрасна, что за её стенами ты можешь что угодно писать, хоть ассемблерный код и сушёные шлюхи

https://gist.github.com/tmteam/96e702b58e7c650232ecb7fb7c139c07

круто.

where T: IMailHandler, new() это не понял

foreach( mail in recieved_mails)

говрят тебе, циклом в фабрике компараторы гоняй ` foreach(var comparator in comparators){ if(comparator(message){ // do something } }

это я и хотел. написал класс, добавил строчку и все работает ))) спасибо.

говрят тебе, циклом в фабрике компараторы гоняй ` foreach(var comparator in comparators){ if(comparator(message){ // do something } }

Это абсолютный аналог моего гиста, только у меня "компаратор" это ключ

Тоесть можно сделать Registrate<T?func<> predicate)

Это абсолютный аналог моего гиста, только у меня "компаратор" это ключ

это я первым делом предложил)

https://gist.github.com/tmteam/96e702b58e7c650232ecb7fb7c139c07

если письмо со спамом будет содержать в теме слово order или author, то вся логика развалится

если письмо со спамом будет содержать в теме слово order или author, то вся логика развалится

это да.

если письмо со спамом будет содержать в теме слово order или author, то вся логика развалится

я представил общее решение исходной задачи. Если нужно что то сложнее - то нужно использовать решение Andrew dementiev, с регистрацие не по ключу, но по предикату

если письмо со спамом будет содержать в теме слово order или author, то вся логика развалится

вернет null при парсинге письма

https://gist.github.com/tmteam/96e702b58e7c650232ecb7fb7c139c07

Любой ioc контейнер по servicekey сделает тоже самое. Если проект будет развиваться di будет 100%. Так зачем вилосипед?

Может кто класс поглядеть, я в нем интероп использовал и диспоз реализовал

А то он хуйню творит

вот простейший вариант решения, который тебе подойдёт

https://dotnetfiddle.net/LUZ0bO

когда будет больше информации о задаче - можно отделить логику парсинга

добавить регистрации или ещё что нить

То файл экплорер вызовер для сейва, то перезапишет, то эксепшн кинет, то в не освободит ресурсы

но в данный момент - это лишнее

https://dotnetfiddle.net/LUZ0bO

Выглядит не очень, особенно перебор: получилось вернул, не получилось на следующем классе попробую...

если сделать регистрации по предикату - то тебе надо будет 2 раза парсить всё, потому что логика парсинга будет разбирать текст письма и во время конструирования типа будет происходить разбор

вот простейший вариант решения, который тебе подойдёт

Спасибо. ))

Выглядит не очень, особенно перебор: получилось вернул, не получилось на следующем классе попробую...

в данном случе - лучший вариант

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

и можно за 1 проход по тексту решить что это заказ и распарсить его

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

ERROR: S client not available

а какая нужна информация о предметной области? у меня есть файлы писем в формате msg, мне надо такие же письма забрать с ящика, распарсить и вернуть клиенту класс с данными.

структура писем не меняется

тема не меняется

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

Плюсую. Сначала делай чтобы работало, а потом красиво.

Никто не посмотрит?)

Никто не посмотрит?)

ну ты хоть ссылку скинь

pastebin.com/03GQjAGb

а какая нужна информация о предметной области? у меня есть файлы писем в формате msg, мне надо такие же письма забрать с ящика, распарсить и вернуть клиенту класс с данными.

как парсить то?

как парсить то?

выдираешь боди и регэксами

вот, пихай всё это в TryCreate и будет теье счастье, потому что когда у тебя будет 7 типов сообщений, парсинг усложнится, если понадобится добавишь регистрации или ещё каких плюшек

С drop down в interop excel работал кто?

pastebin.com/03GQjAGb

я бы удалил финализатор

Как selected установить?

В нем может быть проблема? Иди он просто не нужен?

pastebin.com/03GQjAGb

и можно ещё добавить проверки на disposed

В нем может быть проблема? Иди он просто не нужен?

он просто не нужен, а если финализатор не нужен - его лучше не писать

pastebin.com/03GQjAGb

ты уверен, что это ок? var newWorkSheet = (Worksheet)_workbook.Worksheets.Item[1]; _worksheets.Add(newWorkSheet);

не уверен на самом деле

увидел в 2 местаз

местах*

выглядит очень странно, не работал с excell, но какой то диссонанс

а мне показалось, что норм выглядит. Типа уже есть по дефолту одна таблица в книге и ее берем

а зачем одну и туже таблицу добавлять в список?

это внешний список же, просто для себя

а на ретурне стоит new которая

(Worksheet)_workbook.Worksheets.Item[1];

тут нет new

var newWorkSheet = (Worksheet)_workbook.Worksheets.Item[1]; - берем дефолтную

_worksheets.Add(newWorkSheet); - кидаем в свой, левый массив

и добавляем её в список

return newWorkSheet; - возвращаем ту самую дефолнтую

вторую строку вообще убрать можно