[SOLUCIONADO] Error en Procedimiento Almacenado

[raul_loz]

Alguien me puede ayudar ya que estoy haciendo dos Procedimientos almacenados, los dos contienen parametros pero no entiendo por que el error:

Código MySQL:

CREATE PROCEDURE SP_FERSA_InsertaCotizacion
(
 p_Folio VARCHAR(100),
 p_Producto VARCHAR(100),
 p_Cantidad INT(11),
 p_Codigo VARCHAR(100),
 p_Descripcion VARCHAR(200),
 p_Precio DECIMAL(18,2),
 p_Importe DECIMAL(18,2),
 p_SubTotal DECIMAL(18,2),
 p_Iva DECIMAL(18,2),
 p_Total DECIMAL(18,2),
 p_FechaRegistro TIMESTAMP
)
BEGIN
 
INSERT INTO detallecotizacion (Folio, Producto, Cantidad, Codigo, Descripcion, Precio, Importe, SubTotal, Iva, Total, FechaRegistro)
            
VALUES  (p_Folio, p_Producto, p_Cantidad, p_Codigo, p_Descripcion, p_Precio, p_Importe, p_SubTotal, p_Iva, p_Total, p_FechaRegistro) -- aqui al cerrar es donde me marca error.
 
END;
 

Y en este otro es lo mismo pero me marca el error al iniciar la columna:

Código MySQL:

CREATE PROCEDURE SP_FERSA_ConsultaFormularioCotizador
(
 p_Producto VARCHAR(100),
 p_tipo VARCHAR(100),
 p_Frente VARCHAR(100),
 p_Profundidad VARCHAR(10),
 p_Altura VARCHAR(100),
 p_Piso VARCHAR(100),
 p_Puertas VARCHAR(100),
 p_Modulos VARCHAR(10)
)
BEGIN
 
SELECT  
c.Producto into p_Producto, 
c.Sub_Producto into p_tipo,  -- aqui al iniciar es donde me marca el error
c.Frente into p_Frente, 
c.Profundidad into p_Profundidad, 
c.Altura into p_Altura, 
c.Piso into p_Piso, 
c.Puertas into p_Puertas
 
FROM producto p , sub_producto sp, combinaciones c
WHERE p.id_prod = sp.id_prod
AND sp.id_prod = c.ID_SUB_PROD
 
 END;
 

Gracias y Saludos

[gnzsoloyo]

Tienes varios errores en el segundo SP:
1) Es un solo INTO seguido de todas las variables. Eso lo puedes ver en el manual de referencia (recomiendo que lo leas, está en la web oficial).
2) Todos los parámetros que pones son de entrada y ninguno de salida, por lo que ese SP no te devolverá datos en ellos.
Esto también se explica claramente en el manual de referencia...

Además:
3) La consulta que pusiste puede devolver más de un registro, pero por parámetro sólo puede devolver valores simples. MySQL no tiene datos de tipo CURSOR, o ARRAY, como otros DBMS, por lo que el SP disparará un error si hay más de un registro.

Código MySQL:

CREATE PROCEDURE SP_FERSA_ConsultaFormularioCotizador
(
 OUT p_Producto VARCHAR(100),
 OUT p_tipo VARCHAR(100),
 OUT p_Frente VARCHAR(100),
 OUT p_Profundidad VARCHAR(10),
 OUT p_Altura VARCHAR(100),
 OUT p_Piso VARCHAR(100),
 OUT p_Puertas VARCHAR(100),
 OUT p_Modulos VARCHAR(10)
)
BEGIN
 
    SELECT  
        c.Producto, 
        c.Sub_Producto, 
        c.Frente, 
        c.Profundidad, 
        c.Altura, 
        c.Piso, 
        c.Puertas
    INTO P_PRODUCTO, P_TIPO, P_FRENTE, P_PROFUNDIDAD, P_ALTURA, P_PISO, P_PUERTAS
    FROM producto p INNER JOIN sub_producto sp ON p.id_prod = sp.id_prod 
        INNER JOIN combinaciones c ON sp.id_prod = c.ID_SUB_PROD;
  END;
 

Yo recomendaría no usar parametros y obtener la tabla resultado que devuelve el SP directamente como tal, pero no sé en qué contexto la estás usando.

[raul_loz]

Iniciado por gnzsoloyo

Tienes varios errores en el segundo SP:
1) Es un solo INTO seguido de todas las variables. Eso lo puedes ver en el manual de referencia (recomiendo que lo leas, está en la web oficial).
2) Todos los parámetros que pones son de entrada y ninguno de salida, por lo que ese SP no te devolverá datos en ellos.
Esto también se explica claramente en el manual de referencia...

Además:
3) La consulta que pusiste puede devolver más de un registro, pero por parámetro sólo puede devolver valores simples. MySQL no tiene datos de tipo CURSOR, o ARRAY, como otros DBMS, por lo que el SP disparará un error si hay más de un registro.

Código MySQL:

