使用EF Core和ASP.NET Web API保存训练数据失败的排查求助

以下是针对你遇到问题的可能原因及对应的排查、解决步骤：

1. 客户端未检查请求响应状态 #

你的WorkoutManager.CreateAsync方法仅捕获了HttpRequestException，但未验证服务器返回的响应状态码。即使服务器因内部错误返回500，客户端也会误以为请求成功，实际数据并未保存。

解决方法：
在客户端请求后添加响应校验：

public static async Task CreateAsync(Workout item)
{
    try
    {
        var json = JsonConvert.SerializeObject(item);
        var data = new StringContent(json, Encoding.UTF8, "application/json");
        var response = await Client._httpClient.PostAsync(Client._url + $"Workout", data);
        // 强制校验响应状态，非2xx会抛出异常
        response.EnsureSuccessStatusCode();
        // 可选：读取服务器响应内容，便于排查
        var responseMsg = await response.Content.ReadAsStringAsync();
        Console.WriteLine($"Server response: {responseMsg}");
    }
    catch (HttpRequestException e)
    {
        Console.WriteLine($"Request error: {e.Message}");
    }
}

2. 实体关系配置缺失 #

EF Core需要明确的关系配置才能正确处理关联实体（如Creator、ExerciseTypeGroups）的级联保存。如果未配置外键和导航属性，可能导致关联数据未被追踪，或保存时忽略关联实体。

解决方法：

• 确保ExerciseTypeGroup类包含指向Workout的外键：

public class ExerciseTypeGroup
{
    [Key]
    public int Id { get; set; }
    // 其他属性
    public int WorkoutId { get; set; } // 外键字段
    public Workout Workout { get; set; } // 导航属性
}

• 在DbContext的OnModelCreating方法中配置实体关系：

protected override void OnModelCreating(ModelBuilder modelBuilder)
{
    // 配置Workout与Creator的关系
    modelBuilder.Entity<Workout>()
        .HasOne(w => w.Creator)
        .WithMany() // 如果User没有Workouts集合，用WithMany()
        .HasForeignKey(w => w.CreatorId)
        .OnDelete(DeleteBehavior.Restrict);

    // 配置Workout与ExerciseTypeGroups的一对多关系
    modelBuilder.Entity<Workout>()
        .HasMany(w => w.ExerciseTypeGroups)
        .WithOne(etg => etg.Workout)
        .HasForeignKey(etg => etg.WorkoutId)
        .OnDelete(DeleteBehavior.Cascade);
}

3. Status属性未设置有效值 #

你使用无参构造函数创建Workout对象，但该构造函数未初始化Status属性。如果Status是枚举类型且数据库字段不允许默认值（如枚举从1开始定义），EF Core会因验证失败导致保存失败。

解决方法：
创建Workout时显式设置Status：

Workout workout = new Workout();
workout.Status = Status.Waiting; // 设置为枚举中的有效值

4. 异步调用阻塞导致操作未完成 #

你的CompleteWorkout方法使用Task.Run(() => WorkoutManager.CreateAsync(workout)).GetAwaiter().GetResult()阻塞异步操作，可能引发死锁或导致请求未完成就终止。

解决方法：
直接await异步方法，避免阻塞：

private async void CompleteWorkout()
{
     Workout workout = new Workout();
     workout.ExerciseTypeGroups = new List<ExerciseTypeGroup>(ExerciseTypeGroups);
     workout.Status = Status.Waiting; // 补充Status设置
     string userIdString = Preferences.Get("UserID", defaultValue: null);

     if (int.TryParse(userIdString, out int userId))
     {
         workout.CreatorId = userId;
         workout.Creator = await UserManaging.ReadAsync(userId);

         // 直接await异步方法
         await WorkoutManager.CreateAsync(workout);
     }
     else
     {
         Console.WriteLine("Failed to parse user ID from preferences.");
     }
}

5. 服务器端DbContext生命周期错误 #

如果服务器端的DbContext被注册为单例，会导致多请求间状态追踪混乱，引发保存失败。ASP.NET Core中DbContext应使用Scoped生命周期。

解决方法：
在服务器Program.cs中正确注册DbContext：

builder.Services.AddDbContext<YourDbContext>(options => 
    options.UseSqlServer(builder.Configuration.GetConnectionString("YourConnectionString")));

排查步骤总结 #

1. 先在客户端添加响应校验，确认请求是否真的成功到达服务器并返回2xx状态。
2. 在服务器端添加日志，记录SaveChangesAsync的执行结果（如影响行数）及异常信息，定位是否是数据库层面的错误。
3. 检查实体类的必填字段（如Status）是否都设置了有效值。
4. 验证DbContext的实体关系配置是否正确，确保关联实体能被级联保存。
5. 确认服务器端数据库连接字符串正确，且数据库用户有写入权限。

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