连接数据库高手们来！

Question

这个代码　功能上应该没问题的，主要是　代码复用性不强，应该把数据库操作这一块抽出来做一个共通类

Answer 1

同上
得多看看设计模式的相关书

Answer 2

应该将参数用Parameter传给Command,不然会存在安全隐患，另外，你的代码是要验证用户名和密码吧，你验证密码的代码呢？        void select(string name, string paw)
        {
            string connectionString = "Data Source=PC-200808301549;Initial Catalog=test1;Integrated Security=True";
            IDbConnection connection = new SqlConnection(connectionString);
            string queryString = "select * from [testta] where ([name] = @name and [password] = @password)";
            IDbCommand command = new SqlCommand();            command.Connection = connection;
            command.CommandText = queryString;            IDbDataParameter parameterName = new SqlParameter();
            parameterName.ParameterName = "@name";
            parameterName.Value = name;
            parameterName.DbType = DbType.String;
            command.Parameters.Add(parameterName);            IDbDataParameter parameterPassword = new SqlParameter();
            parameterPassword.ParameterName = "@password";
            parameterPassword.Value = paw;
            parameterPassword.DbType = DbType.String;
            command.Parameters.Add(parameterPassword);            connection.Open();
            int i = command.ExecuteNonQuery();
            conn.Close();            //这部分代码最好删除掉,将函数的返回值类型改为bool型,提示信息由调用时给出
            if (i != 0)
            {
                MessageBox.Show("登临成功", "提示", MessageBoxButtons.OK);
            }
        } 

Answer 3

楼上说的非常好。。支持