Incorrect Syntax near 'Some Decimal Value(20.12134) '

Question

0.00/5 (No votes)

See more:

THIS IS MY C# CODE

C#

private void button5_Click(object sender, EventArgs e)
        {
            string connection = @"Data Source=(local);Initial Catalog=Project;Integrated        Security=True";
            SqlConnection sc = new SqlConnection(connection);
            string query2 = "insert into dbo.temp1 (weight,height,bmi)values('" + this.WeightTxt.Text + "','" + this.heightTxt.Text+ "','"+this.ResultTxt.Text+"'";
            SqlConnection conn = new SqlConnection(connection);
            SqlCommand cmdDataBase2 = new SqlCommand(query2, conn);
            SqlDataReader myReader;
            try
            {
                conn.Open();
                myReader = cmdDataBase2.ExecuteReader();
                MessageBox.Show("Data Saved Bacchi..");
                
                while (myReader.Read())
                {

                }
                conn.Close();

            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }
        }

THIS IS MY DATABASE CREATE STATEMENT

SQL

create table temp1
(
weight numeric(3),
height numeric (3),
bmi numeric(7,7)
);

Posted 24-Feb-14 19:40pm

Member 10445097

Updated 24-Feb-14 19:55pm

CoderPanda

v2

Add a Solution

2 solutions

Solution 1

Hi
When you are declaring table the table, make use of numeric data type properly. It can be like below

create table temp1
(
weight numeric(3,2),
height numeric (3,2),
bmi numeric(7,7)
);

Generally weight and height have decimal values. So when you are inserting data into the table that you have declared (i'm refering to the table syntax you have writen), we will get the below error
Arithmetic overflow error converting numeric to data type numeric.

I think the problem is with table declaration

Posted 24-Feb-14 20:37pm

Gopi Kishan Mariyala

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)

ravikhoda · Accepted Answer · 2014-02-24T20:38:00

Solution 2

see your query it missing the closing ) for the values block.
try below and check if that works...

C#

string query2 = "insert into dbo.temp1 (weight,height,bmi)values('" + this.WeightTxt.Text + "','" + this.heightTxt.Text+ "','"+this.ResultTxt.Text+"'");

Posted 24-Feb-14 20:38pm

ravikhoda

Comments

♥…ЯҠ…♥ 25-Feb-14 3:00am

Good catch ;-)

Faisalabadians 25-Feb-14 8:29am

yeah, he is missing some thing... also when he is storing query in string variable he must use expression {0},{1},{2} so that code should looks good