Обновить все объекты в коллекции с помощью LINQ

375

Есть ли способ использовать LINQ?

foreach (var c in collection)
{
    c.PropertyToSet = value;
}

Чтобы уточнить, я хочу перебирать каждый объект в коллекции и затем обновлять свойство для каждого объекта.

Мой вариант использования: у меня есть куча комментариев в сообщении в блоге, и я хочу перебирать каждый комментарий в сообщении в блоге и устанавливать дату-время в блоге, чтобы оно составляло +10 часов. Я мог бы сделать это в SQL, но я хочу сохранить его на бизнес-уровне.

  • 10
    Интересный вопрос. Лично я предпочитаю, как у вас это выше - гораздо яснее, что происходит!
  • 2
    Я пришел сюда в поисках ответа на тот же вопрос и решил, что будущим разработчикам будет так же просто, меньше кода и легче понять, как сделать это так, как вы это делали в своем OP.
Показать ещё 2 комментария
Теги:
linq
foreach

15 ответов

652
Лучший ответ

Хотя вы можете использовать метод расширения ForEach, если вы хотите использовать только фреймворк, который вы можете сделать

collection.Select(c => {c.PropertyToSet = value; return c;}).ToList();

ToList необходим для того, чтобы немедленно оценить выбор из-за ленивой оценки.

  • 6
    Я проголосовал за это, потому что это довольно хорошее решение ... единственная причина, по которой мне нравится метод расширения, заключается в том, что он немного яснее понять, что именно происходит ... однако ваше решение все еще довольно мило
  • 0
    Я согласен, мой ответ в основном для тех, кто не хочет использовать метод расширения.
Показать ещё 22 комментария
264
collection.ToList().ForEach(c => c.PropertyToSet = value);
  • 3
    Что я должен сделать, чтобы обновить более одного свойства?
  • 27
    @SanthoshKumar: используйте collection.ToList().ForEach(c => { c.Property1ToSet = value1; c.Property2ToSet = value2; });
Показать ещё 9 комментариев
54

Я делаю это

Collection.All(c => { c.needsChange = value; return true; });
  • 0
    Я думаю, что это самый чистый способ сделать это.
  • 22
    Этот подход, безусловно, работает, но он нарушает намерение метода расширения All() , что может привести к путанице, когда кто-то еще читает код.
Показать ещё 2 комментария
18

Я фактически нашел метод расширения, который будет делать то, что я хочу красиво

public static IEnumerable<T> ForEach<T>(
    this IEnumerable<T> source,
    Action<T> act)
{
    foreach (T element in source) act(element);
    return source;
}
  • 4
    приятно :) Lomaxx, возможно, добавьте пример, чтобы люди могли видеть его в «действии» (бум!).
  • 1
    Это единственный полезный подход, если вы действительно хотите избежать цикла foreach (по любой причине).
Показать ещё 2 комментария
8

Использование:

ListOfStuff.Where(w => w.Thing == value).ToList().ForEach(f => f.OtherThing = vauleForNewOtherThing);

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

5

Для этого не существует встроенного метода расширения. Хотя определение одного достаточно прямолинейно. В нижней части сообщения указан метод Iterate. Его можно использовать так

collection.Iterate(c => { c.PropertyToSet = value;} );

Итерационный источник

public static void Iterate<T>(this IEnumerable<T> enumerable, Action<T> callback)
{
    if (enumerable == null)
    {
        throw new ArgumentNullException("enumerable");
    }

    IterateHelper(enumerable, (x, i) => callback(x));
}

public static void Iterate<T>(this IEnumerable<T> enumerable, Action<T,int> callback)
{
    if (enumerable == null)
    {
        throw new ArgumentNullException("enumerable");
    }

    IterateHelper(enumerable, callback);
}

private static void IterateHelper<T>(this IEnumerable<T> enumerable, Action<T,int> callback)
{
    int count = 0;
    foreach (var cur in enumerable)
    {
        callback(cur, count);
        count++;
    }
}
  • 0
    Является ли Iterate необходимым, что не так с Count, Sum, Avg или другим существующим методом расширения, который возвращает скалярное значение?
  • 1
    это довольно близко к тому, что я хочу, но немного .. вовлечено. Пост блога, который я разместил, имеет похожую реализацию, но с меньшим количеством строк кода.
Показать ещё 4 комментария
4

Я пробовал несколько вариантов этого, и я продолжаю возвращаться к решению этого парня.

http://www.hookedonlinq.com/UpdateOperator.ashx

