Update query problem for datagridview values updating time

Question

0.00/5 (No votes)

See more:

C#

Any one please give me some ideas update query
SqlCommand cmd2 = new SqlCommand(&quot;update sales set  empnames=@empnames+&#39;&quot; + comboBox1.Text +&quot;&#39;,categories=@categories, weight=@weight,per=@per,wastage=@wastage,customer=@customer,party=@party,amtcst=@amtcst,amtparty=@amtparty,amtfinal=@amtfinal where date=@date&quot;, con2);
In the above query empnames add 2 times in database it meas 
EX : 
Empnames column : ramram, I want stored single name only. Give me some ideas.

What I have tried:

Update query problem for datagridview values updating time

Posted 26-Nov-16 5:10am

Boopalslm

Updated 26-Nov-16 6:19am

Add a Solution

Comments

OriginalGriff 26-Nov-16 11:38am

This is not a good question - we cannot work out from that little what you are trying to do.
Remember that we can't see your screen, access your HDD, or read your mind.
Use the "Improve question" widget to edit your question and provide better information.

Boopalslm 26-Nov-16 11:57am

HariHari 1 chain 2 2 2 0.04 2 0.130 5 -4.87
Hari 2 chain 3 3 3 0.09 3 0.130 5 -4.87

This is my update qury
SqlCommand cmd2 = new SqlCommand("update sales set empnames=@empnames+'" + comboBox1.Text + "', date=@date, categories=@categories, weight=@weight,per=@per,wastage=@wastage,customer=@customer,party=@party,amtcst=@amtcst,amtparty=@amtparty,amtfinal=@amtfinal where date=@date", con2);

See empnames names cell in first row empnames add 2 times
How to solve this problem.

1 solution

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)

OriginalGriff · Answer 1 · 2016-11-26T06:19:00

At a guess - and since you don't seem to want to provide actual information that's all it can be - your combobox contains the empname, and the @empnames parameter contains it as well. Or the @empnames parameter contains it twice.

We can't tell, because we can't read your code, or see anything other than exactly what you type.
So use the debugger, look at the exact data you are sending to SQL and it shoudl be obvious. We can't do that for you...

But don't do that. If you are aware of parameterised queries, then why are you not using them throughout, instead of risking SQL Injection by concatenating strings?