Является ли list потокобезопасным?

Question

[Константин Теплоухов]

Если list данные от туда удаляются другими потоками (вообщем введут работу с этим листом)
Сделал таймер в котором вот такой код

if(File.Exists("Lines.txt"))
            {
                File.Delete("Lines.txt");
            }
            File.AppendAllLines("Lines.txt", myList);

Является ли это потокобезопасным решением и не вызовет ли проблем ?

Comments

[twobomb]

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

Answer 1

[Sergey В.]

List не является потокобезопасным.
Для работы с коллекциями из разных потоков, в C# предусмотерны потокобезопасные коллекции, которые различаются подходом к записи/чтению.
Тут ещё можно почитать про это.

Comments

[Константин Теплоухов]

те мне нужно для начала остановить все потоки , сделать запись и запустить их снова ?
ps: не хочется переписывать list,ы просто ради того чтоб в таймере раз в долгое время делать запись ...

foreach (var thr in listThread)
            {
                thr.Suspend();
            }
            if(File.Exists("Lines.txt"))
            {
                File.Delete("Lines.txt");
            }
            File.AppendAllLines("Lines.txt", myList);
            foreach (var thr in listThread)
            {
                thr.Resume();
            }

[shurshur]

Константин Теплоухов, можно использовать блокировки, не знаю что там у вас в C#, в POSIX threads это делается с помощью мьютексов (mutex).

[Sergey В.]

Константин Теплоухов, при использовании System.Collections.Concurrent Вам не нужно заниматься арбитражом.
В текущей ситуации - если архитектура проекта позволяет, можно воспользоваться старыми добрыми локерами.
Но потом всё-равно - надо закопать )

[Василий Банников]

Константин Теплоухов,

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

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

Но лучше пересмотреть архитектуру.
Например, если у тебя есть несколько потоков, которые создают работу для другого потока - может быть лучше использовать System.Threading.Channels, где одни потоки будут писать в канал, а другие читать из него и выполнять работу.