Fatal error encountered during command execution.

Question

0.00/5 (No votes)

See more:

I have a Problem during MySQL in C#. I want update data in my MySQL Database.

C#

String MyCon = "SERVER=********** +
            DATABASE=asterisk +
             UID=root +
             PASSWORD=**** + Convert Zero Datetime = True";

private void btnsubmit_Click(object sender, EventArgs e)
        {
            MySqlConnection con = new MySqlConnection(MyCon);
            con.Open();                          
            MySqlCommand cmd = new MySqlCommand(@"Update 
userid, password, role, first_name, last_name, user_group, user_level, active 
from aster_users 
Set password=@password, role=@role, first_name=@first_name, last_name=@last_name,
user_level=@user_level, user_group=@user_group, active=@active 
where userid = userid ", con);
            cmd.ExecuteNonQuery();
            cmd.Parameters.AddWithValue("@pasword",txtconformpassword.Text );
            cmd.Parameters.AddWithValue("@role",cmbrole.Text);
            cmd.Parameters.AddWithValue("@first_name",txtfirstname.Text);
            cmd.Parameters.AddWithValue("@last_name",txtlastname.Text);
            cmd.Parameters.AddWithValue("@user_group",cmbUser1.Text);
            cmd.Parameters.AddWithValue("@user_level",cmbuser.Text);
            cmd.Parameters.AddWithValue("@active",cmbstatus.Text);
            
            MessageBox.Show("updated......");
            con.Close();
        }

help me...

Posted 16-Nov-15 19:16pm

Member 12078840/divya

Updated 16-Nov-15 20:33pm

George Jonsson

v4

Add a Solution

Comments

Amit Jadli 17-Nov-15 1:29am

command execution will come after the value initialization..
so put the cmd.ExecuteNonQuery(); after adding values..like this..

cmd.Parameters.AddWithValue("@pasword",txtconformpassword.Text );
cmd.Parameters.AddWithValue("@role",cmbrole.Text);
cmd.Parameters.AddWithValue("@first_name",txtfirstname.Text);
cmd.Parameters.AddWithValue("@last_name",txtlastname.Text);
cmd.Parameters.AddWithValue("@user_group",cmbUser1.Text);
cmd.Parameters.AddWithValue("@user_level",cmbuser.Text);
cmd.Parameters.AddWithValue("@active",cmbstatus.Text);
cmd.ExecuteNonQuery();

Member 12078840/divya 17-Nov-15 1:37am

thank you for reply now also i got that error . i think mysql is incorrect. i did not know more about sql please check it.please

3 solutions

Solution 2

cut

cmd.ExecuteNonQuery();

and paste it before

MessageBox.Show("updated......");

Posted 16-Nov-15 19:34pm

Arasappan

Solution 3

As you don't say what kind of error you get I assume that this part is causing you problems.

C#

where userid = userid "

You are inserting the literal string "userid", not the value of the variable userid.

C#

string sqlCommand = string.Format(@"UPDATE 
userid, password, role, first_name, last_name, user_group, user_level, active 
FROM aster_users 
SET password=@password, role=@role, first_name=@first_name, last_name=@last_name,
user_level=@user_level, user_group=@user_group, active=@active 
WHERE userid = '{0}';", userid);

// This line can be used to check if the SQL command looks OK.
// It also makes it easy to copy the command into the SQL query tool for testing
Debug.WriteLine(sqlCommand);

MySqlCommand cmd = new MySqlCommand(sqlCommand, con);

It is not dangerous to split a long code chunk into smaller parts. Doing so makes it easier to debug the code.

Personally I find it easier to read an SQL string if I use capital letters for the SQL pre-defined words. This is just my personal taste and there are many opinions about this.

Posted 16-Nov-15 20:44pm

George Jonsson

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Mehdi Gholam · Accepted Answer · 2015-11-16T19:28:00

ExecuteNonQuery() should be the last command:

C#

cmd.Parameters.AddWithValue("@pasword",txtconformpassword.Text );
cmd.Parameters.AddWithValue("@role",cmbrole.Text);
cmd.Parameters.AddWithValue("@first_name",txtfirstname.Text);
cmd.Parameters.AddWithValue("@last_name",txtlastname.Text);
cmd.Parameters.AddWithValue("@user_group",cmbUser1.Text);
cmd.Parameters.AddWithValue("@user_level",cmbuser.Text);
cmd.Parameters.AddWithValue("@active",cmbstatus.Text);

cmd.ExecuteNonQuery();// <---- moved to after the parameters