¿Tiene algun error este codigo SQL de MySQL?

[Coplo]

CREATE TABLE user (
id int(8) unsigned NOT NULL auto_increment,
user varchar(20) collate latin1_general_ci default NULL,
password varchar(25) collate latin1_general_ci default NULL,
PRIMARY KEY (`ID`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci AUTO_INCREMENT=8 ;

[gnzsoloyo]

No tiene ningún error puntual, salvo que estás usando palabras reservadas como nombres de campos (USER y PASSWORD, por caso). No es buena idea porque puede dar lugar a errores indetectables en las consultas.
Cuando forzosamente debas usar estos nombres, es conveniente que los encierres y los manejes siemrpe entre acentos agudos (`), para evitar inconvenientes.

[Coplo]

Entonces este seria el modo perfeto de crear el codigo:

CREATE TABLE `users` (
`ID` int(8) unsigned NOT NULL auto_increment,
`user` varchar(20) collate latin1_general_ci default NULL,
`password` varchar(25) collate latin1_general_ci default NULL,
PRIMARY KEY (`ID`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci AUTO_INCREMENT=8 ;

[gnzsoloyo]

Ten cuidado con esto: AUTO_INCREMENT=8
Le estás diciendo que comience del numero 8 de ID, con lo que se pierden 7 entradas de índice y registros.

[Coplo]

Gracias por dicirmelo.
Ahora esta bien asi?

CREATE TABLE `users` (
`id` int unsigned NOT NULL auto_increment,
`user` varchar(20) collate latin1_general_ci default NULL,
`password` varchar(25) collate latin1_general_ci default NULL,
PRIMARY KEY (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci;

[gnzsoloyo]

Si, el código está OK.
Sigo recomendando que no uses palabras reservadas. Eventualmente pueden producirse problemas...

[Coplo]

Y como me recomiendas que sea la tabla?
Me puedes dar un ejemplo porfavor?
GRACIAS

[huesos52]

CREATE TABLE `users` (
`id` int unsigned NOT NULL auto_increment,
`user` varchar(20) collate latin1_general_ci default NULL,
`password` varchar(25) collate latin1_general_ci default NULL,
PRIMARY KEY (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci;

A lo que se refiere gnzsoloyo es que no utilices campos llamdos user, password en la declaración de una tabla.

La forma de corregirlo, es ponerle otro nombre a los campos. Por ejemplo:

Código mysql:

CREATE TABLE `users` (
`id` int unsigned NOT NULL auto_increment,
`user_s` varchar(20) collate latin1_general_ci default NULL,
`password_s` varchar(25) collate latin1_general_ci default NULL,
PRIMARY KEY (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci;
 

saludos

[Coplo]

Y que tal asi?

Código MySQL:

CREATE TABLE 'usuarios' (
'id' INT UNSIGNED NOT NULL AUTO_INCREMENT,
'usuario' VARCHAR(20) COLLATE latin1_general_ci DEFAULT NULL,
'clave' VARCHAR(30) COLLATE latin1_general_ci DEFAULT NULL,
PRIMARY KEY ('id')
) ENGINE=MyISAM DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci;
 

Y que otras decomendaciones me pueden dar?
GRACIAS

[huesos52]

Una cosa es el apostrofe y otra las comillas.

Quedaría así:

Código sql:

CREATE TABLE `usuarios` (
`id` INT UNSIGNED NOT NULL AUTO_INCREMENT,
`usuario` VARCHAR(20) COLLATE latin1_general_ci DEFAULT NULL,
`clave` VARCHAR(30) COLLATE latin1_general_ci DEFAULT NULL,
PRIMARY KEY ('id')
) ENGINE=MyISAM DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci;
 

[Coplo]

Gracias, he modificado un poco el codigo y le quite la linea PRIMARY KEY (`id`).
De tal forma que quedaria asi:

Código sql:

CREATE TABLE `usuarios` (
`id` INT UNSIGNED NOT NULL AUTO_INCREMENT PRIMARY KEY,
`usuario` VARCHAR(20) COLLATE latin1_general_ci DEFAULT NULL,
`clave` VARCHAR(30) COLLATE latin1_general_ci DEFAULT NULL
) ENGINE=MyISAM DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci;