将模型写入数据库的实现是否合理？求优化方案

我编写了如下代码用于将模型值写入数据库，但不确定这种实现方式是否正确。

我希望复用SaveData方法来处理不同流程的数据写入（本文仅展示SignUp流程），为此想到的方案是在各流程中创建DataTable并传入SaveData方法。作为新手，我对这种实现的效率与合理性存在疑问。

以下是将模型值转换为DataTable的SignUp方法：

public static void SignUp(AccountSignUp model)
{
    DataTable DataTable = new DataTable();
    DataTable.Columns.Add(new DataColumn("Id", typeof(Int32)));
    DataTable.Columns.Add(new DataColumn ("Username", typeof(string)));
    DataTable.Columns.Add(new DataColumn ("FirstName", typeof(string)));
    DataTable.Columns.Add(new DataColumn ("LastName", typeof(string)));
    DataTable.Columns.Add(new DataColumn ("EmailAddress", typeof(string)));
    DataTable.Columns.Add(new DataColumn("PasswordHash", typeof(string)));

    DataRow DataRow = DataTable.NewRow();
    DataRow["UserName"] = model.UserName;
    DataRow["PasswordHash"] = model.Password;
    DataRow["Firstname"] = model.FirstName;
    DataRow["LastName"] = model.LastName;
    DataRow["EmailAddress"] = model.EmailAddress;
    DataTable.Rows.Add(DataRow);
    DataAccess.SaveData(DataTable);
}

以下是将DataTable写入数据库的SaveData方法：

public static void SaveData(DataTable data)
{
    using (SqlBulkCopy copy = new SqlBulkCopy(ConnectionString)) 
    {
        copy.DestinationTableName = "UserAccounts"; //WIP/Debugging 
        copy.WriteToServer(data);
    }
}

请问这种实现方式是否合适？是否存在更高效的方案？我认为应该可以直接将模型作为行传入数据库，无需先创建DataTable。

这种实现是否合适？ #

• 功能上可行：代码能完成单条用户数据写入的需求，SqlBulkCopy也能正常处理DataTable数据。
• 但存在明显不合理性：
  • 效率浪费：SqlBulkCopy是为批量插入大量数据设计的，单条数据用它反而会增加初始化DataTable、SqlBulkCopy实例的额外开销，效率远不如普通单条插入。
  • 冗余代码：每个流程都要手动创建DataTable、定义列、映射模型字段，重复代码多，后期模型字段修改时，所有相关逻辑都要同步修改，维护成本高。
  • 映射风险：代码中DataRow的键名存在大小写不一致（比如Firstname和FirstName），若数据库列名严格区分大小写，会导致插入失败。

更高效合理的方案 #

方案1：参数化SQL单条插入（适配单数据场景） #

针对单条数据写入，直接用参数化SQL，避免DataTable和SqlBulkCopy的额外开销，同时防止SQL注入：

// 单条账户插入方法
public static void InsertAccount(AccountSignUp model)
{
    string sql = @"INSERT INTO UserAccounts (Username, FirstName, LastName, EmailAddress, PasswordHash)
                   VALUES (@Username, @FirstName, @LastName, @EmailAddress, @PasswordHash)";

    using (SqlConnection conn = new SqlConnection(ConnectionString))
    using (SqlCommand cmd = new SqlCommand(sql, conn))
    {
        cmd.Parameters.AddWithValue("@Username", model.UserName);
        cmd.Parameters.AddWithValue("@FirstName", model.FirstName);
        cmd.Parameters.AddWithValue("@LastName", model.LastName);
        cmd.Parameters.AddWithValue("@EmailAddress", model.EmailAddress);
        cmd.Parameters.AddWithValue("@PasswordHash", model.Password);

        conn.Open();
        cmd.ExecuteNonQuery();
    }
}

// SignUp流程直接调用
public static void SignUp(AccountSignUp model)
{
    DataAccess.InsertAccount(model);
}

方案2：通用批量插入（适配多流程批量场景） #

如果需要支持多流程的批量数据写入，可以封装通用的模型转DataTable方法，减少重复代码，同时保留SqlBulkCopy的批量优势：

// 通用模型转DataTable工具方法
public static DataTable ToDataTable<T>(IEnumerable<T> items)
{
    DataTable dataTable = new DataTable(typeof(T).Name);
    PropertyInfo[] props = typeof(T).GetProperties(System.Reflection.BindingFlags.Public | System.Reflection.BindingFlags.Instance);
    
    foreach (PropertyInfo prop in props)
    {
        dataTable.Columns.Add(prop.Name, Nullable.GetUnderlyingType(prop.PropertyType) ?? prop.PropertyType);
    }

    foreach (T item in items)
    {
        var values = new object[props.Length];
        for (int i = 0; i < props.Length; i++)
        {
            values[i] = props[i].GetValue(item, null) ?? DBNull.Value;
        }
        dataTable.Rows.Add(values);
    }

    return dataTable;
}

// 通用批量保存方法
public static void BulkSaveData<T>(IEnumerable<T> items, string tableName)
{
    using (SqlBulkCopy copy = new SqlBulkCopy(ConnectionString))
    {
        copy.DestinationTableName = tableName;
        copy.WriteToServer(ToDataTable(items));
    }
}

// SignUp流程调用（单条也可用，批量场景更划算）
public static void SignUp(AccountSignUp model)
{
    DataAccess.BulkSaveData(new List<AccountSignUp> { model }, "UserAccounts");
}

方案3：使用ORM框架（长期最优选择） #

如果项目规模较大，推荐使用Entity Framework Core、Dapper等ORM框架，它们能直接将模型映射到数据库表，无需手动处理DataTable或SQL语句，大幅减少重复代码，兼顾性能与安全性：
比如用Dapper实现单条插入：

public static void SignUp(AccountSignUp model)
{
    using (SqlConnection conn = new SqlConnection(ConnectionString))
    {
        string sql = @"INSERT INTO UserAccounts (Username, FirstName, LastName, EmailAddress, PasswordHash)
                       VALUES (@UserName, @FirstName, @LastName, @EmailAddress, @Password)";
        conn.Execute(sql, model);
    }
}

总结 #

• 单条数据写入：优先用参数化SQL或ORM，避免SqlBulkCopy+DataTable的组合，减少不必要开销。
• 批量数据写入：封装通用模型转DataTable方法配合SqlBulkCopy，或使用ORM的批量插入功能。
• 避免手动硬编码DataTable列和字段映射，减少维护成本和出错概率。

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