独自のクラスオブジェクト属性がオーバーライドされます

Question

属性Stringを持つオブジェクトを作成しました。もう1つはList<String>です。 私はstatic List<MyObject>を作成しました。ここで私はすべてのオブジェクトを追加します。 今、私の問題は、2番目の属性がオーバーライドされています。例えば

私は3つのオブジェクトがあります。

Object1: "Name"; List with 3 Strings 
Object2: "Name2"; List with 2 Strings 
Object3: "Name3"; List with 5 Strings 

を私は私のオブジェクトリストにここでそれらを追加した場合、それはとても

Name; List with 5 Strings 
Name2; List with 5 Strings 
Name3; List with 5 Strings 

それは内の他のすべてのオブジェクトへの第二の属性をオーバーライドするようになります。リスト。

コード：

for (int i = 0; i < 100; i++) 
{ 
    if (elo.ReadObjMask(i) > 0) 
    { 
     var iRet = elo.PrepareObjectEx(0, 0, i); 
     maskenname = elo.ObjMName(); 
     if (maskenname != "") 
     { 
      for (int e = 0; e < 50; e++) 
      { 
       string eigenschaft = elo.GetObjAttribName(e); 

       if (eigenschaft != "" && eigenschaft != "-") 
       { 
        eigenschaften.Add(eigenschaft); 
       } 
      } 
      allMasks.Add(maskenname); 

     } 
     else 
     { 
      // Do nothing 
     } 
     EloMask emask = new EloMask(maskenname, eigenschaften); 
     staticVariables.allMask.Add(emask); 
     eigenschaften.Clear(); 
    } 
} 

ここでは私のオブジェクトのクラスである：ここで

public class EloMask 
{ 
    public string name; 
    public List<String> eigenschaften; 

    public EloMask(string iname, List<String> ieigenschaften) 
    { 
     name = iname; 
     eigenschaften = ieigenschaften; 
    } 
} 

Answer 1

List<string>は、コピーではなくリストへの参照を渡すため、常に同じインスタンスを指します。その結果、リストはクリアされ、そのリストを渡した各EloMaskのために再度入力されます。

代わりに新しいリストを作成し、あなたの問題を解決するには、次の

if (elo.ReadObjMask(i) > 0) 
{ 
    var iRet = elo.PrepareObjectEx(0, 0, i); 
    maskenname = elo.ObjMName(); 

    // create a new list here!!! 
    var eigenschaften = new List<string>(); 

    if (maskenname != "") 
    { 
     for (int e = 0; e < 50; e++) 
     { 
      string eigenschaft = elo.GetObjAttribName(e); 

      if (eigenschaft != "" && eigenschaft != "-") 
      { 
       eigenschaften.Add(eigenschaft); 
      } 
     } 

     allMasks.Add(maskenname); 

    } 

    EloMask emask = new EloMask(maskenname, eigenschaften); 
    staticVariables.allMask.Add(emask); 

    // clearing the list is no longer needed     
} 

Answer 2

はあなたがやりたいことができる方法の例です：

public static List<Person> PersonsList = new List<Person>(); 
    public static Random rd = new Random(); 
    static void Main(string[] args) 
    { 

     for (int i = 0; i < 5; i++) 
     { 
      List<string> tmpAbilities = new List<string>() {((char)rd.Next(255)).ToString(), ((char)rd.Next(255)).ToString() , ((char)rd.Next(255)).ToString() }; 
      Person tmpPerson = new Person("TmpName_"+i,tmpAbilities); 
      PersonsList.Add(tmpPerson); 
     } 
     foreach (var persona in PersonsList) 
     { 
      Console.WriteLine(persona); 
     } 
    } 


    public class Person 
    { 

     public string Name { get; set; } 

     public List<string> Abilities; 

     public Person(string name,List<string> abilities) 
     { 
      Name = name; 
      Abilities = abilities; 
     } 

     public override string ToString() 
     { 
      string retVal = $"Name: {Name}\n"; 
      foreach (var ability in Abilities) 
      { 
       retVal += $"Ability : {ability}\n"; 
      } 
      return retVal; 
     } 
    } 

Answer 3

 for (int i = 0; i < 100; i++) 
     { 

      if (elo.ReadObjMask(i) > 0) 
      { 
       // Create a new listin here 
       eigenschaften = new List<string>(); 

       var iRet = elo.PrepareObjectEx(0, 0, i); 
       maskenname = elo.ObjMName(); 
       if (maskenname != "") 
       { 
        for (int e = 0; e < 50; e++) 
        { 
         string eigenschaft = elo.GetObjAttribName(e); 

         if (eigenschaft != "" && eigenschaft != "-") 
         { 
          eigenschaften.Add(eigenschaft); 
         } 
        } 
        allMasks.Add(maskenname); 

       } 

       else 
       { 
        // Do nothing 
       } 
       EloMask emask = new EloMask(maskenname, eigenschaften); 
       staticVariables.allMask.Add(emask); 
      } 
     }