Как удалить отдельный конкретный объект из ConcurrentBag <>?

С новым ConcurrentBag<T>в .NET 4, как удалить из него определенный конкретный объект, когда он доступен только TryTake()и TryPeek()?

Я подумываю использовать, TryTake()а затем просто добавить полученный объект обратно в список, если я не хочу его удалять, но мне кажется, что я что-то упускаю. Это правильный способ?

Короткий ответ: вы не можете сделать это простым способом.

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

Две альтернативы для вас:

• Удалите все предметы и запомните их, пока не найдете тот, который хотите удалить, а затем верните остальные. Обратите внимание: если два потока попытаются сделать это одновременно, у вас возникнут проблемы.
• Используйте более подходящую структуру данных, например ConcurrentDictionary .

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

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

ConcurrentBag отлично подходит для обработки списка, в котором вы можете добавлять элементы и перечислять из многих потоков, а затем в конечном итоге выбросить его, как предполагает его название :)

Как сказал Марк Байерс , вы можете воссоздать новый ConcurrentBag, который не содержит элемент, который вы хотите удалить, но вы должны защитить его от попадания нескольких потоков с помощью блокировки. Это однострочный:

myBag = new ConcurrentBag<Entry>(myBag.Except(new[] { removedEntry }));

Это работает и соответствует духу, для которого был разработан ConcurrentBag.

Марк прав в том, что ConcurrentDictionary будет работать так, как вы хотите. Если вы хотите по-прежнему использовать ConcurrentBag , следуйте инструкциям, которые, на ваш взгляд, неэффективны.

var stringToMatch = "test";
var temp = new List<string>();
var x = new ConcurrentBag<string>();
for (int i = 0; i < 10; i++)
{
    x.Add(string.Format("adding{0}", i));
}
string y;
while (!x.IsEmpty)
{
    x.TryTake(out y);
    if(string.Equals(y, stringToMatch, StringComparison.CurrentCultureIgnoreCase))
    {
         break;
    }
    temp.Add(y);
}
foreach (var item in temp)
{
     x.Add(item);
}

Как вы упомянули, TryTake()это единственный вариант. Это также пример в MSDN . Reflector также не показывает никаких других скрытых внутренних методов, представляющих интерес.

public static void Remove<T>(this ConcurrentBag<T> bag, T item)
{
    while (bag.Count > 0)
    {
        T result;
        bag.TryTake(out result);

        if (result.Equals(item))
        {
            break; 
        }

        bag.Add(result);
    }

}

Это мой класс расширения, который я использую в своих проектах. Он может удалить отдельный элемент из ConcurrentBag, а также может удалить список элементов из корзины

public static class ConcurrentBag
{
    static Object locker = new object();

    public static void Clear<T>(this ConcurrentBag<T> bag)
    {
        bag = new ConcurrentBag<T>();
    }


    public static void Remove<T>(this ConcurrentBag<T> bag, List<T> itemlist)
    {
        try
        {
            lock (locker)
            {
                List<T> removelist = bag.ToList();

                Parallel.ForEach(itemlist, currentitem => {
                    removelist.Remove(currentitem);
                });

                bag = new ConcurrentBag<T>();


                Parallel.ForEach(removelist, currentitem =>
                {
                    bag.Add(currentitem);
                });
            }

        }
        catch (Exception ex)
        {
            Debug.WriteLine(ex.Message);
        }
    }

    public static void Remove<T>(this ConcurrentBag<T> bag, T removeitem)
    {
        try
        {
            lock (locker)
            {
                List<T> removelist = bag.ToList();
                removelist.Remove(removeitem);                

                bag = new ConcurrentBag<T>();

                Parallel.ForEach(removelist, currentitem =>
                {
                    bag.Add(currentitem);
                });
            }

        }
        catch (Exception ex)
        {
            Debug.WriteLine(ex.Message);
        }
    }
}

public static ConcurrentBag<String> RemoveItemFromConcurrentBag(ConcurrentBag<String> Array, String Item)
{
    var Temp=new ConcurrentBag<String>();
    Parallel.ForEach(Array, Line => 
    {
       if (Line != Item) Temp.Add(Line);
    });
    return Temp;
}

как насчет:

bag.Where(x => x == item).Take(1);

Работает, не уверен насколько эффективно ...