Опять же, это другое решение. Но я скомпилировал код в маленькую библиотеку и использую ее довольно регулярно.

Я собираюсь вставить его код здесь, так как его сайт (блог) перестает существовать в какой-то момент в будущем. (Там нет ничего хуже, чем просмотр сообщения, в котором говорится: "Вот точный ответ, который вам нужен", "Щелчок" и "Мертвый URL".)

    public static class UpdateExtensions {

    public delegate void Func<TArg0>(TArg0 element);

    /// <summary>
    /// Executes an Update statement block on all elements in an IEnumerable<T> sequence.
    /// </summary>
    /// <typeparam name="TSource">The source element type.</typeparam>
    /// <param name="source">The source sequence.</param>
    /// <param name="update">The update statement to execute for each element.</param>
    /// <returns>The numer of records affected.</returns>
    public static int Update<TSource>(this IEnumerable<TSource> source, Func<TSource> update)
    {
        if (source == null) throw new ArgumentNullException("source");
        if (update == null) throw new ArgumentNullException("update");
        if (typeof(TSource).IsValueType)
            throw new NotSupportedException("value type elements are not supported by update.");

        int count = 0;
        foreach (TSource element in source)
        {
            update(element);
            count++;
        }
        return count;
    }
}



int count = drawingObjects
        .Where(d => d.IsSelected && d.Color == Colors.Blue)
        .Update(e => { e.Color = Color.Red; e.Selected = false; } );
  • 1
    Вы можете использовать Action<TSource> вместо создания дополнительного делегата. Это, возможно, не было доступно на момент написания этого, хотя.
  • 0
    Да, на тот момент это была старая школа DotNet. Хороший комментарий Фрэнк.
Показать ещё 1 комментарий
2

Я написал несколько методов расширения, чтобы помочь мне в этом.

namespace System.Linq
{
    /// <summary>
    /// Class to hold extension methods to Linq.
    /// </summary>
    public static class LinqExtensions
    {
        /// <summary>
        /// Changes all elements of IEnumerable by the change function
        /// </summary>
        /// <param name="enumerable">The enumerable where you want to change stuff</param>
        /// <param name="change">The way you want to change the stuff</param>
        /// <returns>An IEnumerable with all changes applied</returns>
        public static IEnumerable<T> Change<T>(this IEnumerable<T> enumerable, Func<T, T> change  )
        {
            ArgumentCheck.IsNullorWhiteSpace(enumerable, "enumerable");
            ArgumentCheck.IsNullorWhiteSpace(change, "change");

            foreach (var item in enumerable)
            {
                yield return change(item);
            }
        }

        /// <summary>
        /// Changes all elements of IEnumerable by the change function, that fullfill the where function
        /// </summary>
        /// <param name="enumerable">The enumerable where you want to change stuff</param>
        /// <param name="change">The way you want to change the stuff</param>
        /// <param name="where">The function to check where changes should be made</param>
        /// <returns>
        /// An IEnumerable with all changes applied
        /// </returns>
        public static IEnumerable<T> ChangeWhere<T>(this IEnumerable<T> enumerable, 
                                                    Func<T, T> change,
                                                    Func<T, bool> @where)
        {
            ArgumentCheck.IsNullorWhiteSpace(enumerable, "enumerable");
            ArgumentCheck.IsNullorWhiteSpace(change, "change");
            ArgumentCheck.IsNullorWhiteSpace(@where, "where");

            foreach (var item in enumerable)
            {
                if (@where(item))
                {
                    yield return change(item);
                }
                else
                {
                    yield return item;
                }
            }
        }

        /// <summary>
        /// Changes all elements of IEnumerable by the change function that do not fullfill the except function
        /// </summary>
        /// <param name="enumerable">The enumerable where you want to change stuff</param>
        /// <param name="change">The way you want to change the stuff</param>
        /// <param name="where">The function to check where changes should not be made</param>
        /// <returns>
        /// An IEnumerable with all changes applied
        /// </returns>
        public static IEnumerable<T> ChangeExcept<T>(this IEnumerable<T> enumerable,
                                                     Func<T, T> change,
                                                     Func<T, bool> @where)
        {
            ArgumentCheck.IsNullorWhiteSpace(enumerable, "enumerable");
            ArgumentCheck.IsNullorWhiteSpace(change, "change");
            ArgumentCheck.IsNullorWhiteSpace(@where, "where");

            foreach (var item in enumerable)
            {
                if (!@where(item))
                {
                    yield return change(item);
                }
                else
                {
                    yield return item;
                }
            }
        }

