bene, il codice sembra in ordine.

è importante notare sulla creazione delle tabelle che generalmente:

quando si crea un campo numerico si una INT(numero di cifre)

per i campi di testo VARCHAR(numero di caratteri < 255, dopodichè diventa TEXT)

per ogni campo precisare se può essere o meno NULL

per ogni campo con un auto increment (nel nostro caso l' id) specificare se deve autoincrementarsi
quindi

se volessimo una tabella con i tuoi campi definendo però che NESSUN campo possa essere null, dobbiamo utilizzare questa sintassi:
codice:
CREATE TABLE email
(
id INT(3),
Nome VARCHAR(20) NOT NULL AUTO_INCREMENT PRIMARY KEY ,
Indirizzo VARCHAR(40) NOT NULL,
Telefono VARCHAR(12) NOT NULL,
Cellulare VARCHAR(15) NOT NULL,
Email VARCHAR(30) NOT NULL)ENGINE=MYISAM
cosa sono AUTO_INCREMENT, PRIMARY KEY e ENGINE ?

AUTO_INCREMENT : vuol dire che ad un nuovo record mysql assegna al campo un valore automatico incrementato da lui rispetto all' indice auto_increment attuale della tabella

PRIMARY KEY: è un campo chiave, su cui verte un indice speciale

ENGINE : il tipo di motore che deve essere utilizzato (noi usiamo myisam ma potrebbe essere per esempio INNODB)