Добавление пользовательских объектов в проверяемых списки в C#

Question

У меня есть объект, который хранит две строки и переопределяет метод ToString(), чтобы вернуть одну из этих строк

  fooBar temp; 
      foreach (string foo in bar) 
      { 
       temp = new fooBar(); <--- Why is this line needed when I 
       am overwriting the code and path on each cycle before adding? 

       temp.code += "."; 
       temp.path += "."; 

       Console.WriteLine(temp); 
       clbFooBar.Items.Add(temp); 
      } 

начинает печать на консоль подчиняется перезаписан код и значения строки пути, но элементы отмеченного элемента списка - это все то же самое (ToString() последнего элемента), что происходит в основном процессе, что не сразу становится очевидным?

Если изменить его clbFooBar.Items.Add(temp.ToString()); она работает абсолютно нормально, но это не было бы так же, как temp.ToString.ToString(), потому что .Add звонки .ToString() в первую очередь? Я понял, как консольную печать и .Add будет действовать так же, видя, как они оба называют ToString()

EDIT: temp = new fooBar(); это исправить мою проблему, я просто хочу знать, почему это исправить, когда интуиция заставляет меня думать, что перезапись двух строк, кода и пути, должно быть достаточно, и поэтому без этой повторной инициализации все элементы в отмеченном списке будут одинаковыми.

Answer 1

Так ты вопрос, кажется, почему ваш код работает правильно, если вы держать линию

temp = new fooBar(); 

в цикле, но наполнит вашу CheckedListBox со всеми равными предметами, если вы ommit этой строки ,

Причина заключается в том, что если вы создаете только один экземпляр fooBar вне цикла, как, что:

fooBar temp; = new fooBar(); 
foreach (string foo in bar) 
{ 
    temp.code += "."; 
    temp.path += "."; 
    clbFooBar.Items.Add(temp); 
} 

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

При создании новых экземпляров внутри вашей петли так:

foreach (string foo in bar) 
{ 
    fooBar temp; = new fooBar(); 
    temp.code += "."; 
    temp.path += "."; 
    clbFooBar.Items.Add(temp); 
} 

у вас есть новый экземпляр для каждой итерации и не изменять уже добавленные экземпляры.

---

Update: только что заметил, что они все еще будут выглядеть так же, как вы установите их все к тому, что значение по умолчанию code и temp есть плюс ".".Так что все еще немного неясно, что вы на самом деле пытаетесь сделать. может быть примерно так:

string s = string.Empty; 
foreach (string foo in bar) 
{ 
    s += "."; 
    fooBar temp; = new fooBar(); 
    temp.code = s; 
    temp.path = s; 
    clbFooBar.Items.Add(temp); 
} 

Answer 2

Вы создаете temp вне цикла foreach, а затем инициализируете его внутри.

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

Вот почему вы получаете значение последнего объекта (потому что на самом деле есть только), когда вы смотрите в списке.

Я предполагаю, что то, что вы на самом деле хотите сделать, это добавить каждый элемент (foo) в строке до CheckedListBox. Если это так, используйте это.

 foreach (string foo in bar) 
     { 
      foo.code += "."; 
      foo.path += "."; 

      Console.WriteLine(foo); 
      clbFooBar.Items.Add(foo); 
     }