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)