Поток в настоящее время используется предыдущей версией потока

Здравствуйте, уважаемые программисты! Писал клиент и сервер. Идея такова, что выбирается listView элемент, затем нажимается кнопка Button1 и сервер отправляет на клиента команду, клиент принимает ее и в ответ шлет сначала эту же команду, затем число строк из своего listView, а затем в цикле пересылаются все строки. Так вот, при первом нажатии срабатывает нормально, результат можно видеть на картинке, а при втором нажатии вылезает ошибка из названия темы. Помогите, кто может, как избавиться от этой ошибки? В архиве прикрепляю проекты Visual Studio клиента и сервера.

Миниатюры

 

Вложения

Клиент - Сервер.7z (168.6 Кб, 8 просмотров)

0

Wolfdp, Вы хорошо все расписали, но боюсь, что не в моих силах реализовать что-то подобное

0

Сообщение от Оаг но боюсь, что не в моих силах реализовать что-то подобное

С чем сложности? Разбить на потоки? Никапсулировать подключение к конкретному клиенту в отдельный класс? Создать потокобезопансую коллекцию? Передать её по ссылке в другой класс для бизнес логики?

0

Wolfdp, Что такое фабрика, потокобезопасная коллекция, агрегатор, MQ, Handlers?

0

Сообщение от Оаг Что такое фабрика

Класс, который на основе входных параметров создает нужный объект. В нашем случае нам нужно создать ClientObject (я бы таки рекомендовал назвать ClientSession или ещё как, а то object -- ну прям ноль инфы что это такое) на основе TcpClient. Можно просто вызвать конструктор и передать нужные вещи в него из класса, где крутиться сам listener. Но тогда в нём нужно также держать элементы, которые нужны исключительно для создания ClientObject (например список Handlers или туже MQ). Фабрика позволяет "запаковать" логику создания в отдельный класс, упростив вызов в местах где непосредственно нужно создать элемент. Ещё сильнее раскрывается такой подход, когда в проекте есть ioc-контейнер, отвечающий за DI, но подозреваю что сейчас вы это изучать не согласитесь.

Конкретно фабрику можно и не делать, не самый критичный элемент.

Сообщение от Оаг потокобезопасная коллекция

Потокобезопасные коллекции. Читаем, разбираем и после применяем. Вот это прям супер обязательно, иначе в противном случае ПОСТОЯННО БУДЕТЕ ЛОВИТЬ БАГИ.

Также всегда можно обернуть нужную логику аля List (Add, Remove, ForEach) в свой кастомный класс, который внутри будет всё выполнять через тот же lock, аще лучше ReadWriteLock.

Сообщение от Оаг агрегатор

Если на пальцах:
- разбиваем весь скоп задач на логические группы (например сохранение данных в БД/файл; обработка данных; мониторинг; формирование запросов внешнему API).
- всё это инкапсулируется по отдельным классам. Т.е. за сохранение в БД должен отвечать только один класс, и он должен уметь только сохранять в БД (можно дополнительно раздробить по таблицам).
- есть "супер класс", который будет инициализировать все эти объекты, стартовать их работу и обеспечивать взаимодействие друг-с-другом через подписки на event или передачу в конструктор/свойства

В идеале такого нужно избегать, но для этого нужен ioc-контейнер.

Сообщение от Оаг MQ

Message Queue. Это очень мощный подход, когда у вас есть набор входящий сообщений из разных источников при этом есть очень много разных подписчиков на эти сообщения. TCP/UDP соединение по своей сути -- канал присылающий сообщения. А значит нам можно ввести подписчиков на эти сообщения, которые как-то будут на это реагировать. Как пример, вам из соединения от клиента может приходить два типа сообщения:
- я сменил статус на "красный/синий"
- запрос "какой статус у остальных?"

Допустим изначально нам нужно просто запоминать какой у клиента статус и отправлять по запросу общий статус. Значит у нас два подписчика:
- тот который возьмёт текущего клиента и поменяет ему статус
- тот который посмотрит общий список и ответит клиенту

Через время решили логировать. Тупо добавляем подписчика на первое сообщение и пишем в БД.
Через время решили что если все перешли в статус "красный" -- отправляем уведомления по email. Плюс один подписчик, который никак не задевает уже существующие.

