如何用C#在SQL指定列插入当前时间？lastlogin列插入失败求助

嘿，我帮你排查下为啥登录时间插不进数据库，顺便给你讲讲正确的做法～

先说说你代码里的问题 #

• 类型不匹配：你把DateTime转成字符串传给SQL参数，但SQL的datetime类型本来就该直接接DateTime对象，转成字符串不仅多此一举，还容易因为系统日期格式差异导致解析失败，这大概率是报错的主要原因。
• 执行顺序搞反了：你先关窗口、开主窗口、弹提示，之后才执行数据库插入——万一当前窗口关闭后，连接或者命令对象被释放了，那插入操作肯定跑不起来啊。
• 可能漏了开连接：你代码里没写cnn.Open()，如果数据库连接没打开，ExecuteNonQuery()直接就报错了。

给你改好的代码示例 #

咱们把顺序调过来，先把数据库操作搞定，再跳界面，同时正确传递参数：

try
{
    // 直接拿当前时间，不用手动格式化
    DateTime currentTime = DateTime.Now;

    // 先确保数据库连接是打开的
    if (cnn.State != System.Data.ConnectionState.Open)
    {
        cnn.Open();
    }

    // 用using包裹命令，自动释放资源
    using (SqlCommand command = new SqlCommand("INSERT INTO dbo.ValidationApp(lastlogin) VALUES(@dtime)", cnn))
    {
        // 直接传DateTime对象，指定参数类型更稳妥
        command.Parameters.Add("@dtime", System.Data.SqlDbType.DateTime).Value = currentTime;
        
        // 执行插入，看看有没有影响行数
        int affectedRows = command.ExecuteNonQuery();
        if (affectedRows > 0)
        {
            MessageBox.Show("Welcome!", "Validation App", MessageBoxButtons.OK, MessageBoxIcon.Information);
            // 插成功了再跳界面
            this.Close();
            Mainform mainform = new Mainform();
            mainform.Show();
        }
        else
        {
            MessageBox.Show("登录时间记录失败啦", "提示", MessageBoxButtons.OK, MessageBoxIcon.Warning);
        }
    }
}
catch (Exception ex)
{
    // 捕获异常，方便排查具体问题
    MessageBox.Show($"出错啦：{ex.Message}", "错误", MessageBoxButtons.OK, MessageBoxIcon.Error);
}
finally
{
    // 用完记得关连接，避免资源浪费
    if (cnn.State == System.Data.ConnectionState.Open)
    {
        cnn.Close();
    }
}

两种正确插入当前时间到SQL列的方法 #

方法1：C#传DateTime对象（推荐） #

直接把DateTime.Now或者DateTime.UtcNow（更推荐UTC时间，避免时区坑）传给参数，ADO.NET会自动处理类型转换，完全不用自己格式化：

command.Parameters.Add("@dtime", SqlDbType.DateTime).Value = DateTime.UtcNow;

方法2：让SQL自己生成时间 #

如果不需要在C#里处理时间，直接用SQL Server的内置函数，这样还能避免客户端和服务器时间不一致的问题：

// 用GETDATE()获取服务器当前时间
SqlCommand command = new SqlCommand("INSERT INTO dbo.ValidationApp(lastlogin) VALUES(GETDATE())", cnn);
// 要是需要更高精度的时间，就用SYSDATETIME()
// SqlCommand command = new SqlCommand("INSERT INTO dbo.ValidationApp(lastlogin) VALUES(SYSDATETIME())", cnn);
command.ExecuteNonQuery();

最后再提醒你几点 #

• 确认数据库里lastlogin列的类型是datetime、datetime2这类时间类型，别用varchar存时间，不然以后查时间范围、排序都麻烦。
• 尽量用using包裹SqlConnection和SqlCommand，确保资源能自动释放，不会内存泄漏。
• 一定要加异常捕获，这样出错的时候能直接看到具体原因，比如是不是权限不够、列名拼错了之类的。

内容的提问来源于stack exchange，提问作者John Rommel Unsana