?
Solved

updating SQL server through C# vs2010

Posted on 2012-03-27
8
Medium Priority
?
426 Views
Last Modified: 2012-06-27
Hi

I have two identical tables.
I use one as a temporary table for importing data to, so it can be manipulated before updating the real table.

I've tried updaing the real table with:

 string SQL = "INSERT INTO ActualReadings " +
                                " (Date,Time,Thing1, thing2, thing3)" +
                        "SELECT  "  +
                        "(Date,Time,Thing1, thing2, thing3)" +
                            " FROM TempReadings";

                //Declare the Command
                SqlCommand cmd = new SqlCommand(SQL, conn);

                //Make the connection
                conn.Open();
               
                //Execute query
                int NumRows = 0;
                NumRows = cmd.ExecuteNonQuery();
               
but get an error, that I believe is related to the fact both tables have an identity column (ID).

What's the best way around this?

Many thanks.
0
Comment
Question by:Spike_66
  • 5
  • 3
8 Comments
 
LVL 10

Expert Comment

by:plummet
ID: 37772635
Hi

Your SQL looks OK to me, you're not trying to insert an identity value unless one of your fields Date, Time, Thing1 etc are an identity field.

What is the error you're getting?
0
 

Author Comment

by:Spike_66
ID: 37776413
Incorrect syntax near ','.
 which is refering to the first "," after the word select.

i'm sure it's something to do with both tables having identity columns (ID) which I'm not inserting but obviuosly want to generate.
The column is set up correctly, but just doesn't want to work on a bulk inport.
0
 
LVL 10

Accepted Solution

by:
plummet earned 2000 total points
ID: 37776429
Try sticking [ and ] around Date and Time, as they are reserved words, and remove those brackets:

string SQL = "INSERT INTO ActualReadings " +
                                " ([Date],[Time],Thing1, thing2, thing3)" +
                        "SELECT  "  +
                        "[Date],[Time],Thing1, thing2, thing3 " +
                            " FROM TempReadings";

Open in new window


Hopefully that will fix it
0
Never miss a deadline with monday.com

The revolutionary project management tool is here!   Plan visually with a single glance and make sure your projects get done.

 

Author Comment

by:Spike_66
ID: 37776587
Many thanks- I noticed the brackets just  before you replied and removing them fixed it.
0
 
LVL 10

Expert Comment

by:plummet
ID: 37776605
Yep, simple as that, not an issue with identity. I'm glad to have helped

Cheers
John
0
 
LVL 10

Expert Comment

by:plummet
ID: 37779285
Hi Spike

Are you going to allocate points for my answer?

Many thanks
John
0
 

Author Comment

by:Spike_66
ID: 37781149
Apologies- I thought I had done it (It must be old age!!)
0
 
LVL 10

Expert Comment

by:plummet
ID: 37781255
I quite understand, judging by your name on here I am 3 years older than you. Now where did I put my glasses...
0

Featured Post

Never miss a deadline with monday.com

The revolutionary project management tool is here!   Plan visually with a single glance and make sure your projects get done.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

In part one, we reviewed the prerequisites required for installing SQL Server vNext. In this part we will explore how to install Microsoft's SQL Server on Ubuntu 16.04.
This shares a stored procedure to retrieve permissions for a given user on the current database or across all databases on a server.
Via a live example combined with referencing Books Online, show some of the information that can be extracted from the Catalog Views in SQL Server.
This videos aims to give the viewer a basic demonstration of how a user can query current session information by using the SYS_CONTEXT function
Suggested Courses

599 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question