黑马程序员技术交流社区

标题: 求助，哪里错了 [打印本页]

作者: 卢京 时间: 2012-9-28 13:19
标题: 求助，哪里错了

选择省时报异常

可是我明明传了参数啊！！！

作者: 许庭洲 时间: 2012-9-28 13:49
首先要确定sql语句含有@proid

作者: 卢京 时间: 2012-9-28 19:44

许庭洲发表于 2012-9-28 13:49
首先要确定sql语句含有@proid

有的啊

作者: 李阳Leo 时间: 2012-9-28 20:22
问题在 DBHelper 中，发来瞧瞧

作者: 卢京 时间: 2012-9-28 22:18

李阳_TickTock 发表于 2012-9-28 20:22
问题在 DBHelper 中，发来瞧瞧

  public static SqlDataReader ExecuteReader(string sql, params SqlParameter[] param)
      {
         SqlConnection conn = new SqlConnection(connString);

         SqlCommand cmd = new SqlCommand(sql, conn);

         if (param != null && param.Length > 0)
         {
            cmd.Parameters.AddRange(param);
         }
         conn.Open();
         SqlDataReader dr = cmd.ExecuteReader(CommandBehavior.CloseConnection);
         return dr;
      }

      public static SqlDataReader ExecuteReader(string sql)
      {
         return ExecuteReader(sql, null);
      }

作者: 李阳Leo 时间: 2012-9-28 22:33

卢京发表于 2012-9-28 22:18
public static SqlDataReader ExecuteReader(string sql, params SqlParameter[] param)
{
...

加参数的方法好像错了，没用过这个方法，待会查查

改成这样试试

if (param.Length > 0)
{
for(int i = 0;i<param.length;i++){
//cmd.Parameters.AddRange(param);
cmd.Parameters.add(param[i]);
}
}

复制代码

作者: 卢京 时间: 2012-9-28 22:57

李阳_TickTock 发表于 2012-9-28 22:33
加参数的方法好像错了，没用过这个方法，待会查查

改成这样试试

还是不行，相同的报错原因

作者: 晋志强 时间: 2012-9-29 00:03
public static int ExecuteCommand(string sql, params SqlParameter[] p)
      {
         SqlCommand cmd = new SqlCommand(sql, Connection);
         cmd.Parameters.AddRange(p);
         return cmd.ExecuteNonQuery();
      }

作者: 卢京 时间: 2012-9-29 08:57

晋志强发表于 2012-9-29 00:03
public static int ExecuteCommand(string sql, params SqlParameter[] p)
{
SqlComm ...

这个不对吧，我要读出一行数据，你返回ExecuteNonQuery，返回的受影响的行数

作者: 卢京 时间: 2012-9-29 09:03

李阳_TickTock 发表于 2012-9-28 22:33
加参数的方法好像错了，没用过这个方法，待会查查

改成这样试试

我昨天改了一下，首先原因出在cboProvince.SelectValue返回的是object类型，而PronviceId要的是int类型，但是我把object转换成int类型，还是不行，最后我吧SelectedValue转换成SelectedIndex就可以了，但是SelectedIndex返回的是索引，刚好主键连续，SelectValue和SelectedIndex相同，要是主键不连续，就出问题了，不知道还要怎么解决

作者: 李阳Leo 时间: 2012-9-29 09:15
本帖最后由李阳_TickTock 于 2012-9-29 09:23 编辑

卢京发表于 2012-9-29 09:03
我昨天改了一下，首先原因出在cboProvince.SelectValue返回的是object类型，而PronviceId要的是int类型， ...

老师一再强调过，主键不能有意义，所以这种做法其实也是错的

既然找到问题原因，那既然SelectedIndex可以传一个整形进去，SelectedValue转成整形一定也可以传进去。

说到这我好像想起来了，这个项目我也做过，那个ComboBox有两个值，一个是items，一个是value，一个item对应一个value，你在初始化的时候，应该把省的名字赋给item，然后把省的id赋给value。这样取值的时候，就可以通过value取出省的id，不知道你是不是这么做的？

我是这么赋值的

//获取并设置省
string sql = null;
DataSet ds = null;
sql = "select id,ShengName from T_Sheng";
ds = ConnectSQLServer(sql);
cmbSheng.DataSource = ds.Tables[0];
cmbSheng.DisplayMember = "ShengName";
cmbSheng.ValueMember = "id";

复制代码

之后获取省的ID，在数据库中搜索对应的市并赋值

/// <summary>
/// 获取并设置市和区号
/// </summary>
void showShi() {
string sql = null;
DataSet ds = null;
sql = "select Id,ShiName,QuHao from T_Shi where ShengID =" + cmbSheng.SelectedValue;
ds = ConnectSQLServer(sql);
cmbShi.DataSource = ds.Tables[0];
cmbShi.DisplayMember = "ShiName";
cmbShi.ValueMember = "id";

复制代码

当然，我没有用那种传参数的方法防注入，那时候做这项目的时候也没注意这点。但是我觉得既然selectedIndex可以传整形进去，只要selectedValue搜出来的真的是个整形，那么把它转成整形传进去也应该没有问题，你可以设断点看一下，selectedValue里面返回的值到底是什么。

说错了不是selected，搞成HTML了，是ComboBox控件，就是你的cbo什么什么，我命名不规范，写成cmb了呵呵

刚看了你的代码，你好像也跟我一样是这么赋值的，我待会也转成参数瞧瞧，可能哪个环节出错了

作者: 卢京 时间: 2012-9-29 10:39
本帖最后由卢京于 2012-9-29 10:40 编辑

李阳_TickTock 发表于 2012-9-29 09:15
老师一再强调过，主键不能有意义，所以这种做法其实也是错的

既然找到问题原因，那既然SelectedIndex可以 ...

我就是这么做的，而且我用了两种方法在做这个省市县三级联动，当然最简单的就是和你写的一样，用DataSet，这个我做出来了，没什么问题。后来我想，DataSet不是要把数据库里的数据全部取出来放到DataSet上，要是数据多，打个比方，如果要做州——国家——省——市——县——镇——乡的多级联动（当然有点夸张）就最好用SqlCommand的ExecuteReader方法做，不用把数据全部取出来了，但是就出现了我上面的这个问题

作者: 李阳Leo 时间: 2012-9-29 10:53

卢京发表于 2012-9-29 10:39
我就是这么做的，而且我用了两种方法在做这个省市县三级联动，当然最简单的就是和你写的一样，用DataSet ...

概念错了:L

连接数据库远比全部取出来耗费资源的多，不然dataSet也没有存在的意义了

要追求尽可能少的读取数据库，因为当连上互联网的时候，网速是未知的，每获取一个数据就读一次数据库，而且还不是长连接，每次都要打开关闭连接，用户很快就崩溃了。

作者: 卢京 时间: 2012-9-29 11:47

李阳_TickTock 发表于 2012-9-29 10:53
概念错了

连接数据库远比全部取出来耗费资源的多，不然dataSet也没有存在的意义了

哈哈，既然做了，就把它做对吧，就当是一次练习SqlDataReader也行，现在我还不知道错哪呢

欢迎光临黑马程序员技术交流社区 (http://bbs.itheima.com/)

黑马程序员IT技术论坛 X3.2