Ребят, всем привет. У меня есть: ConcurrentDictionary<WebSocketClient, WebSocket> _sockets = new ConcurrentDictionary<WebSocketClient,

Question

DotNetRuChat

Jan

Ребят, всем привет. У меня есть: ConcurrentDictionary<WebSocketClient, WebSocket> _sockets = new ConcurrentDictionary<WebSocketClient,

WebSocket>();
Нужно получить список значений WebSocket по условию.
Какой метод правильнее и почему? Или оба фигня?

#backend #dot #programming #russian

0

25.04.2021

11 ответов

17 просмотров

Jan Автор вопроса

0

25.04.2021

Jan Автор вопроса

ioannis
Смотря для чего. Зачем нужен parallel?

Чтобы потом всем WebSocket, которые вернул метод, отправить сообщение. На кодревью сказали, что из ConcurrentDictionary извлекать значения чем-то помимо TryGetValue не стоит, но в моем случае TryGetValue не вариант. Насчет обычного foreach вместо Parallel.ForEach не знаю, прочитал, что лучше использовать второй вариант. Он вроде потокобезопасный, а первый вариант просто копию создает. Не уверен.

0

25.04.2021

ioannis

Jan
Чтобы потом всем WebSocket, которые вернул метод, ...

Ну перечисление по конкурентному словарю порождает новый список, который не меняется. Потому про второй метод даже не знаю, что не понравилось. А первый - ну он очень не очень. Складывать надо тоже через try*, в конкурентную коллекцию, да и непонятно, зачем параллельность в фильтрации, если ее не просят

0

25.04.2021

Jan Автор вопроса

ioannis
Ну перечисление по конкурентному словарю порождает...

Ок, спасибо

0

25.04.2021

ioannis

Jan
Ок, спасибо

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

0

25.04.2021

Dr. Friedrich von Never

Jan

Оба варианта выглядят так, будто там race condition.

0

26.04.2021

Jan Автор вопроса

Dr. Friedrich von Never
Оба варианта выглядят так, будто там race conditio...

Меня, кажется, осенило. Раньше было: ConcurrentDictionary<WebSocketClient, WebSocket> _sockets Сделаю так: ConcurrentDictionary<WebSocketClient, ConcurrentDictionary<ConnectionType, ConcurrentDictionary<string, WebSocket>>> _sockets (где string - уникальный ид соединения) Тогда в метод можно будет передать: WebSocketClient wsClient (состоит из ProjectId и StreamId) и ConnectionType connectionType И вытащить список нужных вебсокетов через _sockets.TryGetValue(wsClient, out var connectionsByType); connectionsByType.TryGetValue(connectionType, out var connections); Всем спасибо за помощь=)

0

26.04.2021

Dr. Friedrich von Never

Jan
Меня, кажется, осенило. Раньше было: ConcurrentDi...