        /// <summary>
        /// Update all elements of IEnumerable by the update function (only works with reference types)
        /// </summary>
        /// <param name="enumerable">The enumerable where you want to change stuff</param>
        /// <param name="update">The way you want to change the stuff</param>
        /// <returns>
        /// The same enumerable you passed in
        /// </returns>
        public static IEnumerable<T> Update<T>(this IEnumerable<T> enumerable,
                                               Action<T> update) where T : class
        {
            ArgumentCheck.IsNullorWhiteSpace(enumerable, "enumerable");
            ArgumentCheck.IsNullorWhiteSpace(update, "update");
            foreach (var item in enumerable)
            {
                update(item);
            }
            return enumerable;
        }

        /// <summary>
        /// Update all elements of IEnumerable by the update function (only works with reference types)
        /// where the where function returns true
        /// </summary>
        /// <param name="enumerable">The enumerable where you want to change stuff</param>
        /// <param name="update">The way you want to change the stuff</param>
        /// <param name="where">The function to check where updates should be made</param>
        /// <returns>
        /// The same enumerable you passed in
        /// </returns>
        public static IEnumerable<T> UpdateWhere<T>(this IEnumerable<T> enumerable,
                                               Action<T> update, Func<T, bool> where) where T : class
        {
            ArgumentCheck.IsNullorWhiteSpace(enumerable, "enumerable");
            ArgumentCheck.IsNullorWhiteSpace(update, "update");
            foreach (var item in enumerable)
            {
                if (where(item))
                {
                    update(item);
                }
            }
            return enumerable;
        }

        /// <summary>
        /// Update all elements of IEnumerable by the update function (only works with reference types)
        /// Except the elements from the where function
        /// </summary>
        /// <param name="enumerable">The enumerable where you want to change stuff</param>
        /// <param name="update">The way you want to change the stuff</param>
        /// <param name="where">The function to check where changes should not be made</param>
        /// <returns>
        /// The same enumerable you passed in
        /// </returns>
        public static IEnumerable<T> UpdateExcept<T>(this IEnumerable<T> enumerable,
                                               Action<T> update, Func<T, bool> where) where T : class
        {
            ArgumentCheck.IsNullorWhiteSpace(enumerable, "enumerable");
            ArgumentCheck.IsNullorWhiteSpace(update, "update");

            foreach (var item in enumerable)
            {
                if (!where(item))
                {
                    update(item);
                }
            }
            return enumerable;
        }
    }
}

Я использую его следующим образом:

        List<int> exampleList = new List<int>()
            {
                1, 2 , 3
            };

        //2 , 3 , 4
        var updated1 = exampleList.Change(x => x + 1);

        //10, 2, 3
        var updated2 = exampleList
            .ChangeWhere(   changeItem => changeItem * 10,          // change you want to make
                            conditionItem => conditionItem < 2);    // where you want to make the change

        //1, 0, 0
        var updated3 = exampleList
            .ChangeExcept(changeItem => 0,                          //Change elements to 0
                          conditionItem => conditionItem == 1);     //everywhere but where element is 1

Для справки: проверка аргумента:

/// <summary>
/// Class for doing argument checks
/// </summary>
public static class ArgumentCheck
{


    /// <summary>
    /// Checks if a value is string or any other object if it is string
    /// it checks for nullorwhitespace otherwhise it checks for null only
    /// </summary>
    /// <typeparam name="T">Type of the item you want to check</typeparam>
    /// <param name="item">The item you want to check</param>
    /// <param name="nameOfTheArgument">Name of the argument</param>
    public static void IsNullorWhiteSpace<T>(T item, string nameOfTheArgument = "")
    {

        Type type = typeof(T);
        if (type == typeof(string) ||
            type == typeof(String))
        {
            if (string.IsNullOrWhiteSpace(item as string))
            {
                throw new ArgumentException(nameOfTheArgument + " is null or Whitespace");
            }
        }
        else
        {
            if (item == null)
            {
                throw new ArgumentException(nameOfTheArgument + " is null");
            }
        }

    }
}
2

Мои 2 пенни: -

 collection.Count(v => (v.PropertyToUpdate = newValue) == null);
  • 7
    Мне нравится мышление, но не совсем понятно, что делает код
  • 13
    Да, я не уверен, что хотел бы этого в рабочем коде.
2

