文档应用编辑已有分类Order值后导航菜单排序异常问题求助

问题根源 #

你当前的代码核心问题是InsertCategory的逻辑仅适配新增分类场景，完全没有考虑编辑已有分类的特殊处理，且本身的移位逻辑存在多处错误：

1. 未对Categories列表按Order字段排序就直接用索引遍历，遍历顺序完全混乱，移位操作自然不符合预期
2. 编辑场景下没有先移除当前编辑的旧分类记录，计算移位时会把自己纳入统计，导致位置计算错误
3. 循环中的if (Categories[i].Order == category.Order) { Categories[i].Order--; break; }逻辑完全无效，相当于修改后立刻回滚还终止循环，仅能处理极小部分新增场景
4. SaveCategory执行顺序错误，先调用InsertCategory再判断分类是否存在，此时内存列表已经被修改，查询逻辑完全失效

修复方案 #

1. 新增统一的排序调整方法 #

替换原来的InsertCategory方法，新增支持新增/编辑双场景的排序逻辑：

// CategoryService.cs
private async Task AdjustCategoryOrder(Category category, bool isNew)
{
    await GetCategories();
    // 先确保列表按Order升序排列，后续操作才有效
    Categories = Categories.OrderBy(c => c.Order).ToList();
    int oldOrder = -1;

    if (!isNew)
    {
        // 编辑场景先取出原有记录，移除后再计算移位，避免干扰
        var existingCat = Categories.First(c => c.Id == category.Id);
        oldOrder = existingCat.Order;
        Categories.Remove(existingCat);
    }

    int targetOrder = category.Order;

    if (!isNew)
    {
        if (targetOrder > oldOrder)
        {
            // 分类往后挪：中间介于旧位置+1到新位置的分类全部前移一位
            foreach (var cat in Categories.Where(c => c.Order > oldOrder && c.Order <= targetOrder))
            {
                cat.Order--;
            }
        }
        else if (targetOrder < oldOrder)
        {
            // 分类往前挪：中间介于新位置到旧位置-1的分类全部后移一位
            foreach (var cat in Categories.Where(c => c.Order >= targetOrder && c.Order < oldOrder))
            {
                cat.Order++;
            }
        }
    }
    else
    {
        // 新增场景：所有大于等于目标位置的分类全部后移一位
        foreach (var cat in Categories.Where(c => c.Order >= targetOrder))
        {
            cat.Order++;
        }
    }

    // 把当前分类加回列表，赋值正确顺序
    category.Order = targetOrder;
    Categories.Add(category);
    // 最终重新排序保证顺序正确
    Categories = Categories.OrderBy(c => c.Order).ToList();
}

2. 修正SaveCategory的执行逻辑 #

调整判断顺序，先确认是新增还是编辑场景再执行对应操作：

// CategoryService.cs
public async Task<Category> SaveCategory(Category category)
{
    var existing = await categoryRepository.GetByIdAsync(category.Id);
    bool isNew = existing == null;

    // 先调整所有分类的排序
    await AdjustCategoryOrder(category, isNew);

    if (isNew)
    {
        await categoryRepository.AddAsync(category);
    }
    else
    {
        await categoryRepository.SaveAsync(category);
    }

    // 批量更新所有被修改了Order的分类到数据库
    await categoryRepository.InsertAsync(Categories);
    EventHelper.NotifyCategoryListChanged(Categories, EventArgs.Empty);

    return category;
}

额外注意事项 #

• 如果你的Order字段允许跳号（比如0、1、3，没有2），可以在排序调整后加一段逻辑重排所有Order为连续值，避免后续出现顺序异常
• 前端传入Order值时建议做边界校验，不能小于0，也不能大于当前分类总数，避免越界错误

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