dataGridView数据插入数据库问题

[星星点灯灯]

向数据库插入数据，出错。其他地方也调用了执行sql的方法，没问题。帮我看看是不是这里的sql语句有问题，急用啊



急用，最好私信一下qq之类的联系方式便于沟通，谢谢大家啦

[bdmh]

这错误是你自己throw出来的

[wind_cloud2011]

  private void button3_Click(object sender, EventArgs e)
        {          
            for (int i = 0; i < this.dataGridView1.Rows.Count - 1; i++)
            {
                string a = dataGridView1.Rows[i].Cells[0].Value.ToString();
                string b = dataGridView1.Rows[i].Cells[1].Value.ToString();
                string c = dataGridView1.Rows[i].Cells[2].Value.ToString();
                insertdata(a,b,c);
            }


        }

        public void insertdata(string s1, string s2, string s3)//插入数据库  
        {
            SqlConnection con = new SqlConnection("Data Source=10.168.1.5;Initial Catalog=data;User ID=sa;password=sa;Integrated Security=False");
            con.Open();
            SqlCommand cmd = new SqlCommand(string.Format("select Count(*) from newtable where a= '{0}'", s1), con);
            if ((int)cmd.ExecuteScalar() > 0)
            {
               listBox1.Items.Add(s1 + " 数据已经存在");
            }
            else
            {
                string sql = "insert into SC(Sno,Cno,score) values('" + s1 + "','" + s2 + "','" + s3 + "')";
                cmd.CommandText = sql;
                cmd.ExecuteNonQuery();
               listBox1.Items.Add(s1 + " 成功添加");
            }
            cmd.Dispose();
            con.Close();
        }  

[wind_cloud2011]

你要sql语句是insert没有问题，opeatedata这里直接就throw,这里应该调用ＳＱＬ执行语句吧

[星星点灯灯]

引用 1 楼 bdmh 的回复:

这错误是你自己throw出来的

那个方法是自己生成的。其他地方也用了执行sql的这个方法，但是不抛异常。这里为什么会抛啊

[星星点灯灯]

[quote=引用 2 楼 wind_cloud2011 的回复:] 我需要多次用到插入的方法，每次列的数量不定。这样写太固定不好用啊

[星星点灯灯]

[於黾]

sql语句values后面没有反括号？ 下次再遇到这种问题，自己断点调试一下，把拼接后的SQL语句拿出来放数据库里执行一下就什么都明白了。 每次SQL拼接有问题就找人来看也不是办法啊

[星星点灯灯]

引用 7 楼 Z65443344 的回复:

sql语句values后面没有反括号？

下次再遇到这种问题，自己断点调试一下，把拼接后的SQL语句拿出来放数据库里执行一下就什么都明白了。

每次SQL拼接有问题就找人来看也不是办法啊

拼接的sql在数据库里面可以正常插入，但在程序里还是抛这个异常


我写了个类，放了这个方法


目的是把dataGridView中的数据插入数据库，每次插入几列不一定。如果这个方法不行的话，麻烦告诉我怎么实现
谢谢大家啦，还没法往下做。。

[lovesun3]

额，要不你把要存的数据 弄个DataSet 然后 写个存储过程，把ds 当XML参数传进去。。省得一条一条插入了。。 比如这种

     
   public void InsertBudgetFee(DataTable dt, int index, Database db, DbTransaction transaction)
        {
            DataSet ds = new DataSet();
            ds.Tables.Add(dt);
            string proc = "";
            switch (index)
            {
                case 1:
                    proc = "PA.InsertMainBudget";
                    break;
                case 2:
                    proc = "PA.InsertOtherBudget";
                    break;
            }
          
            DbCommand cmd = db.GetStoredProcCommand(proc);
            db.AddInParameter(cmd, "pHeadId", DbType.String, KeyValue);
            db.AddInParameter(cmd, "pXml", DbType.Xml, ds.GetXml());
            db.AddInParameter(cmd, "pUserId", DbType.Int32, CurrentUser.UserID);
            db.ExecuteNonQuery(cmd, transaction);



        }

[於黾]

你数据库连接串没给进去啊. 程序怎么知道你想连哪个数据库? conn在new的时候将数据库连接字符串带进去

[星星点灯灯]

妈蛋，是单词写错了。。