Вопрос

Если у меня есть:

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 содержал только четыре строки. Почему это так?

Это было полезно?

Решение

Concat возвращает новую последовательность без изменения исходного списка . Попробуйте myList1.AddRange(myList2).

Другие советы

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

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

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

targetList = list1.Concat(list2).ToList();

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

Стоит также отметить, что 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

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

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

взгляните на мою реализацию its safe from null lists

 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(); 
Лицензировано под: CC-BY-SA с атрибуция
Не связан с StackOverflow
scroll top