c# 导致 mysql = System.Collections.Generic.List

Question

在表“skupina”中，我们有值“pc1,pc2,pc3,...”。但在这段代码中，我得到了 mysql System.Collections.Generic.List 的结果，但应该是 "pc1,pc2,pc3,..."。我的错误在哪里？

private void button1_Click(object sender, EventArgs e)
{
    using (MySqlConnection cnn = new MySqlConnection("Server=10.7.18.35;Database=OitDB;Uid=martin;Pwd=;"))
    {
        MySqlDataAdapter da = new MySqlDataAdapter("SELECT namepc FROM skupina where nazovskup= 'mojask' ", cnn); 
        DataSet ds = new DataSet();
        da.Fill(ds, "skupina");

        List<string> skName = new List<string>();
        foreach (DataRow row in ds.Tables["skupina"].Rows)
        {
            skName.Add(row["namepc"].ToString());

            string constring = "Server=10.7.18.35;Database=OitDB;Uid=martin;Pwd=;";
            var Query = "INSERT INTO OitDB.skup(uzivatel)VALUES('" + skName + "')";
            MySqlConnection conDatabase = new MySqlConnection(constring);
            MySqlCommand cmdDatabase = new MySqlCommand(Query, conDatabase);
            MySqlDataReader myReader;

            try
            {
                conDatabase.Open();
                myReader = cmdDatabase.ExecuteReader();
                MessageBox.Show("Správa odoslaná!");
                while (myReader.Read())
                {
                }
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }
        }           
    }
}

here is error

Answer 1

您正在隐式调用 ToString()在 List<T> 上.这并没有达到您的预期，因为 List<T>不会覆盖 ToString() .你可以使用

// Note: the ToArray() call is only required if you're targeting .NET 3.5, which
// is massively out of date, and really shouldn't be used IMO.  
string.Join(", ", skName.ToArray())

得到一个逗号分隔的结果...但你真的不应该在你目前正在做的地方做它。相反，您应该使用参数化 SQL，例如

var query = "INSERT INTO OitDB.skup (uzivatel) VALUES (@name)";
...
cmdDatabase.Parameters.Add("@name", MySqlDbType.VarChar).Value
    = string.Join(", ", skName.ToArray());

始终，始终对值使用参数化 SQL。您应该仅为非值部分动态构建 SQL - 在这种情况下，您需要非常非常小心。

我还建议使用 using命令和数据读取器的指令...目前您打开了两个不同的连接，这似乎是不必要的...