1

Есть несколько потоков, которые добавляют значения в:

public static List<TimeSpan> SpisokVremeni { get; set; }

А есть поток, который хочет перебрать этот список SpisokVremeni и скопировать значения в свой локальный список.

Но вот при переборе этого SpisokVremeni в foreach, возникают проблемы, другие потоки добавляют значения и естественно вылетает исключение.

Как такой момент обойти ? Как получить копию без исключений ?

Пробовал так, но на потоки которые добавляют значения в SpisokVremeni - это никак не повлияло:

    List<TimeSpan> kopiyaSpiskaVremeni = new();

    lock (locker)
    {
        foreach (TimeSpan vremya in MyClass.SpisokVremeni)
        {
            kopiyaSpiskaVremeni.Add(vremya);
        }
    }

    return kopiyaSpiskaVremeni;
12
  • @aepot: Наверное, не совсем дубликат? Здесь автор не просит потокобезопасную коллекцию, а спрашивает, почему проблемы возникают несмотря на lock.
    – VladD
    Commented 2 июн. 2021 в 13:10
  • @VladD потому что он обращается к коллекции в сторонних местах не используя этот же лок. Решение - потокобезопасная коллекция, которая обеспечивает целостность данных при работе из нескольких потоков одновременно. Использование одного лока в 100500 местах кода - это хоть и решение, но костыль.
    – aepot
    Commented 2 июн. 2021 в 13:13
  • @VladD к тому же использование мониторов lock для операций чтения/записи в коллекцию - мягко говоря, не очень хорошее решение, так как при работе с коллекцией лучше реализовать R/W Lock или использовать элементы синхронизации дотнета, уже реализуюзие это.
    – aepot
    Commented 2 июн. 2021 в 13:17
  • @aepot: С потокобезопасными коллекциями есть много проблем. Например, её сложно реализовать «правильно», т. к. разный код имеет разное представление о том, что именно нужно защищать и как именно будет использоваться код (docs.microsoft.com/en-us/archive/blogs/ericlippert/…). Во-вторых, такое решение не является composable (как бы это сказать по-русски?): если вам нужно защитить комбинацию из двух коллекций, что будем делать?
    – VladD
    Commented 2 июн. 2021 в 13:18
  • @VladD ну транзации - это уже другая, отдельная история. Как я понял, в вопросе речь не про транзакции. В любом случае лочить между собой операции чтения, когда нет операций записи - странное занятие. Если настаиваете - я переоткрою, мне не принципиально.
    – aepot
    Commented 2 июн. 2021 в 13:22

1 ответ 1

4

Синхронизация — это кооперативная задача. Все, кто работают с разделяемыми данными, должны придерживаться протокола общения.

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


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


Кстати, ваш код можно упростить до такого:

lock (locker)
    return MyClass.SpisokVremeni.ToList();

Но, как вы уже сами выяснили, вам нужно использовать блокировку (тем же объектом locker!) и там, где происходит запись.

6
  • 1
    а второй и третий абзац не противоречат друг другу? В одном говорится, что блокировать надо и чтение и запись, в другом, что блокировку на время записи только
    – Grundy
    Commented 2 июн. 2021 в 13:05
  • @Grundy: третий абзац говорит, что блокировку не нужно распространять на время получения данных для записи из какого-то другого источника. Уточню это.
    – VladD
    Commented 2 июн. 2021 в 13:07
  • Да, так лучше звучит, в первом варианте казалось что речь про один список
    – Grundy
    Commented 2 июн. 2021 в 13:09
  • 1
    @VladD правильно ли я понял ? Я должен одним и тем же locker заблокировать участок копирования и участок записи. - Верно ? Запись я никак не блокировал xD Commented 2 июн. 2021 в 13:25
  • 1
    @Алексей верно.
    – aepot
    Commented 2 июн. 2021 в 13:27

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.