Можно обойтись и без него.

Сообщение от Оаг Handlers

Набор обработчиков входящих сообщений. Можно применить вместе с MQ, можно и отдельно. Как писал выше -- вся суть что у вас есть входящее сообщение, которое вы скармливаете такому обработчику, и он уже знает что с ним делать. Основная идея -- отделить считывание пришедшего сообщения от его обработки. В противном случае добавление/изменение логики для каждого отдельного сообщения будет проходить больно.

0

Wolfdp, Прочитал про потокобезопасные коллекции. И какую из них использовать? ConcurrentBag<T>? И в нее должны закидываться clientobject (session)?

Добавлено через 19 минут
И все-таки, мне интересно, как например на предприятии разворачивать подобную систему. Есть например 250 клиентских ПК и 1 сервер. Это надо заставить сисадмина оббежать все 250 ПК и там вбить в настройки IP-адрес сервера везде?

0

Сообщение от Оаг Это надо заставить сисадмина оббежать все 250 ПК и там вбить в настройки IP-адрес сервера везде?

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

0

Пытаюсь вызвать tcpListener?.Dispose(); и визуал студио пишет ошибку: tcpListener не содержит определения Dispose, и не удалось найти доступный метод расширения Dispose, принимающий тип tcpListener в качестве первого аргумента.

0

Сообщение от Оаг И какую из них использовать?

ConcurrentDictionary

Сообщение от Оаг Это надо заставить сисадмина оббежать все 250 ПК и там вбить в настройки IP-адрес сервера везде?

IP прописывается в настройках. Настройки нужно делать так, чтобы клиент мог их подправить самостоятельно. Более кошерный подход указывать не IP, а доменное имя (услуга чаще платная). Дабы упростить себе жизнь, в проектах C# существует понятие "конфигурация" и по умолчанию создаются две: DEBUG и RELEASE. Для них можно прописать две независимые настройки, которые будут применяться автоматически. Как результат, пока вы разрабатываете, то ссылаетесь скажем на 127.0.0.1, а когда делаете финальный билд для рассылки бухам -- просто переключаетесь на RELEASE и отдаете соответствующий файл.

Сообщение от Оаг tcpListener?.Dispose()

Если у вас версия Framework (всё что до версии 4.8 без приписки Core) -- тот там листенер не является Disposeble объектом. И перед тем как спрашивать такое вообще у кого-либо, рекомендую:
- осознанно прочитать что написано.
- глянуть документацию по той версии, которую используете.
- погуглить или использовать AI-чат.

Желательно именно в такой последовательности.

0

Сообщение от Wolfdp когда делаете финальный билд для рассылки бухам -- просто переключаетесь на RELEASE и отдаете соответствующий файл.

Как я это вижу. Покупает у моей компании программный продукт другая компания. Они говорят, на какой ip надо сконфигурировать клиенты. Разработчик конфигурирует продукт и отдает бухам, которые отсылают его заказчику. Хотя с точки зрения информационной безопасности это звучит не очень.
Либо второй вариант, что сначала компания покупает продукт, потом у себя конфигурируют на нужный ip-шник и потом клонирует приложение и ставит на свои компьютеры.
По поводу онлайн-игры тоже неизвестно, как оно там работает.

Сообщение от Wolfdp Если у вас версия Framework (всё что до версии 4.8 без приписки Core)

У меня Framework 4.8, без приставки core. Согласен, надо поменьше глупых вопросов задавать

0

Сообщение от Оаг Либо второй вариант, что сначала компания покупает продукт, потом у себя конфигурируют на нужный ip-шник и потом клонирует приложение и ставит на свои компьютеры.

Стратегия развертывания может подразумевать необходимость сисадмина или девопса. Нередко нужны вообще непосредственно разработчики. Есть те или иные механизмы упрощения, но волшебной кнопки "запустить и забить" не существует. Серверная часть в 99.999% случаев требует настроек в том или ином виде. Клиентская -- либо подготовки инсталера, либо пошагового мануала по установке, либо специального конфигурационного файла (всякие VPN-клиенты работают по последнему принципу).

Самый простой способ -- написать подробную инструкцию на всё и вся.
Чуть кошерней -- выделить техподдержку.

0

