Question

为什么此代码会抛出错误？

using (MySqlConnection cn = new MySqlConnection(VarribleKeeper.MySQLConnectionString))
{
    {
        MySqlCommand Command = new MySqlCommand();
        Command.Connection = cn;

        Command.CommandText = "UPDATE TeleworksStats SET Ja= ('" + JaTak +
            "') WHERE Brugernavn = " + VarribleKeeper.Brugernavn + "' AND Dato = " +
            DateTime.Today.ToString("yyyy-MM-dd") + "";
        cn.Open();
        Command.ExecuteNonQuery();
        //Ryd op
        Command.Dispose();
        cn.Close();
    }
}

Answer 1

您在'和Brugernavn =后错过了一个引用Dato：

Brugernavn = "... '" + VarribleKeeper.Brugernavn + "' AND Dato = '" +
             DateTime.Today.ToString("yyyy-MM-dd") + "'";

另外，我强烈建议您始终使用parameterized queries来避免SQL Injection这样：

Command.CommandText =
     "UPDATE TeleworksStats SET Ja = @Ja WHERE Brugernavn = @Brugernavn and ...";
Command.Parameters.AddWithValue("@Ja", JaTak);
Command.Parameters.AddWithValue("@Brugernavn",  VarribleKeeper.Brugernavn);

虽然直接指定类型并且使用Value属性比AddWithValue更好。检查一下：Can we stop using AddWithValue() already?

Answer 2

我认为您还有更多需要注意的事项，而不是忘记'列Brugernavn列的值和Dato列的单引号，我认为您还需要记住更多内容。

使用using statement按照您的连接方式处理Command对象，而不是手动调用Close或Dispose方法。
使用paramterized queries代替字符串连接。这类代码适用于SQL Injection攻击。
您似乎尝试使用字符串表示法保存DateTime值。 不要这样做！如果您想将DateTime值保留在数据库中，则需要直接传递。将您的Dato列更改为DateTime type。阅读：Bad habits to kick : choosing the wrong data type

using(var cn = new MySqlConnection(VarribleKeeper.MySQLConnectionString))
using(var Command = cn.CreateCommand())
{
   Command.CommandText = @"UPDATE TeleworksStats SET Ja = @Ja 
                           WHERE Brugernavn = @Brugernavn  AND Dato = @Dato";
   Command.Parameters.Add("@Ja", MySqlDbType.VarChar).Value = JaTak;
   Command.Parameters.Add("@Ja", MySqlDbType.VarChar).Value = VarribleKeeper.Brugernavn;
   Command.Parameters.Add("@Ja", MySqlDbType.DateTime).Value = DateTime.Today;
   // I assumed your column types. You should write proper column types instead.

   cn.Open();
   Command.ExecuteNonQuery();
}

Where子句抛出错误

2 个答案: