帮我改一下修改密码的代码，在线等候

 private void button1_Click(object sender, EventArgs e)
        {
            string User_ID = textBox1.Text;
            string User_PWD = textBox2.Text;
            string a = textBox3.Text;
            int count = 0;
            try
            {
                string ConnectionString = "Data Source=fly;user=fly;password=fly";//写连接串    
                OracleConnection conn = new OracleConnection(ConnectionString);//创建一个新连接   

                conn.Open();
                OracleCommand cmd = conn.CreateCommand();
                cmd.CommandText = "select * from alarm_user where user_ID='" + User_ID + "'";//在这儿写sql语句    
                OracleDataReader odr = cmd.ExecuteReader();//创建一个OracleDateReader对象
                string pwd = "";
                while (odr.Read())//读取数据，如果odr.Read()返回为false的话，就说明到记录集的尾部了    
                {
                    count++;
                    pwd = odr.GetOracleString(1).ToString();
                    //MessageBox.Show(odr.GetOracleString(1).ToString());//输出字段1，这个数是字段索引，具体怎么使用字段名还有待研究    
                }
                odr.Close();
                cmd.Clone();
                conn.Close();
                  
                if (count == 0)
                {
                    MessageBox.Show("没有此用户");
                    textBox2.Text = "";
                    textBox1.Text = "";
                    this.textBox1.Focus();

                }
                else
                {
                    if (User_PWD.Equals(pwd))
                    {
                        string ConnectionString = "Data Source=fly;user=fly;password=fly";//写连接串    
                        OracleConnection conn1 = new OracleConnection(ConnectionString);//创建一个新连接   

                        conn1.Open();
                        OracleCommand cmd1 = conn.CreateCommand();
                        cmd1.CommandText = "update alarm_user set user_pwd='" + a + "'";//在这儿写sql语句 
                        OracleDataReader odr1 = cmd.ExecuteReader();//创建一个OracleDateReader对象
                        cmd1.Clone();
                        odr1.Close();
                        conn1.Close();
                        MessageBox.Show("修改成功！！");
                        textBox1.Text = "";
                        textBox2.Text = "";
                        textBox3.Text = "";
                    }

               
                    else
                    {
                        MessageBox.Show("密码错误，请重新输入!");
                        textBox2.Text = "";
                        textBox3.Text = "";
                        this.textBox1.Focus();
                    }

                    

                
                }
            }
            catch (Exception ee)
            {
                MessageBox.Show(ee.Message.ToString());
            }    
        }

就是一个用户修改密码的代码，输入账号，旧密码和新密码后实现把oracle里的数据库里的密码给改掉，这个代码是有问题的，我不知道应该怎么改了，请大家帮下忙了

29 个解决方案

#1

自己顶起....

#2

  cmd1.CommandText = "update alarm_user set user_pwd='" + a + "'";//
少条件啊

#3

cmd1.CommandText = "update alarm_user set user_pwd='" + a + "'"+ " where user_ID="+ '" + User_ID + "'";//

#4

后面加的这个条件是什么意思

#5

加的这个我明白了，可是现在问题是我运行时它提示我这个if语句里的
if (User_PWD.Equals(pwd))
                    {
                        string ConnectionString = "Data Source=fly;user=fly;password=fly";//写连接串    
                        OracleConnection conn1 = new OracleConnection(ConnectionString);//创建一个新连接   

                        conn1.Open();
                        OracleCommand cmd1 = conn.CreateCommand();
                        cmd1.CommandText = "update alarm_user set user_pwd='" + a + "'";//在这儿写sql语句 
                        OracleDataReader odr1 = cmd.ExecuteReader();//创建一个OracleDateReader对象
                        cmd1.Clone();
                        odr1.Close();
                        conn1.Close();
                        MessageBox.Show("修改成功！！");
                        textBox1.Text = "";
                        textBox2.Text = "";
                        textBox3.Text = "";
                    }


里的这句 string ConnectionString = "Data Source=fly;user=fly;password=fly";//写连接串 
中的connectionstring有问题，在父级已经表示其它内容了

#6

没人知道了么，帮看看...

#7

在别处定义过ConnectionString 变量啊

#8

对，注意看我上面的代码，在程序一开始我就定义了一个，然后用完了我给关了，然后再后面的一个if语句里我还要打开数据库一次，可是这第二次打开就不让我定义这一个变量了，我应该怎么改？》

