求一段关于绑定数据的C#代码

[yct0605]

自定义一个类用于查询数据显示在gridview里面，由于需要多次查询想写个公共的类来一起调用：

public DataTable ReadCDType(表名, 字段,初值)
{
   sql = "select * from 表明 where 字段=初值";     
   return objCDMng.ExecuteDataSet(sql).Tables[0];
}

请问一下这个思路是不是正确的，如果正确怎么写这段代码？请指教，多谢了！如果有更好的方法欢迎赐教！

[zibaozhangyu]

一个非常没有意义的公用代码，还不如直接写。。。。。。你这个根本起不到公用的目的

[cxudong]

 //公有方法，根据Sql语句，返回一个结果数据集
        public DataSet GetDataSetSql(string XSqlString)
        {
            Open();
            SqlDataAdapter Adapter = new SqlDataAdapter(XSqlString, Connection);
            DataSet Ds = new DataSet();
            Adapter.Fill(Ds);
            Close();
            return Ds;
        }

[wuyq11]

不要通过方法传值，定义属性，通过属性传值。或用工厂类看看petshop

[yct0605]

[Quote=引用 4 楼 lovvver 的回复:]
这样的思路不好，原因是UI和业务逻辑层以及数据库层严重耦合在一起了，也就是如果数据库层发生了变动，那么每一层都要面临改动。
你要使用DataTable也没关系，你可以考虑尽量降低每个层之间的耦合，尽量避免使用DataTable的列名。
[/Quote]
多谢了，由于是初学，只是有这个想法，还没有实践过，谢谢大家了！

[我在深圳搬砖-Justin]

建议 实践一下就知道 对错...

[我在深圳搬砖-Justin]

思路争正确

[surlew]

public DataTable ReadCDType(表名, 字段,初值)
{
   sql = "select * from "+表明+" where "+字段+"='"+初值+"'";     
   return objCDMng.ExecuteDataSet(sql).Tables[0];
}

如上

[lovvver]

这样的思路不好，原因是UI和业务逻辑层以及数据库层严重耦合在一起了，也就是如果数据库层发生了变动，那么每一层都要面临改动。
你要使用DataTable也没关系，你可以考虑尽量降低每个层之间的耦合，尽量避免使用DataTable的列名。