Как вы объединяете списки в C #?

171

Если бы у меня был:

List<string> myList1;
List<string> myList2;

myList1 = getMeAList();
// Checked myList1, it contains 4 strings

myList2 = getMeAnotherList();
// Checked myList2, it contains 6 strings

myList1.Concat(myList2);
// Checked mylist1, it contains 4 strings... why?

Я запускал код, подобный этому, в Visual Studio 2008 и устанавливал точки останова после каждого выполнения. После myList1 = getMeAList();, myList1содержит четыре строки, и я нажал кнопку «плюс», чтобы убедиться, что они не все нулевые.

После myList2 = getMeAnotherList();, myList2содержит шесть строк, и я проверил , чтобы убедиться , что они не были пустыми ... После myList1.Concat(myList2);myList1 содержал только четыре строки. Это почему?

Matt
источник

Ответы:

97

Попробуй это:

myList1 = myList1.Concat(myList2).ToList();

Concat возвращает IEnumerable <T>, то есть два списка, вместе взятые, он не изменяет ни один из существующих списков. Кроме того, поскольку он возвращает IEnumerable, если вы хотите присвоить его переменной List <T>, вам придется вызывать ToList () для возвращаемого IEnumerable <T>.

Джонатан Рупп
источник
6
Теперь, когда я перечитал вопрос, .AddRange () звучит так, как на самом деле хочет ОП.
Джонатан Рапп
@Kartiikeya, если он говорит, что аргументы недействительны, у вас нет оператора использования для System.Linq, или один из них не являетсяIEnumerable<T>
Джонатан Рапп
12
targetList = list1.Concat(list2).ToList();

Работает нормально, я так думаю. Как уже было сказано, Concat возвращает новую последовательность и, преобразуя результат в List, отлично справляется со своей задачей.

Баласубрамани М
источник
4

Стоит также отметить, что Concat работает в постоянном времени и в постоянной памяти. Например, следующий код

        long boundary = 60000000;
        for (long i = 0; i < boundary; i++)
        {
            list1.Add(i);
            list2.Add(i);
        }
        var listConcat = list1.Concat(list2);
        var list = listConcat.ToList();
        list1.AddRange(list2);

дает следующие метрики времени / памяти:

After lists filled mem used: 1048730 KB
concat two enumerables: 00:00:00.0023309 mem used: 1048730 KB
convert concat to list: 00:00:03.7430633 mem used: 2097307 KB
list1.AddRange(list2) : 00:00:00.8439870 mem used: 2621595 KB
Дмитрий Андриевский
источник
2

Я знаю, что это старо, но я наткнулся на этот пост, быстро подумав, что Конкат будет моим ответом. Союз отлично сработал для меня. Обратите внимание, он возвращает только уникальные значения, но, зная, что я все равно получал уникальные значения, это решение сработало для меня.

namespace TestProject
{
    public partial class Form1 :Form
    {
        public Form1()
        {
            InitializeComponent();

            List<string> FirstList = new List<string>();
            FirstList.Add("1234");
            FirstList.Add("4567");

            // In my code, I know I would not have this here but I put it in as a demonstration that it will not be in the secondList twice
            FirstList.Add("Three");  

            List<string> secondList = GetList(FirstList);            
            foreach (string item in secondList)
                Console.WriteLine(item);
        }

        private List<String> GetList(List<string> SortBy)
        {
            List<string> list = new List<string>();
            list.Add("One");
            list.Add("Two");
            list.Add("Three");

            list = list.Union(SortBy).ToList();

            return list;
        }
    }
}

Выход:

One
Two
Three
1234
4567
Esaith
источник
2

Посмотрите на мою реализацию. Это безопасно от нулевых списков.

 IList<string> all= new List<string>();

 if (letterForm.SecretaryPhone!=null)// first list may be null
     all=all.Concat(letterForm.SecretaryPhone).ToList();

 if (letterForm.EmployeePhone != null)// second list may be null
     all= all.Concat(letterForm.EmployeePhone).ToList(); 

 if (letterForm.DepartmentManagerName != null) // this is not list (its just string variable) so wrap it inside list then concat it 
     all = all.Concat(new []{letterForm.DepartmentManagerPhone}).ToList();
Башир Аль-Момани
источник