Database INSERT statement

Question

Do I need to create a new statement each time I execute a query?

This line doesn't work (I have already established connection with the database )

Statement.execute("INSERT INTO PS2 (uuid , team, kills , deaths , rank_lvl, rank_name, pts) "
                    + "VALUES"
                    + " (`" + name.toString() + "` , `none` , 0 , 0 , 1 , `Starter` , 0)");

I create the table with this code :

Statement.execute("CREATE TABLE IF NOT EXISTS PS2"
                + "(uuid VARCHAR(45),"
                + "team VARCHAR(20),"
                + "kills INTEGER,"
                + "deaths INTEGER,"
                + "rank_lvl INTEGER,"
                + "rank_name VARCHAR(25),"
                + "pts INTEGER)");

After executing the code , the database has been created . Afterwards after trying to insert row, the row is not created!

There is no error , there is just no rows in the database , @AlexK. i am testing — ntakouris
– ntakouris, Commented Jul 2, 2014 at 9:49
What RDBMS do you use? Some of them are caching data and while you does'nt commit or flush there is no data written to disk. — PeterMmm
– PeterMmm, Commented Jul 2, 2014 at 9:49
And never ever use a concat string for SQL. Use a reusable PreparedStatement. — PeterMmm
– PeterMmm, Commented Jul 2, 2014 at 9:52

zessx · Accepted Answer · 2014-07-02 10:04:44Z

2

You're using backticks ` instead of quotes ' to wrap your values. Backticks are only used to wrap column names.

Statement.execute("INSERT INTO PS2 (`uuid` , team, kills , deaths , rank_lvl, rank_name, pts) "
    + "VALUES"
    + " ('" + name.toString() + "' , 'none' , 0 , 0 , 1 , 'Starter' , 0)");

As mention by Caweren, you should be careful of SQL reserved keywords. UUID could be one of them (with an Oracle V9i database, for example), so you should wrap it with backticks.

edited Jul 2, 2014 at 10:04

answered Jul 2, 2014 at 9:57

zessx

68.9k29 gold badges139 silver badges166 bronze badges

Sign up to request clarification or add additional context in comments.

8 Comments

zessx Over a year ago

I would manually try this request, to know it the problem come from the request, or from the code.

ntakouris Over a year ago

It worked manually :S , using this piece of code : INSERT INTO PS2(uuid , team , kills , deaths , rank_lvl , rank_name , pts) VALUES('460b9dd9-af9c-4942-a49d-28d365658e7a' , 'none' , 0 ,0 , 1 , 'Starter', 0)

ntakouris Over a year ago

There are no errors when executing the INSERT statement BUT the row is not created!

zessx Over a year ago

Then it come from somewhere else. What kind of database are you using ?

zessx Over a year ago

Everything is correct. Does any INSERT work via PHPMyAdmin (even on other tables) ?

|

Caweren · Accepted Answer · 2014-07-02 09:53:06Z

0

uuid is a reserved keyword for SQL, you should wrap it in quotes: `uuid`

answered Jul 2, 2014 at 9:53

Caweren

2441 silver badge10 bronze badges

2 Comments

ntakouris Over a year ago

The database got created without it being wrapped by quotes

Caweren Over a year ago

Tables can be created using reserved words, but to use them in statements or clauses you have wrap them in quotes.

Collectives™ on Stack Overflow

Database INSERT statement

2 Answers 2

8 Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

8 Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related