CREATE PROCEDURE SP_FERSA_ConsultaFormularioCotizador
(
 OUT p_Producto VARCHAR(100),
 OUT p_tipo VARCHAR(100),
 OUT p_Frente VARCHAR(100),
 OUT p_Profundidad VARCHAR(10),
 OUT p_Altura VARCHAR(100),
 OUT p_Piso VARCHAR(100),
 OUT p_Puertas VARCHAR(100),
 OUT p_Modulos VARCHAR(10)
)
BEGIN
 
    SELECT  
        c.Producto, 
        c.Sub_Producto, 
        c.Frente, 
        c.Profundidad, 
        c.Altura, 
        c.Piso, 
        c.Puertas
    INTO P_PRODUCTO, P_TIPO, P_FRENTE, P_PROFUNDIDAD, P_ALTURA, P_PISO, P_PUERTAS
    FROM producto p INNER JOIN sub_producto sp ON p.id_prod = sp.id_prod 
        INNER JOIN combinaciones c ON sp.id_prod = c.ID_SUB_PROD;
  END;
 

Yo recomendaría no usar parametros y obtener la tabla resultado que devuelve el SP directamente como tal, pero no sé en qué contexto la estás usando.

Lo que estoy haciendo es un SP para conectarlo mediante ASP.NET a una pagina de un cotizador en donde cuando se escoja un producto vaya filtrando los sub_productos que tiene , es por eso que necesito los parametros pero noto que me esta marcando unos errores el SP que me escribiste me esta marcando errores de punto y coma por lo que no deja crear el SP.

Código MySQL:

CREATE PROCEDURE SP_ConsultaCotizador
(
 OUT p_Producto VARCHAR(100),
 OUT p_tipo VARCHAR(100),
 OUT p_Frente VARCHAR(100),
 OUT p_Profundidad VARCHAR(10),
 OUT p_Altura VARCHAR(100),
 OUT p_Piso VARCHAR(100),
 OUT p_Puertas VARCHAR(100),
 OUT p_Modulos VARCHAR(10)
)
BEGIN
 
    SELECT  
        c.Producto, 
        c.Sub_Producto, 
        c.Frente, 
        c.Profundidad, 
        c.Altura, 
        c.Piso, 
        c.Puertas
    INTO p_Producto, p_tipo, p_Frente, p_Profundidad, p_Altura, p_Piso, p_Puertas
    FROM producto p INNER JOIN sub_producto sp ON p.id_prod = sp.id_prod 
        INNER JOIN combinaciones c ON sp.id_prod = c.id_prod; -- AL CERRAR CON ; ES DONDE ME MARCA EL ERROR DE PUNTO Y COMA
  END; -- AQUI TAMBIEN AL CERRAR ME MARCA ERROR DE PUNTO Y COMA
 

[gnzsoloyo]

Empecemos por un punto importante: ¿De qué forma estás intentando crear el SP? ¿Con ASP.Net o lo estás creando directametne con una interfaz (MySQL Workbench, phpMyadmin, consola de MySQL, etc.)?

Esto es imoprtante porque dependiendo de cómo lo cargas será la indicación que te de.

[raul_loz]

Iniciado por gnzsoloyo

Empecemos por un punto importante: ¿De qué forma estás intentando crear el SP? ¿Con ASP.Net o lo estás creando directametne con una interfaz (MySQL Workbench, phpMyadmin, consola de MySQL, etc.)?

Esto es imoprtante porque dependiendo de cómo lo cargas será la indicación que te de.

Lo estoy haciendo con MySQL Workbench.

Saludos.

[gnzsoloyo]

El editor de SP es suficiente para lo que necesitas. Escribe solamente el código, dejando que el Workbench ponga el cierre. No le pongas nada más alla del END que te escribe. NADA.
Concentrate en poner los parametros del prototipo y el body, sin agregados.

El Workbench se encargará del resto.

[raul_loz]

Iniciado por gnzsoloyo

El editor de SP es suficiente para lo que necesitas. Escribe solamente el código, dejando que el Workbench ponga el cierre. No le pongas nada más alla del END que te escribe. NADA.
Concentrate en poner los parametros del prototipo y el body, sin agregados.

El Workbench se encargará del resto.

Estoy deacuerdo con lo que me dices de no escribir nada despues del END el problema esta en el ultimo inner join al finalizar la linea me marca error de punto y coma y por eso no me deja crear el SP, y en la salida del error dice: Error Code: 1064. You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'END' at line 25


CREATE PROCEDURE SP_FERSA_ConsultaCotizacion
(
OUT p_Producto VARCHAR(100),
OUT p_tipo VARCHAR(100),
OUT p_Frente VARCHAR(100),
OUT p_Profundidad VARCHAR(10),
OUT p_Altura VARCHAR(100),
OUT p_Piso VARCHAR(100),
OUT p_Puertas VARCHAR(100),
OUT p_Modulos VARCHAR(10)
)
BEGIN

SELECT
co.Producto,
co.Sub_Producto,
co.Frente,
co.Profundidad,
co.Altura,
co.Piso,
co.Puertas
INTO p_Producto, p_tipo, p_Frente, p_Profundidad, p_Altura, p_Piso, p_Puertas
FROM producto p INNER JOIN sub_producto sp ON p.id_prod = sp.id_prod
INNER JOIN combinaciones co ON sp.id_prod = co.id_prod -- <--Al terminar esta linea me marca el error de punto y coma
END

[raul_loz]

Iniciado por gnzsoloyo

El editor de SP es suficiente para lo que necesitas. Escribe solamente el código, dejando que el Workbench ponga el cierre. No le pongas nada más alla del END que te escribe. NADA.
Concentrate en poner los parametros del prototipo y el body, sin agregados.

El Workbench se encargará del resto.

Ya quedaron resueltos los errores lo hice con el diseñador de Workbench.