How to fix update error?

Black Diamond

Hi everyone,

I hava a code to update sql table, but when i look at the table nothings happen its empty this is my code:

Vote = count.ToString();
conn.Open();
CountUpdate.Parameters.Add(SqlCommand CountUpdate = new SqlCommand("UPDATE FinalVotes SET Name='"+btnCandidates1.Text+"' where Votes='"+Vote+"'", conn);
CountUpdate.Parameters.Add("@Name", SqlDbType.VarChar,100).Value=DBNull.Value;"@Votes", SqlDbType.VarChar,100).Value =DBNull .Value;

CountUpdate.ExecuteNonQuery();

conn.Close();

please help me out this problem...thanks

5 Replies

Know the answer? Post it — somebody with the same question will find it here.

Vulpes15y ago
Accepted answer
The code is a jumble (probably pasted in from VS) but what I suspect you need is this:

Vote = count.ToString();
conn.Open();
SqlCommand CountUpdate = new SqlCommand("UPDATE FinalVotes SET Name = @Name where Votes = @Votes", conn);
CountUpdate.Parameters.Add("@Name", SqlDbType.VarChar,100).Value = btnCandidates1.Text;
CountUpdate.Parameters.Add("@Votes", SqlDbType.VarChar,100).Value = Vote
CountUpdate.ExecuteNonQuery();
conn.Close();
Satish Bhat15y ago
Place a break point in the beginning of the code, debug and let us know the outcome of ExecuteNonQuery(), ExecuteReader() and ExecuteNonQuery() in your code.

Also I didn't understand the following statements:
int countvote=0;
for (int i = 0; i <= countvote; i++)
{
.
.

In the above code, the for loop will run only once. Or did you miss any line which updates countvote? Otherwise you can remove the for loop.
Black Diamond15y ago
private void btnCandidates1_Click_1(object sender, EventArgs e)
{
if (CaptainVoteCheckbox1.Visible == false)
{
CaptainVoteCheckbox1.Visible = true;
int countvote=0;
for (int i = 0; i <= countvote; i++)
{
MessageBox.Show("You have Only One Captain to Vote!");
btnCandidates2.Enabled = false;
btnCandidates3.Enabled = false;
btnCandidates4.Enabled = false;
btnCandidates5.Enabled = false;

try
{
string insert = "INSERT INTO CaptainVotes (Name) VALUES (@Param)";
SqlCommand cmd = new SqlCommand(insert, conn);
cmd.Parameters.Add("@Param", SqlDbType.VarChar, 100);
cmd.Parameters["@Param"].Value = btnCandidates1.Text;
conn.Open();
cmd.ExecuteNonQuery();
}
catch (SqlException ex)
{
MessageBox.Show(ex.Message);
}
finally
{
conn.Close();
}
}

string Vote;
int count = 0;
SqlCommand cmd1 = new SqlCommand("Select * from CaptainVotes where Name='" + btnCandidates1.Text + "'", conn);
SqlDataReader dr = null;
conn.Open();
dr = cmd1.ExecuteReader();

while (dr.Read() == true)
{

++count;
}
conn.Close();

Vote = count.ToString();
conn.Open();
SqlCommand CountUpdate = new SqlCommand("UPDATE FinalVotes SET Name = @Name where Votes = @Votes", conn);
CountUpdate.Parameters.Add("@Name", SqlDbType.VarChar,100).Value = btnCandidates1.Text;
CountUpdate.Parameters.Add("@Votes", SqlDbType.VarChar, 100).Value = Vote;
CountUpdate.ExecuteNonQuery();
conn.Close();

}

Hi guy's,

This is my complete code in my button where i place this code ....because still nothing happen in my sql ...even if i replace my some of the code you'de advice....thanks for the continuing support...
Sam Hobbs15y ago
Note that Vulpes is saying the same thing as I said except I said it as an explanation and Vulpes said it as sample code. In the code that Vulpes posted, the CommandText is being set by passing the command in the SqlCommand constructor, which is just a matter of personal preference whether you want to do it that way.
Sam Hobbs15y ago
Nothing happens because there is nothing to do. Look at the articles in this web site for many samples that do work.

You don't show what CountUpdate is. I assume it is a SqlCommand, and if so then you need to set the CommandText Property and you are not doing that.