[SOLUCIONADO] ¿Qué estoy haciendo mal? [Procedimiento almacenado]

[LinkServ]

Hola a todos,

Soy yo nuevamente, esta vez mi problema es el siguiente:

Después de unas cuantas horas leyendo acerca de los procedimiento almacenados,
he creado uno propio, el objetivo de mi procedimiento es añadir un nuevo usuario a una tabla (Usuarios) o modificar (dependiendo de cómo mande a llamar dicho procedimiento).

Cada usuario nuevo obtiene una id automáticamente (incrementandole un 1 al número de la id del último registro).

Funciona bien cuando se trata de agregar al primer usuario con la id 0 en la tabla, lo mismo con el segundo usuario con la id 1, pero al querer insertar el tercer usuario con la id 2 es donde me bloqueo... Pues me quiere agregar el tercer usuario con la id 1.

Mi procedimiento almacenado es el siguiente:

Código SQL:

CREATE PROCEDURE CrearUsuario (IN idx CHAR(9), IN nombrex VARCHAR(25), IN apellidox VARCHAR(25), IN edadx INT(2), IN generox CHAR(1), IN ciudadx VARCHAR(30), IN estadox VARCHAR(30), IN paisx VARCHAR(30))
BEGIN
    IF (idx = "Agregar") THEN
        SET @numUsuario = (SELECT MAX(id) FROM Usuarios);
        SET @numUsuario = IF(@numUsuario IS NULL, 0, @numUsuario + 1);
        INSERT INTO Usuarios(id, nombre, apellido, edad, genero, ciudad, estado, pais) VALUES(@numUsuario, nombrex, apellidox, edadx, generox, ciudadx, estadox, paisx);
    ELSE
        UPDATE Usuarios SET nombre = nombrex, apellido = apellidox, edad = edadx, genero = generox, ciudad = ciudadx, estado = estadox, pais = paisx WHERE id = idx;
    END IF;
END$$
 

Espero sus respuestas, gracias.

[gnzsoloyo]

Primer consejo: No estás escribiendo PHP, sino SQL. Si un parámetro de entrada se usa como entero, no debes ingresar por él nada que no sean enteros. Nada.
El parámetro "idx" lo estás usando tanto para ingresar "Agregar" como el numero de ID del usuario, y eso es un error conceptual. O es una cosa, o es la otra, pero no debes usarlo para ambas posibilidades.
Además, tampoco lo necesitas como caracter, porque simplemente conque entre el numero, o un cero, puedes contar cuantos registros hay con ese ID y si no devuelve ninguno es que hay qe agregar el usuario (no debe existir jamás un ID cero en la tabla, eso es absurdo).
Por otro lado, a menos que vayas a usar el numero del nuevo usuario fuera del SP inmediatamente, y no se cierre la conexión a la base en ningún momento, no conviene usar variables de usuario global de MySQL, sino locales del SP.
Si lo que necesitas es recuperar el ID generado, simplemente le pones un SELECT numUsuario al final, y la ejecución del SP te devolverá por resultado el numero generado.

La idea sería mas o menos así:

Código MySQL:

CREATE PROCEDURE CrearUsuario (
    IN idx CHAR(9), 
    IN nombrex VARCHAR(25), 
    IN apellidox VARCHAR(25), 
    IN edadx INT(2), 
    IN generox CHAR(1), 
    IN ciudadx VARCHAR(30), 
    IN estadox VARCHAR(30), 
    IN paisx VARCHAR(30))
BEGIN
    DECLARE numUsuario INT DEFAULT 0;
    IF (SELECT count(1) FROM Usuarios WHERE id = idx) = 0 THEN
        SELECT MAX(id) FROM Usuarios INTO numusuario;
        SET numUsuario = numUsuario + 1;
        INSERT INTO Usuarios(id, nombre, apellido, edad, genero, ciudad, estado, pais) 
        VALUES(numUsuario, nombrex, apellidox, edadx, generox, ciudadx, estadox, paisx);
    ELSE
        UPDATE Usuarios 
          SET nombre = nombrex, 
          apellido = apellidox, 
          edad = edadx, 
          genero = generox, 
          ciudad = ciudadx, 
          estado = estadox, 
          pais = paisx 
        WHERE id = idx;
    END IF;
    SELECT numUsuario;