Сообщение от Оаг заставить сисадмина оббежать все 250 ПК

хороший сисадмин никуда не бегает (и хрен заставишь!). У сисадмина есть куча инструментов для удаленного управления и настройки ПК на предприятии.

0

0

Wolfdp, в целом то согласен.
Просто, относительно автора вопроса существуют трудности в виде весьма малых знаний в программировании как таковом, не говоря уже о сетевом уровне, меж-поточном взаимодействии. Ну и конечно же, такие вещи как MSI и вообще установщики - темный лес.
Оаг, без обид - честная критика: при первом взгляде только на форму (макет WinForms) с неподписанными кнопками и очень странными размерами/компоновкой элементов, становится ясно, что за среду программирования посадили школьника в первый раз. В этом случае сложновато что-то дельное советовать, вернее, советовать то можно, но эти советы принесут 0.001% пользы. Без хорошей такой базы тут не обойтись, а обучать этому на форуме ессесно вряд ли кто возьмется.

0

wizard41, Это правда, я все понимаю. Касаемо размеров, внешнего вида формы и неподписанных кнопок, я руководствуюсь принципом сначала написать какую-то логику, а потом уже в конце делать как надо, переименовывать controlы и т.д. Да и я скинул очень сырую версию, где только начинал проверять, как все работает. Но вообще с точки зрения реальной коммерческой разработки, я так понимаю, там сразу все делается красиво, но такого опыта у меня нет.

0

Оаг, вообще да, принято сперва разработать "модель", т.е. сам фактический алгоритм, а затем на него навешивать "фасад" в виде окна. Такой подход удобен тем, что его можно протестировать любым типом приложения, той же консолью, не имея при этом макета окна как такового.

Но, понятно дело, если опыта нет, то бывает трудно себе представить результат тех или иных действий - на них хочется посмотреть. Поэтому их пытаются сразу куда-то вывести, в то же окно.

А здесь возникает вторая проблема: нет разделения логики модели от UI-потока. И результаты становятся непредсказуемыми, ну и в целом программа начинает работать не так как ожидалось.

Сообщение от Оаг Но вообще с точки зрения реальной коммерческой разработки, я так понимаю, там сразу все делается красиво, но такого опыта у меня нет.

Лучше сразу привыкать делать правильно. Пусть будет дольше по времени, зато эффективнее в плане дальнейшего обслуживания.
Hello World написал, вывел в окно? 5 + 5 посчитал, результат вывел в TextBox? Все, теперь забывай что делал и начинай делать нормально.

0

Как подключаться до победного, если сервер недоступен? После первого же раза программа останавливается с логичной ошибкой "Подключение не установлено, т.к. конечный компьютер отверг запрос на подключение". Нигде не могу найти пример вызова Connect в цикле или пример обработки этой ошибки.

0

Оаг, в моем примере пробовал клиента подключать к серверу, когда тот не работает? Снизу просто появится сообщение типа "такого сервера нет..." или что-то в этом роде, не помню. И никаких ошибок нет.
Так вот вместо этого сообщения можно делать задержку и повторять подключение снова... Клиент будет пробовать подключаться к серваку пока материнская плата не сгорит. Или БП.

1

Сообщение от Оаг Как подключаться до победного, если сервер недоступен?

Вводишь флаг "переподключаться автоматически". При любом дисконекте кидаешь свой event Disconnected, который в параметрах будеть иметь необязательное поле Error. Подписываешься на это событие и чекаешь Error. Если есть (в идеале ещё и тип ошибки анализировать, ибо если там "некоректный IP адресс, то долбиться нет смысла") и стоит флаг "автоподключение" -- пинаем таймаут + Connect.

Следует учесть моменты, что юзер вручную может отключать (можно в тех же параметрах Disconnected добавить флаг) и тогда ни в коем случае не должно попадать на обработку переподключения.

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

Добавлено через 1 минуту

1

Wolfdp, Понял.
Ну, когда контролов немного, то не запутаюсь. Да и всегда можно двойным кликом тыкать по контролу в конструкторе формы и он перебросит тебя в нужный метод.

0

wizard41, Здравствуйте, вы говорили, что мой метод получения директории exe-файла программы никуда не годится. А как бы вы его реализовали?

0