Есть множество

[Flags]
public enum MySet{None=0,Val1=1,Val1=2,Val3=4}

И нужны функции которые будут работать в многопоточном коде. Вот они
private MySet mySet;
public bool IsSet(MySet s)
{
 return mySet & s == s;
}
public void OnSet(MySet s)
{
 mySet += s;
}

public void OfSet(MySet s)
{
 mySet -= s;
}

Как такое сделать в многопоточной среде? Или можно оставить так и проблем не будет?

Здравствуйте, Аноним, Вы писали:

А>Есть множество
…
А>И нужны функции которые будут работать в многопоточном коде. Вот они
…
А>Как такое сделать в многопоточной среде? Или можно оставить так и проблем не будет?

Нужно использовать тег [c#], битовые опрерации сложения и умножения и, например, lock.

Здравствуйте, _FRED_, Вы писали:

_FR>Здравствуйте, Аноним, Вы писали:

А>>Есть множество
_FR>…
А>>И нужны функции которые будут работать в многопоточном коде. Вот они
_FR>…
А>>Как такое сделать в многопоточной среде? Или можно оставить так и проблем не будет?

_FR>Нужно использовать тег [c#], битовые опрерации сложения и умножения и, например, lock.

Извините за наглость, а код не покажете? Только начинаю писать и не хочется на грабли наступать в такой мелочи

Здравствуйте, Аноним, Вы писали:

А>>>Есть множество
_FR>>…
А>>>И нужны функции которые будут работать в многопоточном коде. Вот они
_FR>>…
А>>>Как такое сделать в многопоточной среде? Или можно оставить так и проблем не будет?
_FR>>Нужно использовать тег [c#], битовые опрерации сложения и умножения и, например, lock.

А>Извините за наглость, а код не покажете? Только начинаю писать и не хочется на грабли наступать в такой мелочи

Типы перечислений (руководство по программированию в C#)
lock Statement (C# Reference)

А>>Извините за наглость, а код не покажете? Только начинаю писать и не хочется на грабли наступать в такой мелочи

_FR>Типы перечислений (руководство по программированию в C#)
_FR>lock Statement (C# Reference)

Спасибо. А подскажите. Есть я делаю Read-Only свойство то в методе get мне надо использовать lock?

Здравствуйте, Аноним, Вы писали:

А>>>Извините за наглость, а код не покажете? Только начинаю писать и не хочется на грабли наступать в такой мелочи

_FR>>Типы перечислений (руководство по программированию в C#)
_FR>>lock Statement (C# Reference)

А>Спасибо. А подскажите. Есть я делаю Read-Only свойство то в методе get мне надо использовать lock?

Это зависит от того, как реализовано get. Что бы лишний раз не думать, можно и залочить.

_FR>Это зависит от того, как реализовано get. Что бы лишний раз не думать, можно и залочить.
Понял. А если я просто читаю поле (не свойство) то его не нужно лочить?

Здравствуйте, Аноним, Вы писали:

_FR>>Это зависит от того, как реализовано get. Что бы лишний раз не думать, можно и залочить.
А>Понял. А если я просто читаю поле (не свойство) то его не нужно лочить?

В случае с вашим кодом (использование enum) можно использовать Interlocked операции для записи изменений и обычное чтение без дополнительных блокировок.

Здравствуйте, Аноним, Вы писали:

А>Понял. А если я просто читаю поле (не свойство) то его не нужно лочить?

Если его размер равен 4-м байт(как в Вашем примере), то не нужно. Даже если кто-то будет его писать в этот момент, ничего страшного не произойдёт. Стоит только пометить переменную как volatile.

Здравствуйте, Аноним, Вы писали:

А>Как такое сделать в многопоточной среде? Или можно оставить так и проблем не будет?

Метод IsSet — потокобезопасен, т.к. там присутствует только одно чтение, а оно безопасно.

Методы OnSet, OfSet — непотокобезопасны, т.к. там присутствует как чтение, так и запись и между ними может вклиниться другой поток. Поэтому, тело метода луче обрамить в lock(_lock), а _lock нужно объявить членом класса (private readonly object _lock = new object().
Сами методы OnSet, OfSet реализованы неправильно, для манипуляции flag-ами нужно использовать битовые операции.

Здравствуйте, TK, Вы писали:

_FR>>>Это зависит от того, как реализовано get. Что бы лишний раз не думать, можно и залочить.
А>>Понял. А если я просто читаю поле (не свойство) то его не нужно лочить?
TK>В случае с вашим кодом (использование enum) можно использовать Interlocked операции для записи изменений и обычное чтение без дополнительных блокировок.

А разве можно на Interlocked сделать добавление/удаление флага?

Здравствуйте, TK, Вы писали:

А>>Понял. А если я просто читаю поле (не свойство) то его не нужно лочить?

TK>В случае с вашим кодом (использование enum) можно использовать Interlocked операции для записи изменений и обычное чтение без дополнительных блокировок.

А можно пример использоания interlocked в применении к данному случаю?

Здравствуйте, Lloyd, Вы писали:

А>>Как такое сделать в многопоточной среде? Или можно оставить так и проблем не будет?
L>Метод IsSet — потокобезопасен, т.к. там присутствует только одно чтение, а оно безопасно.

А если у енама underlying-тип — восьмибайтный, тоже будет "безопасно"?

Здравствуйте, _FRED_, Вы писали:

L>>Метод IsSet — потокобезопасен, т.к. там присутствует только одно чтение, а оно безопасно.

_FR>А если у енама underlying-тип — восьмибайтный, тоже будет "безопасно"?

А где ты нашел восьмибайтный enum?

Здравствуйте, _FRED_, Вы писали:

_FR>А если у енама underlying-тип — восьмибайтный, тоже будет "безопасно"?
+1

И даже если 4-хбайтный, но не выровнен на границу (нарпимер при Pack == 1), то тоже может оказаться непотокобезопасным.

Здравствуйте, Lloyd, Вы писали:

L>А где ты нашел восьмибайтный enum?

enum MyEnum: long

Здравствуйте, _FRED_, Вы писали:

_FR>А разве можно на Interlocked сделать добавление/удаление флага?

Можно с применением Interlocked.CompareExchange

Здравствуйте, Jolly Roger, Вы писали:

L>>А где ты нашел восьмибайтный enum?

JR>

JR>enum MyEnum: long
JR>

JR>

И где ты это нашел?

Здравствуйте, Lloyd, Вы писали:

L>И где ты это нашел?

В смысле? Если Вы имеете в виду приведённый ТС пример, то ведь очевидно, что это просто демонстрация вопроса. Просто утверждение

Метод IsSet — потокобезопасен, т.к. там присутствует только одно чтение, а оно безопасно.

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

Здравствуйте, Jolly Roger, Вы писали:

JR>Просто утверждение

JR>

JR>Метод IsSet — потокобезопасен, т.к. там присутствует только одно чтение, а оно безопасно.

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

Я даже более скажу, любое утвердение без контекста теряет смысл. А контекст определен топиком.