END$$
 

[LinkServ]

Gracias nuevamente por tu respuesta, gnzsoloyo.

He mirado el SP que colocaste y lo probé pero me daba el siguiente error:

#1048 - Column 'id' cannot be null

Analicé bien el código y probé modificando la siguiente línea:

SELECT MAX(id) FROM Usuarios INTO numusuario;

por esta:

SELECT MAX(id) FROM Usuarios;

Entonces, ahí si me funcionó el SP cuando agregué el primer registro a la tabla Usuarios.

Después intenté agregar otro registro y ya no me dejaba por lo mismo del error que me salía con mi SP:

#1062 - Duplicate entry '1' for key 'PRIMARY'

[gnzsoloyo]

#1048 - Column 'id' cannot be null

Eso se puede dar en el contexto de ese SP si y sólo si la tabla está vacía. Cuando ya existe al menos un registro eso no se puede dar, si se usa:

Código SQL:

SELECT MAX(id) FROM Usuarios INTO numusuario;
 

Para resolverlo no hay que haer lo que hiciste, porque se genera una duplicidad de datos. Lo que corresponde es verificar si la variable se volvió nula...

Código MySQL:

CREATE PROCEDURE CrearUsuario (
    IN idx CHAR(9), 
    IN nombrex VARCHAR(25), 
    IN apellidox VARCHAR(25), 
    IN edadx INT(2), 
    IN generox CHAR(1), 
    IN ciudadx VARCHAR(30), 
    IN estadox VARCHAR(30), 
    IN paisx VARCHAR(30))
BEGIN
    DECLARE numUsuario INT DEFAULT 0;
    IF (SELECT count(1) FROM Usuarios WHERE id = idx) = 0 THEN
        SELECT MAX(id) FROM Usuarios INTO numusuario;
        -- Controlando que sea nulo, y por tanto la tabla este vacia.
        SET numUsuario = IFNULL(numUsuario, 0) + 1;
        INSERT INTO Usuarios(id, nombre, apellido, edad, genero, ciudad, estado, pais) 
        VALUES(numUsuario, nombrex, apellidox, edadx, generox, ciudadx, estadox, paisx);
    ELSE
        UPDATE Usuarios 
          SET nombre = nombrex, 
          apellido = apellidox, 
          edad = edadx, 
          genero = generox, 
          ciudad = ciudadx, 
          estado = estadox, 
          pais = paisx 
        WHERE id = idx;
    END IF;
    SELECT numUsuario;
END$$
 

Para que se entienda: NULL no es un valor, es un estado. Pero toda operación de suma, multiplicación, división, resta, o concatenación de cualquier valor con NULL, da... NULL.
Entre esas situaciones está que MAX(), MIN(), AVG(), aplicado sobre una tabla sin registros, o una consulta que no devuelve registros, da pro resultado NULL.
Además, el hecho de que inicialices una variable con un valor por default, no implica que el valor se conserve si (como en este caso), le terminas asignando un resutlado NULL. En ese caso prevalece el NULL, y por eso se rompe todo.
La solución es sencilla: Debes controlar toda posibilidad de que se presente el NULL y no hacer operaciones con valores así.

Nota final: Como NULL no es un dato, tampoco se pueden usar operadores lógicos con el, sólo se puede verificar si es o no nulo. Por eso en ciertas ocasiones puede ser conveniente usar IF EXISTS, IS NULL o IS NOT NULL, para validar el resultado de consultas y subconsultas.

[LinkServ]

Muchísimas gracias, gnzsoloyo, muy buenas y claras tus explicaciones.

Saludos :D