#9

 if (User_PWD.Equals(pwd))
                    {
                        string ConnectionString = "Data Source=fly;user=fly;password=fly";//写连接串

把   string ConnectionString = "Data Source=fly;user=fly;password=fly";
去掉

#10

去掉就错了啊，因为之前的打开数据库在我用完之后就给关了，现在要再次打开数据库把那个去掉了不就打不开了么

#11

真晕，
string ConnectionString = "Data Source=fly;user=fly;password=fly";//写连接串    
 OracleConnection conn1 = new OracleConnection(ConnectionString);//创建一个新连接   


ConnectionString 只是一个string变量，已经赋好值了，和你打开关闭数据库有什么关系

#12

写update 语句时，要注意一定要加上where条件，标出ID的编号，然后用一个整形初始值变量为零，试试是否被改变，如果改变为一的话，说明修改成功，如果为零的话，再去看ID对应抽出的外键表ID是否正确，总之，如果为零的话，则修改未成功，抛出自定义异常提示！

#13

引用 11 楼 yutian_01261027 的回复:

真晕，
string ConnectionString = "Data Source=fly;user=fly;password=fly";//写连接串   
OracleConnection conn1 = new OracleConnection(ConnectionString);//创建一个新连接 


ConnectionString 只是一个string变量，已经赋好值了，和你打开关闭数据库有什么关系

你的意思是我只要把string ConnectionString = "Data Source=fly;user=fly;password=fly";//写连接串  这一句去掉了可以了呗

#14

大家能否在我题目所给的代码里告诉我哪里需要修改，这样直观一点...

#15

把下面的string ConnectionString = "Data Source=fly;user=fly;password=fly"去掉
你定义多了，程序肯定不能通过

#16

我去掉了，可是当更改时就弹出一个对话框，说是无效操作，连接被关闭，这是怎么回事

#17

没有人能再帮着解释一下了么？

#18

if (User_PWD.Equals(pwd))
                    {
                        string ConnectionString = "Data Source=fly;user=fly;password=fly";//写连接串    
                        OracleConnection conn1 = new OracleConnection(ConnectionString);//创建一个新连接   
if下面的这两句去掉...还有在updata语句里面你应该加个条件..至少要知道你要更改的是哪个用户的密码吧..

#19

没防SQL注入

#20

引用 18 楼 duanfengo7 的回复:

if (User_PWD.Equals(pwd))
                    {
                        string ConnectionString = "Data Source=fly;user=fly;password=fly";//写连接串   
                        OracleConnection conn1 = new OracleConnection(ConnectionString);//创建一个新连接 
if下面的这两句去掉...还有在updata语句里面你应该加个条件..至少要知道你要更改的是哪个用户的密码吧..

条件我已经加完了，是按照3楼说的那样加的，我按照你说的把那两个都去掉了，可还是提示无效操作，连接已关闭，我在前面已经有odr.Close();
                cmd.Clone();
                conn.Close();
了

#21

引用 19 楼 newuser2008 的回复:

没防SQL注入

没明白，能具体讲一下么？

#22

网上找一下SQl注入

#23

至于string ConnectionString = "Data Source=fly;user=fly;password=fly"; 
这个问题，建议你在最开始就Public string ConnectionString = "Data Source=fly;user=fly;password=fly"; 这样会少很多麻烦

#24

代码很乱，不过先改一处看看，先把你下面所有代码里的连接字符串改为1

if (User_PWD.Equals(pwd))
                    {
                        string  ConnectionString1 = "Data Source=fly;user=fly;password=fly";//写连接串    
                        OracleConnection conn1 = new OracleConnection( ConnectionString1);//创建一个新连接   

                        conn1.Open();
                        OracleCommand cmd1 = conn.CreateCommand();
                        cmd1.CommandText = "update alarm_user set user_pwd='" + a + "'";//在这儿写sql语句 
                        OracleDataReader odr1 = cmd.ExecuteReader();//创建一个OracleDateReader对象
                        cmd1.Clone();
                        odr1.Close();
                        conn1.Close();
                        MessageBox.Show("修改成功！！");
                        textBox1.Text = "";
                        textBox2.Text = "";
                        textBox3.Text = "";
                    }

#25

        private void button1_Click(object sender, EventArgs e)
        {
            string User_ID = textBox1.Text;
            string User_PWD = textBox2.Text;
            string a = textBox3.Text;
            int count = 0;
            try
            {
                string ConnectionString = "Data Source=fly;user=fly;password=fly";//写连接串    
                OracleConnection conn = new OracleConnection(ConnectionString);//创建一个新连接   

                conn.Open();
                OracleCommand cmd = conn.CreateCommand();
                cmd.CommandText = "select * from alarm_user where user_ID='" + User_ID + "'";//在这儿写sql语句    
                OracleDataReader odr = cmd.ExecuteReader();//创建一个OracleDateReader对象
                string pwd = "";
                while (odr.Read())//读取数据，如果odr.Read()返回为false的话，就说明到记录集的尾部了    
                {
                    count++;
                    pwd = odr.GetOracleString(1).ToString();
                    //MessageBox.Show(odr.GetOracleString(1).ToString());//输出字段1，这个数是字段索引，具体怎么使用字段名还有待研究    
                }
                odr.Close();
                cmd.Clone();
                conn.Close();
                  
                if (count == 0)
                {
                    MessageBox.Show("没有此用户");
                    textBox2.Text = "";
                    textBox1.Text = "";
                    this.textBox1.Focus();

                }
                else
                {
                    if (User_PWD.Equals(pwd))
                    {                        

                        conn.Open();
                        OracleCommand cmd = conn.CreateCommand();
                        cmd.CommandText = "update alarm_user set user_pwd='" + a + "' where User_ID ='"+User_ID+"'";//在这儿写sql语句 .不知道你的用户表里面的字段是不是这个.
                       // OracleDataReader odr1 = cmd.ExecuteReader();//创建一个OracleDateReader对象
                        int i = cmd.ExecuteNonQuery();
                        cmd.Clone();
                        odr.Close();
                        conn.Close();
                        if(i > 0)
                        { 
                          MessageBox.Show("修改成功！！");
                          textBox1.Text = "";
                          textBox2.Text = "";
                          textBox3.Text = "";
                        }
                    }

               
                    else
                    {
                        MessageBox.Show("密码错误，请重新输入!");
                        textBox2.Text = "";
                        textBox3.Text = "";
                        this.textBox1.Focus();
                    }

                    

                
                }
            }
            catch (Exception ee)
            {
                MessageBox.Show(ee.Message.ToString());
            }    
        }

#26

编译通不过。。。。。

#27

引用 24 楼 libinguest 的回复:

代码很乱，不过先改一处看看，先把你下面所有代码里的连接字符串改为1

if (User_PWD.Equals(pwd))
                    {
                        stringConnectionString1 = "Data Source=fly;user=fly;password=fly";//写连接串   
                        OracleConnection conn1 = new OracleConnection(ConnectionString1);//创建一个新连接 

                        conn1.Open();
                        OracleCommand cmd1 = conn.CreateCommand();
                        cmd1.CommandText = "update alarm_user set user_pwd='" + a + "'";//在这儿写sql语句
                        OracleDataReader odr1 = cmd.ExecuteReader();//创建一个OracleDateReader对象
                        cmd1.Clone();
                        odr1.Close();
                        conn1.Close();
                        MessageBox.Show("修改成功！！");
                        textBox1.Text = "";
                        textBox2.Text = "";
                        textBox3.Text = "";
                    }

这个方法我试过了，虽然能正常运行，也能提示修改成功，可是在数据库里的信息却并没有被修改

#28

25楼正解，已解决...

#29

不过要修改一下

 private void button1_Click(object sender, EventArgs e)
        {
            string User_ID = textBox1.Text;
            string User_PWD = textBox2.Text;
            string a = textBox3.Text;
            int count = 0;
            try
            {
                string ConnectionString = "Data Source=fly;user=fly;password=fly";//写连接串    
                OracleConnection conn = new OracleConnection(ConnectionString);//创建一个新连接   

                conn.Open();
                OracleCommand cmd = conn.CreateCommand();
                cmd.CommandText = "select * from alarm_user where user_ID='" + User_ID + "'";//在这儿写sql语句    
                OracleDataReader odr = cmd.ExecuteReader();//创建一个OracleDateReader对象
                string pwd = "";
                while (odr.Read())//读取数据，如果odr.Read()返回为false的话，就说明到记录集的尾部了    
                {
                    count++;
                    pwd = odr.GetOracleString(1).ToString();
                    //MessageBox.Show(odr.GetOracleString(1).ToString());//输出字段1，这个数是字段索引，具体怎么使用字段名还有待研究    
                }
                odr.Close();
                cmd.Clone();
                conn.Close();
                  
                if (count == 0)
                {
                    MessageBox.Show("没有此用户");
                    textBox2.Text = "";
                    textBox1.Text = "";
                    this.textBox1.Focus();

                }
                else
                {
                    if (User_PWD.Equals(pwd))
                    {                        

                        conn.Open();
                        OracleCommand cmd1 = conn.CreateCommand();
                        cmd1.CommandText = "update alarm_user set user_pwd='" + a + "' where User_ID ='"+User_ID+"'";//在这儿写sql语句 .不知道你的用户表里面的字段是不是这个.
                       // OracleDataReader odr1 = cmd1.ExecuteReader();//创建一个OracleDateReader对象
                        int i = cmd1.ExecuteNonQuery();
                        cmd1.Clone();
                        odr.Close();
                        conn.Close();
                        if(i > 0)
                        { 
                          MessageBox.Show("修改成功！！");
                          textBox1.Text = "";
                          textBox2.Text = "";
                          textBox3.Text = "";
                        }
                    }

               
                    else
                    {
                        MessageBox.Show("密码错误，请重新输入!");
                        textBox2.Text = "";
                        textBox3.Text = "";
                        this.textBox1.Focus();
                    }

                    

                
                }
            }
            catch (Exception ee)
            {
                MessageBox.Show(ee.Message.ToString());
            }    
        }