C#如何创建泛型类封装仓储层返回结果与响应消息

你自己写的GenericResponse<T>泛型响应类思路完全可行，属于后端分层开发里非常通用的结果包装模式，核心作用就是把业务数据和操作状态、提示信息绑定后跨层传递，避免用抛异常、out参数传消息这类难维护的写法，完全符合常规开发实践。
不过现有代码有几个可以调整的细节：

• 错误分支下不要给ModelToReturn赋值空的ApplicationUser实例，直接设为null即可，否则上层判断时很容易把空对象当成有效数据，出现逻辑漏判
• 目前类里只有消息字段，建议补充操作成功标记、业务状态码字段，上层不用靠「消息是否为空」判断请求是否成功，逻辑更可靠
• 可以给通用响应类补充几个快捷构造方法，不用每次new完挨个给属性赋值，减少重复代码

你要实现的需求本质上就是.NET生态广泛应用的Result模式，和你自己写的结构逻辑完全一致，只是字段设计更严谨，标准实现参考如下：

/// <summary>
/// 带业务数据的通用操作返回结果
/// </summary>
/// <typeparam name="T">业务数据类型</typeparam>
public class ApiResult<T>
{
    /// <summary>
    /// 操作是否成功
    /// </summary>
    public bool Success { get; set; }
    /// <summary>
    /// 提示/错误消息
    /// </summary>
    public string Message { get; set; } = string.Empty;
    /// <summary>
    /// 业务状态码，可对齐自己系统的错误码规范
    /// </summary>
    public int Code { get; set; }
    /// <summary>
    /// 返回的业务数据
    /// </summary>
    public T? Data { get; set; }

    /// <summary>
    /// 构造成功结果
    /// </summary>
    public static ApiResult<T> Ok(T data, string message = "")
    {
        return new ApiResult<T>
        {
            Success = true,
            Code = 200,
            Data = data,
            Message = message
        };
    }

    /// <summary>
    /// 构造失败结果
    /// </summary>
    public static ApiResult<T> Fail(string message, int code = 400)
    {
        return new ApiResult<T>
        {
            Success = false,
            Code = code,
            Data = default,
            Message = message
        };
    }
}

/// <summary>
/// 无业务数据的通用操作返回结果，用于增删改等不需要返回数据的场景
/// </summary>
public class ApiResult
{
    public bool Success { get; set; }
    public string Message { get; set; } = string.Empty;
    public int Code { get; set; }

    public static ApiResult Ok(string message = "")
    {
        return new ApiResult { Success = true, Code = 200, Message = message };
    }

    public static ApiResult Fail(string message, int code = 400)
    {
        return new ApiResult { Success = false, Code = code, Message = message };
    }
}

用这个结构改造你之前的仓储代码会简洁很多：存储过程返回非0响应码时直接return ApiResult<ApplicationUser>.Fail(responseMessage)，查询成功就return ApiResult<ApplicationUser>.Ok(user)，不用每次手动给每个属性赋值。

• 这个结果包装类可以在仓储层、服务层、控制器层全程通用，控制器拿到结果后直接判断Success字段即可：成功就返回Ok(result.Data)，失败就返回BadRequest(result.Message)或对应状态码，逻辑非常统一
• 如果全项目统一使用，可以进一步封装自定义的IActionResult返回类型，或者加全局过滤器自动处理Result类型的返回，不用每个Action都手动写判断逻辑
• 不建议把数据库返回的原始提示、异常信息直接透传给前端，最好在服务层做一层转义，避免暴露数据库结构、系统内部错误信息带来安全风险
• 你现有代码里先调用ExecuteNonQuery再调用ExecuteReader的写法要注意，执行完非查询后再读取结果集，需要确认存储过程确实返回了结果集，否则会抛出异常，建议直接用ExecuteReaderAsync一次性处理输出参数和结果集读取。

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