Debería usar un Stored Procedure?

[gnzsoloyo]

Vamos a algunos detalles:

1) Cuando haces una inserción de este tipo, en el que se ven afectadas más de una tabla, es conveniente usar transacciones, de modo que si algo falla, se deshagan todas las inserciones (cuestiones de consistencia).

2) Las variables de entrada no se usan con "@". Eso es de SQL Server (Microsoft) y no de MySQL. Las variables se usan simplemente declarandolas en el prototipo y usandolas con su nombre.

3) Las variables con "@" son variables de usuario y siguen existiendo y con su valor mientras dure la conexión. No es conveniente usarlas si no es estrictamente necesario precisamente por su permanencia, y cada vez que se las invoca deben ser inicializadas para evitar que guarden basura. Son muy útiles en cierto tipo de operaciones, pero no es buena prática usarlas en el prototipo de los SP, entre otras cosas porque si por alguna razón dos SP distintos que usan la misma variable son invocados por el mismo usuario en la misma sesión (threads de ejecución diferentes), el cambio de valor de la variable en un thread afectará el valor de la variable en el otro thread, simplemente porque apuntan a lo mismo...

4) Las variables no deben bajo ninguna circunstancia tener el mismo nombre de campos, tablas, bases o cualquier otro objeto de base de datos, porque el parser confundirá los objetos, ya que jerarquiza los mismos y en ese caso una tabla o un campo tienen mayor jerarquía que una variable, en consecuencia intentará acceder al campo o tabla y no a la variable, lo que puede dar lugar a errores aleatorios.

Este sería un modelo, basado en tu ejemplo:

Código MySQL:

CREATE PROCEDURE nuevoUsuario (
  IN vuser_id int,
  IN vuser_login varchar(25),
  IN vuser_pass varchar(30),
  IN vuser_activation_key varchar(40),
  IN vuser_name varchar(30),
  IN vuser_mail varchar(45),
  IN vuser_birthday date,
  IN vuser_sexo char(1),
  IN vuser_pais smallint,
  IN vfecha_registro timestamp,
  IN vip_registro varchar(15))
 
BEGIN
 
START TRANSACTION;
 
  INSERT INTO tbl_user(
    user_login,
    user_pass,
    user_activation_key)
  VALUES(
    vuser_login,
    vuser_pass,
    vuser_activation_key);
 
  INSERT INTO tbl_user_personal(
    vuser_id,
    vuser_name,
    vuser_mail,
    vuser_birthday,
    vuser_sexo,
    vuser_pais)
  VALUES (
    vuser_id,
    vuser_name,
    vuser_mail,
    vuser_birthday,
    vuser_sexo,
    vuser_pais);
 
  INSERT INTO tbl_registro (
    vuser_id,
    vfecha_registro,
    vip_registro)
  VALUES (
    vuser_id,
    NOW(),
    vip_registro);
 
COMMIT;
 
END
 

Es de buenas prácticas Poner el sentido de entrada/salida de las variables, por más que el IN sea implícito. Facilita cualquier análisis rápido y además de ese modo no te olvidas de poner los OUT o INOUT cuando corresponden.