[Resolved] mysql database

Anthony_john5

I've tried creating a new table in my MYSQL databse and this is the code outputted:

CREATE TABLE `phpbb_ecards_details` (

`id` TINYINT NOT NULL ,
`submitter` VARCHAR NOT NULL ,
`name` TINYTEXT NOT NULL ,
`date` VARCHAR NOT NULL ,
`website` VARCHAR NOT NULL 
)

and I get this error

MySQL said:

You have an error in your SQL syntax near 'NOT NULL, name TINYTEXT NOT NULL, date VARCHAR NOT NULL, website VARCHAR N' at line 1

I've tried different setting for things and null and not null, but it doesn't seem to work. I've read the documentation and still don't get it. Can anyone offer any help?

Anthony

blind487

It may be as simple as adding a semicolon at the end of your statement. If you have tried that... then try creating a table with just one entry and see if that works, then add another entry.

LordShryku

You need to specify a length for varchar, like so

CREATE TABLE `phpbb_ecards_details` (

   `id` TINYINT NOT NULL ,
   `submitter` VARCHAR(30) NOT NULL ,
   `name` TINYTEXT NOT NULL ,
   `date` VARCHAR(30) NOT NULL ,
   `website` VARCHAR(30) NOT NULL 
)

Moonglobe

also since when is tinytext a datatype?

edit: ok....... so it is.......... my bad.........

LordShryku

Originally posted by Moonglobe
also since when is tinytext a datatype?

AH HAHAHAHAHAHA!

Oh....sorry 😃

Moonglobe

i just dont like the idea of setting limits on the size of my data.......... so i never looked into stuff like that.......... 😃

xblue

just 2 additional thoughts:

I wouldn't use tinyint for the id unless you are very very sure that table will never have more than 127 records, and storing dates in a proper date/datetime (some prefer unix timestamps though) type makes it easier to handle them later.

Anthony_john5

Originally posted by LordShryku
You need to specify a length for varchar, like so

CREATE TABLE `phpbb_ecards_details` (

   `id` TINYINT NOT NULL ,
   `submitter` VARCHAR(30) NOT NULL ,
   `name` TINYTEXT NOT NULL ,
   `date` VARCHAR(30) NOT NULL ,
   `website` VARCHAR(30) NOT NULL 
)

[/B]

Thanks, I've got it sorted now. 🙂

Originally posted by xblue
just 2 additional thoughts:

I wouldn't use tinyint for the id unless you are very very sure that table will never have more than 127 records, and storing dates in a proper date/datetime (some prefer unix timestamps though) type makes it easier to handle them later.

Thanks, I've made the id field smallint and the date field into a 'date'

Anthony