C# Discussion Boards - CodeProject

Vivek Vijayan28-Jul-09 4:50

28-Jul-09 4:50

yes i tried specifying the columns
but no use
any prob with insert query?????
String query = "insert into mydb(username,password) values('" + textBox1.Text + "','" + textBox2.Text + "')";

This code was posted by me...

musefan28-Jul-09 4:55

28-Jul-09 4:55

try...

INSERT INTO mydb ([username],[password]) VALUES ('" + textBox1.Text + "','" + textBox2.Text + "')

Also...

- Confirm that 'mydb' is a table name and not the name of your database.
- make sure your database columns have the corrent datatype
- look into using parameters

Life goes very fast. Tomorrow, today is already yesterday.

moon_stick28-Jul-09 4:43

28-Jul-09 4:43

What values does this method return? You're catching the exception but not doing anything with it (apart from changing the return value). Assuming you're using VS, set a breakpoint on the line "return 0;" and check to see that the value of 'e' (the exception) is.

It definitely isn't definatley

Vivek Vijayan28-Jul-09 4:52

28-Jul-09 4:52

i debuggged the code
but no exception is caught
may be prob with insert statemnt???
any prob with insert query?????
String query = "insert into mydb(username,password) values('" + textBox1.Text + "','" + textBox2.Text + "')";

This code was posted by me...

moon_stick28-Jul-09 5:07

28-Jul-09 5:07

Try changing your code to this:

SqlConnection con = connection();
SqlCommand cmd = new SqlCommand();
cmd.CommandText = query;
cmd.Connection = con;
con.Open();
cmd.ExecuteNonQuery();           
con.Close();
cmd.Dispose();

and see if you get an exception - if the insert has failed, the database should indicate this.

It definitely isn't definatley

moon_stick28-Jul-09 4:38

28-Jul-09 4:38

With regards to the query, I meant the actual string value that is being passed to the database, rather than the statement you use to construct it. Incidentally, is 'mydb' the name of the table or the name of a database containing the table (i.e. should it actually be "insert into mydb.dbo.mytable..."?).

What about the rest of the questions I asked? Have you tried running a profiler on the database to see if the query is actually received. Does the c# statement actually exectute?

It definitely isn't definatley

Vivek Vijayan28-Jul-09 4:44

28-Jul-09 4:44

no exception is caught
i don knwo where is the problem

what is the meaning of this
"Have you tried running a profiler on the database to see if the query is actually received. "

This code was posted by me...

moon_stick28-Jul-09 4:49

28-Jul-09 4:49

A database profiler is a tool which monitors a database and reports back when certain events occur, such as connections to the database, queries, triggers etc. SQL Server comes with its own profiler called (originally enough) SQL Server Profiler - I don't tend to use other databases but I'm sure there are similar tools available for other platforms.

It definitely isn't definatley

Vivek Vijayan28-Jul-09 5:05

28-Jul-09 5:05

do we use cmd.ExecuteNonQuery()
for inserting??

This code was posted by me...

moon_stick28-Jul-09 5:10

28-Jul-09 5:10

Yes that's fine. Broadly speaking, you use ExecuteNonQuery() to execute a SQL statement that doesn't return a result set, ExecuteQuery() to execute a statement that returns tabular data and ExecuteScalar() to execute a query which returns a single value (i.e. if you've done a count(*) or other scalar operation).

It definitely isn't definatley

Ian Shlasko28-Jul-09 4:16

Ian Shlasko

28-Jul-09 4:16

If you want a meaningful response, you're going to have to be a lot more specific.

What exception are you getting? What line is triggering it? What query are you sending?

We're programmers, not psychics.

Proud to have finally moved to the A-Ark. Which one are you in?
Developer, Author (Guardians of Xen)

Vivek Vijayan28-Jul-09 4:25

28-Jul-09 4:25

no exception is caught
insert query is
String query = "insert into mydb values('" + textBox1.Text + "','" + textBox2.Text + "')";

This code was posted by me...

Vimalsoft(Pty) Ltd28-Jul-09 4:17

Vimalsoft(Pty) Ltd

28-Jul-09 4:17

Good Day Vivek

Today was a long day, and seing this kind of post in the forums bores me to death. First of all when you post something here on codeproject you must use the pre tags so that the code will be formated well and it can differ from your message. secondly we cant know the problem until we see what is contained in the variable "query" , and another thing there is a bad coding practise in your code, you start returning without closing the connection.

Vuyiswa Maseko,

Few companies that installed computers to reduce the employment of clerks have realized their expectations.... They now need more and more expensive clerks even though they call them "Developers" or "Programmers."

C#/VB.NET/ASP.NET/SQL7/2000/2005/2008
http://www.vuyiswamaseko.somee.com
http://www.vuyiswamaseko.tiyaneProperties.co.za
vuyiswa@its.co.za
http://www.itsabacus.co.za/itsabacus/

Vivek Vijayan28-Jul-09 4:23

28-Jul-09 4:23

i don feel that pasting the code for close is necessary....

This code was posted by me...

Vimalsoft(Pty) Ltd28-Jul-09 4:24

Vimalsoft(Pty) Ltd

28-Jul-09 4:24

Then i cant help

Vivek Vijayan28-Jul-09 4:26

28-Jul-09 4:26

no need tnx

This code was posted by me...

musefan28-Jul-09 4:45

28-Jul-09 4:45

Vivek Vijayan wrote:
i don feel that pasting the code for close is necessary....

Your right, it is not necessary... when you provide a better description of what the problem is. But as everyone is having to guess how bad your code is, people are going to ask questions.

Life goes very fast. Tomorrow, today is already yesterday.

Vivek Vijayan28-Jul-09 4:55

28-Jul-09 4:55

this community is for helping not for findng mistakes only

This code was posted by me...

musefan28-Jul-09 5:22

28-Jul-09 5:22

Vivek Vijayan wrote:
this community is for helping not for findng mistakes only

Finding mistakes is helping

Life goes very fast. Tomorrow, today is already yesterday.

musefan28-Jul-09 4:21

28-Jul-09 4:21

I would say you have a compile error for no catch/finally block but that's the best I can give you for the example you have shown...

Oh, no wait... it's coming to me... 'the.. answer.. is.. .. ..' ..Damn, lost it!

Life goes very fast. Tomorrow, today is already yesterday.

Vivek Vijayan28-Jul-09 4:28

28-Jul-09 4:28

if i am writing code for try{}
i know how to write catch and finally
tnx

This code was posted by me...

musefan28-Jul-09 4:40