incorrect syntax

Question

0.00/5 (No votes)

See more:

SQL-Server-2005

, +

I have a mistake in my project it shows the error like Incorrecr syntax neare ','. on updating my query

(" update payment set BATCHCODE=" + textBox2.Text + ", BATCHMEMBERS=" + comboBox6.Text + ", BATCHNAME='" + textBox10.Text + "', COURSE='" + comboBox5.Text + "', BRANCH='" + comboBox4.Text + "', COLLEGE='" + textBox8.Text + "', TITLE='" + textBox3.Text + "', TECHNOLOGY='" + comboBox2.Text + "', TYPE='" + comboBox1.Text + "',PROJECTSTATUS='" + comboBox3.Text + "',DELIVERYDATE='" + textBox9.Text + "', PROJECTCOST=" + textBox4.Text + ", PAIDAMOUNT=" + textBox12.Text + ",PAYMENT=" + textBox6.Text + ", BALANCE=" + textBox7.Text + " WHERE REGNO=" + textBox1.Text + "");

this is my code for updating when i add other field with payment the fields where updating, but when i enter other fields without payment it does'nt updating . please give sajections

Posted 31-Oct-10 21:20pm

sevenbell

Updated 31-Oct-10 21:24pm

Hiren solanki

v2

Add a Solution

Comments

Hiren solanki 1-Nov-10 3:26am

improved code readability.

4 solutions

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)

RDBurmon · Answer 1 · 2010-10-31T23:01:00

SQL

(" update payment set BATCHCODE=" + textBox2.Text + ", BATCHMEMBERS=" + comboBox6.Text + ", BATCHNAME='" + textBox10.Text + "', COURSE='" + comboBox5.Text + "', BRANCH='" + comboBox4.Text + "', COLLEGE='" + textBox8.Text + "', TITLE='" + textBox3.Text + "', TECHNOLOGY='" + comboBox2.Text + "', [TYPE]='" + comboBox1.Text + "',PROJECTSTATUS='" + comboBox3.Text + "',DELIVERYDATE='" + textBox9.Text + "', PROJECTCOST=" + textBox4.Text + ", PAIDAMOUNT=" + textBox12.Text + ",PAYMENT=" + textBox6.Text + ", BALANCE=" + textBox7.Text + " WHERE REGNO=" + textBox1.Text + "");

I mark my changes as a bold

also make sure that BATCHCODE , BATCHMEMBERS are not a string data type (i mean nvarchar,varchar etc....)

cp.soni · Answer 2 · 2010-11-01T00:33:00

Solution 3

Please make sure that this Dynamic SQL have values to set in the fields.
e.g. it should not look like as follows

update payment set BATCHCODE=, BATCHMEMBERS=, .....

there must be some values or null to assign in the columns for a update statement.

Posted 1-Nov-10 0:33am

cp.soni

Comments

RDBurmon 1-Nov-10 8:15am

I think CP , if wstar make sure that BATCHCODE , BATCHMEMBERS are not a string data type (i mean nvarchar,varchar etc....)
If it is a string data type the he must use BATCHCODE='" + textBox2.Text + "'.........

cp.soni 4-Nov-10 10:38am

Yes, that is correct, but it could be possible that nothing is being passed from the controls which He should make sure.

souidi abderrahman · Answer 3 · 2010-11-03T04:58:00

maybe you need to verify your query in sql :

<pre lang="sql">update payment set
 BATCHCODE="'" + textBox2.Text + "'",
 BATCHMEMBERS="'" + comboBox6.Text + "'",
  BATCHNAME="'" + textBox10.Text + "'",
  COURSE="'" + comboBox5.Text + "'",
  BRANCH="'" + comboBox4.Text + "'",
  COLLEGE="'" + textBox8.Text + "'",
  TITLE="'" + textBox3.Text + "'",
  TECHNOLOGY="'" + comboBox2.Text + "'",
  [TYPE]= "'" + comboBox1.Text + "'",
  PROJECTSTATUS="'" + comboBox3.Text +"'",
  DELIVERYDATE="'" + textBox9.Text + "'",
  PROJECTCOST="'" + textBox4.Text + "'",
  PAIDAMOUNT="'" + textBox12.Text + "'",
  PAYMENT="'" + textBox6.Text + "'",
  BALANCE="'" + textBox7.Text + "'"
   WHERE REGNO="" + textBox1.Text + """

Hiren solanki · Answer 4 · 2010-10-31T21:31:00

Solution 1

I can not find any mistake in UPDATE method, You just make sure that your any of TextBox or ComboBox value doesn't containing "," char.

My advise would be to have parameterized query update rather then attaching value inline.

See THIS[^] For more detail.

Please vote and Accept Answer if it Helped.

Posted 31-Oct-10 21:31pm

Hiren solanki

Comments

TweakBird 4-Nov-10 2:19am

Mr Solanki,

i got error like this for your posted link. see the error message below.

HTTP Error 404.0 - Not Found
The resource you are looking for has been removed, had its name changed, or is temporarily unavailable.

incorrect syntax

4 solutions

Solution 2

Solution 3

Solution 4

Solution 1

Add your solution here

Preview 0