为什么我的清单是空的？

Question

类管理：

namespace AnimalShelter 
{ 
    class Administration 
    { 
    private List<Animal> animalList; 

    public Administration() 
    { 
     animalList = new List<Animal>(); 
    } 

    public bool AddAnimal(Animal animal) 
    { 
     bool status = false; 
     foreach (Animal TempAnimal in animalList) 
     { 
      if (TempAnimal.ChipRegistrationNumber == animal.ChipRegistrationNumber) 
      { 
       status = false; 
      } 
      else 
      { 
       status = true; 
       animalList.Add(animal); 
      } 
     } 
     return status; 
    } 

正如你所看到的，我做了一个列表，在constuctor对其进行初始化。

在AddAnimal方法中，如果数字与其他数字相同，则循环访问列表。如果数字匹配，则不能使用该特定ChipNumber添加其他动物。

现在我的表格：

private Administration administration; 

public AdministrationForm() 
{ 
    administration = new Administration(); 
} 

private void btnCreateAnimal_Click(object sender, EventArgs e) 
{ 
      Animal cat1 = new Cat(params..); 
      if (administration.AddAnimal(cat1)) 
      { 
       MessageBox.Show("Following animal has succesfully been 
       added: " + Environment.NewLine + cat1.ToString()); 
      } 
      else 
      { 
       MessageBox.Show("Animal with this chipnumber already exists. 
       Please Choose another one."); 
      } 
} 

所以在这里，我只是陈述轻松如果方法正确，创造了动物。如果它是假的，不要添加动物。

问题：如果我将动物添加到列表中，列表始终为空。就好像它不断地刷新/清除列表。我在这里做错了什么？

希望这可以理解，否则让我知道！

Answer 1

您需要将添加mthod外foreach循环

public bool AddAnimal(Animal animal) 
    { 
     bool status = false; 
     foreach (Animal TempAnimal in animalList) 
     { 
      if (TempAnimal.ChipRegistrationNumber == animal.ChipRegistrationNumber) 
      { 
       status = false; 
      } 
      else 
      { 
       status = true; 

      } 
     } 
    if(status) 
animalList.Add(animal); 
     return status; 
    } 

Answer 2

因为您创建后列表为空，当您添加第一个动物时foreach从不执行，因为列表中没有任何内容。替换：

public bool AddAnimal(Animal animal) 
{ 
    bool status = true; 
    foreach (Animal TempAnimal in animalList) 
    { 
     if (TempAnimal.ChipRegistrationNumber == animal.ChipRegistrationNumber) 
     { 
      status = false; 
     } 
     else 
     { 
      status = true; 
     } 
    } 
    if (status) 
    { 
     animalList.Add(animal); 
    } 
    return status; 
} 

使用LINQ，我们可以把它缩短：

public bool AddAnimal(Animal animal) 
{ 
    if (!animalList.Any(a => a.ChipRegistrationNumber == animal.ChipRegistrationNumber)) 
    { 
     animalList.Add(animal); 
     return true; 
    } 
    return false; 
} 

Answer 3

你不需要status可言。简单更好。

public bool AddAnimal(Animal animal) 
{ 
    foreach (Animal TempAnimal in animalList) 
     if (TempAnimal.ChipRegistrationNumber == animal.ChipRegistrationNumber) return false; 
    animalList.Add(animal); 
    return true; 
} 

这是有2种方法的等效：

public bool IsDuplicate(Animal animal) 
{ 
    foreach (Animal TempAnimal in animalList) 
     if (TempAnimal.ChipRegistrationNumber == animal.ChipRegistrationNumber) return true; 
    return false; 
} 
public bool AddAnimal(Animal animal) 
{ 
    if (IsDuplicate(animal)) return false; 
    animalList.Add(animal); 
    return true; 
} 

Add返回true，如果该项目已被添加，假当存在重复，因此该项目还没有被添加。