Entity Framework添加自引用实体时遇堆栈溢出错误的解决办法

嘿，我看了你遇到的问题，堆栈溢出的根源其实很明显，咱们一步步拆解解决：

1. 最核心的问题：Id属性的递归调用 #

先看你的Category类里的Id属性实现：

public int Id { get => this.Id; set => this.Id = value; }

这个写法完全是自递归死循环！当你尝试获取Id值时，get => this.Id会再次触发Id的getter，无限重复下去，直接撑爆调用栈导致堆栈溢出。这是必须第一时间修复的问题。

修正后的Id属性应该用标准的自动属性，因为你已经加了[Key]和[DatabaseGenerated]特性，EF会自动处理主键的生成：

public int Id { get; set; }

2. 控制器逻辑的优化 #

你的控制器代码里有一些可以简化和避免潜在问题的地方：

• 添加子分类时，不需要用Include(c => c.SubCategory)查询父分类的子集合，EF会自动维护导航属性和外键的关系，你只需要设置子分类的IsSubCategoryOf外键即可。
• 可以简化对象初始化的代码，避免重复复制属性。

修正后的控制器代码示例：

try
{
    Category categoryDb = new Category
    {
        CategoryName = category.CategoryName,
        AddedAt = category.AddedAt,
        IsActive = category.IsActive
    };

    if (category.IsSubCategoryOf == -1)
    {
        // 根分类，直接添加到上下文
        _context.Categories.Add(categoryDb);
    }
    else
    {
        // 子分类，直接设置外键关联父分类
        categoryDb.IsSubCategoryOf = category.IsSubCategoryOf;
        _context.Categories.Add(categoryDb);
    }

    _context.SaveChanges();
    return Ok(categoryDb);
}
catch (Exception ex)
{
    // 建议在这里记录异常日志，方便后续调试
    // _logger.LogError(ex, "添加分类失败");
    return BadRequest("添加分类失败，请检查输入参数");
}

如果你更倾向于通过父对象关联子分类（而不是直接设外键），可以用Find方法（比Where(...).FirstOrDefault()更高效，因为会优先查上下文缓存）：

else
{
    var parentCategory = _context.Categories.Find(category.IsSubCategoryOf);
    if (parentCategory == null)
    {
        return NotFound("指定的父分类不存在");
    }
    parentCategory.SubCategory.Add(categoryDb);
    // 不需要单独Add子分类，EF会自动跟踪并处理
    _context.SaveChanges();
}

3. 其他潜在问题的预防 #

为了避免空引用异常，建议在Category类的构造函数里初始化导航属性：

public Category()
{
    SubCategory = new List<Category>();
    Ads = new List<Ad>();
}

另外，你在DbContext里的EF配置是正确的，HasOne(c => c.ParentCategory).WithMany(c => c.SubCategory).HasForeignKey(c => c.IsSubCategoryOf)这个映射完全符合自引用分类的关系需求。

最后总结 #

先修复Id属性的递归死循环，这是导致堆栈溢出的根本原因。然后优化控制器的逻辑，简化代码并避免不必要的查询。最后初始化导航属性预防空引用，这样就能正常添加分类数据啦！

内容的提问来源于stack exchange，提问作者EMRE KAAN SEYLİM