Esta bien planteado este procedure? Se podria con triggers?

[franroa]

Tengo dos tablas relacionadas con otra entidad que tiene las claves foraneas de cada una: files y portals asociadas por portalfiles. Lque quiero es que al subir un archivo, se meta automaticamente en el portal que le pida (el procedure funciona perfectamente). Mi pregunta es si esta bien planteado así, me da la sensación de que estoy forzando a la base de datos. Pero no he encontrado otra manera.


La verdad es que tambien queria saber si esto que hago se puede con triggers, pero un trigger no admite el parametro "portalID" para saber a que portal asociar...

Muchas gracias!

Código SQL:

DELIMITER $$
 
CREATE PROCEDURE subeArchivoPortal (
 
IN Name VARCHAR(60),
 
IN Description  VARCHAR(50),
 
IN Path VARCHAR(60),
 
IN TYPE VARCHAR(15),
 
IN Premium VARCHAR(5),
 
IN Uploader INT,
 
IN Autor VARCHAR(50),
 
IN Url VARCHAR (5),
 
IN Level  VARCHAR(10),
 
IN Portal INT
 
) BEGIN
 
 
DECLARE idFile INT DEFAULT 0;
 
 
                SELECT AUTO_INCREMENT FROM information_schema.TABLES WHERE TABLE_SCHEMA = 'dbXXXXXXXXXX' AND TABLE_NAME = 'files' INTO idFile;
 
 
INSERT INTO files  (fName ,fDescription ,fType,fDate, fPath ,fUploader,fAutor,fLevel,fUrl,fPremium) VALUES(Name ,Description ,TYPE,CURDATE(), Path ,Uploader,Autor,Level,Url,Premium);
 
 
INSERT INTO portalfiles (pfPortalID, pfFileID ) VALUES (Portal, idFile);
END$$
DELIMITER ;
 

[gnzsoloyo]

¿Bien planteado?
No.

El auto_increment, usado de esa forma repetiría valores, porque ese campo contiene el último valor generado en un un INSERT sobre la tabla. Si lo lees sin afectarlo simplemente tomas el mismo valor para eso.

Por otro lado, hacer eso es total y absolutamente innecesario en el contexto que lo usas, ya que para eso existe una función específica (el manual de referencia tiene buenas explicaciones de este tipo de cosas).

Además estás usando palabras reservadas como nombres de variable en el prototipo, por lo que el resultado de esto es incierto, si es que compila.

Lo que necesitarías sería mas o menos:

Código MySQL:

DELIMITER $$
 
CREATE PROCEDURE subeArchivoPortal (
    IN vName VARCHAR(60),
    IN vDescription  VARCHAR(50),
    IN vPath VARCHAR(60),
    IN vType VARCHAR(15),
    IN vPremium VARCHAR(5),
    IN vUploader INT,
    IN vAutor varchar(50),
    IN vUrl VARCHAR (5),
    IN vLevel  VARCHAR(10),
    IN vPortal INT
    ) 
BEGIN
    DECLARE idFile INT DEFAULT 0;
    
    INSERT INTO files (fName, fDescription, fType, fDate, fPath, fUploader, fAutor, fLevel, fUrl, fPremium) 
    VALUES(vName, vDescription, vType, CURDATE(), vPath, vUploader, vAutor, vLevel, vUrl, vPremium);
    
    SET idFiles = LAST_INSERT_ID();
    
    INSERT INTO portalfiles (pfPortalID, pfFileID ) 
    VALUES (vPortal, idFile);
END$$
DELIMITER ;