ASP.NET MVC中如何初始化含管理员的公共静态List<Person>？

咱们来梳理下你代码里的几个核心问题，以及对应的修复方案：

1. 静态列表初始化时机错误 #

你目前是在Index Action里添加管理员，但这个逻辑只有用户访问Index页面时才会触发。如果用户先访问ViewCustomer或者其他页面，myListPerson里就不会有默认的管理员用户。而且如果用户多次刷新Index页面，还会重复添加同一个管理员实例（因为myEmployee是控制器的实例字段，每次添加的都是同一个对象引用，列表里会出现多条重复的管理员条目）。

2. 实例字段复用导致的潜在数据覆盖问题 #

你的myEmployee和myCustomer是控制器的实例成员，不是在每个Action里创建的新对象。比如在ViewCustomer里，每次请求都会复用同一个myCustomer实例，修改属性后再添加到静态列表——这会导致列表里的所有Customer条目其实都指向同一个对象，后续的请求会覆盖之前的条目数据！

修复方案 #

第一步：用静态构造函数初始化默认管理员 #

把静态列表的初始化和管理员添加放到静态构造函数里，这样当HomeController类第一次被加载时就会执行，而且只会执行一次，确保不管用户先访问哪个页面，myListPerson里都有默认的管理员。

第二步：在每个Action里创建新的对象实例 #

不要复用控制器级别的myCustomer和myEmployee，而是在需要添加新用户时，创建对应的新对象，彻底避免数据覆盖问题。

修改后的控制器代码： #

public class HomeController : Controller 
{
    // 静态列表声明
    public static List<Person> myListPerson;

    // 静态构造函数：类第一次加载时执行，仅执行一次
    static HomeController()
    {
        myListPerson = new List<Person>();
        // 添加默认管理员
        Employee admin = new Employee();
        admin.Name = "1";
        admin.Surname = "1";
        admin.Email = "1";
        admin.Cell = 0;
        admin.IDNumber = 0;
        admin.EmployeeID = "Emp" + 1000;
        // 视图里用到了Username字段，原代码未赋值，这里补充
        admin.Username = "admin";
        myListPerson.Add(admin);
    }

    // 把Cus_Number改成静态的，否则每次控制器实例化都会重置为1000
    private static int Cus_Number = 1000;

    public ActionResult ViewCustomer(string Name, string Surname, string Email, int Cell, int IDNumber) 
    {
        Cus_Number += 1;
        // 每次创建新的Customer对象
        Customer myCustomer = new Customer();
        myCustomer.Name = Name;
        myCustomer.Surname = Surname;
        myCustomer.Email = Email;
        myCustomer.Cell = Cell;
        myCustomer.IDNumber = IDNumber;
        myCustomer.CustomerID = "Cus" + Cus_Number;
        myListPerson.Add(myCustomer);
        return View(myListPerson);
    } 

    public ActionResult Index() 
    {
        // 管理员已在静态构造函数中添加，无需重复处理
        return View(myListPerson);
    } 
}

额外说明 #

你的视图里显示了Username字段，但原代码中没有给管理员赋值，所以我在静态构造函数里补充了admin.Username = "admin"，确保视图能正常显示该字段。

为什么这样改能解决问题？ #

• 静态构造函数是.NET类加载时的专属入口，只会执行一次，完美保证默认管理员只会被添加一次，不受请求次数和访问顺序影响。
• 每个Action里创建新的对象实例，彻底避免了复用同一个对象导致的列表数据被覆盖的问题。
• 把Cus_Number改成静态的，避免了每次控制器实例化（每个请求都会创建新的控制器实例）时重置数值，保证生成的CustomerID不会重复。

内容的提问来源于stack exchange，提问作者user9856347