Нет, LINQ не поддерживает способ массового обновления. Единственный более короткий путь - использовать метод расширения ForEach - Почему нет метода расширения ForEach для IEnumerable?

1

Хотя вы специально попросили linq-решение, и этот вопрос довольно старый, я отправляю не linq-решение. Это связано с тем, что linq (= интегрированный запрос lanuguage) используется для запросов к коллекциям. Все linq-методы не изменяют базовую коллекцию, они просто возвращают новый (или более точный итератор в новую коллекцию). Таким образом, все, что вы делаете, например, с помощью Select не влияет на базовую коллекцию, вы просто получаете новую.

Конечно, вы можете сделать это с помощью ForEach (кстати, это не linq, а расширение в List<T>). Но это буквально использует foreach любом случае, но с лямбда-выражением. Кроме того, каждый linq-метод внутренне выполняет итерацию вашей коллекции, например, используя foreach или for, однако он просто скрывает ее от клиента. Я не считаю это более читабельным и не ремонтируемым (подумайте о редактировании своего кода при отладке метода, содержащего лямбда-выражения).

Сказав это, не используйте Linq для изменения элементов в вашей коллекции. Лучший способ - это решение, которое вы уже задали в своем вопросе. С помощью классического цикла вы можете легко перебирать свою коллекцию и обновлять ее элементы. На самом деле все эти решения, основанные на List.ForEach, не отличаются друг от друга, но гораздо труднее читать с моей точки зрения.

Поэтому вы не должны использовать linq в тех случаях, когда вы хотите обновить элементы своей коллекции.

  • 2
    Не по теме: я согласен, и оооочень много случаев злоупотребления LINQ, примеры людей, запрашивающих «высокопроизводительные цепочки LINQ», чтобы сделать то, что может быть достигнуто с помощью одного цикла, и т. Д. Я благодарен, что НЕ используется LINQ слишком укоренился в меня, и, как правило, не использовать его. Я вижу людей, использующих цепочки LINQ для выполнения одного действия, не понимая, что почти каждый раз, когда используется команда LINQ, вы создаете еще один цикл for "под капотом". Я чувствую, что это синтетический сахар - создавать менее многословные способы выполнения простых задач, а не заменять стандартное кодирование.
1

Вот метод расширения, который я использую...

    /// <summary>
    /// Executes an Update statement block on all elements in an  IEnumerable of T
    /// sequence.
    /// </summary>
    /// <typeparam name="TSource">The source element type.</typeparam>
    /// <param name="source">The source sequence.</param>
    /// <param name="action">The action method to execute for each element.</param>
    /// <returns>The number of records affected.</returns>
    public static int Update<TSource>(this IEnumerable<TSource> source, Func<TSource> action)
    {
        if (source == null) throw new ArgumentNullException("source");
        if (action == null) throw new ArgumentNullException("action");
        if (typeof (TSource).IsValueType)
            throw new NotSupportedException("value type elements are not supported by update.");

        var count = 0;
        foreach (var element in source)
        {
            action(element);
            count++;
        }
        return count;
    }
  • 0
    Почему "элементы типа значения не поддерживаются обновлением" ?? Ничто не мешает этому!
  • 0
    Это было специфично для проекта, над которым я работал. Я полагаю, это не имеет значения в большинстве случаев. В последнее время я переработал это и переименовал в Run (...), удалил тип значения и изменил его, чтобы он возвращал void, и удалил код счета.
Показать ещё 3 комментария
1

Вы можете использовать Magiq, структуру пакетных операций для LINQ.

0

Вы можете использовать LINQ для преобразования вашей коллекции в массив и затем вызвать Array.ForEach():

Array.ForEach(MyCollection.ToArray(), item=>item.DoSomeStuff());

Очевидно, что это не будет работать с коллекциями структур или встроенных типов, таких как целые числа или строки.

0

Я предполагаю, что вы хотите изменить значения внутри запроса, чтобы вы могли написать для него функцию

void DoStuff()
{
    Func<string, Foo, bool> test = (y, x) => { x.Bar = y; return true; };
    List<Foo> mylist = new List<Foo>();
    var v = from x in mylist
            where test("value", x)
            select x;
}

class Foo
{
    string Bar { get; set; }
}

Но не читай, если это то, что ты имеешь в виду.

  • 0
    Это происходит в правильном направлении, потому что для перечисления v потребуется что-то другое, иначе ничего не будет сделано.

Ещё вопросы

Сообщество Overcoder
Наверх
Меню