Mysql procédure Stockée à l'aide du curseur

Il n'y a pas de THEN ni END IF mots-clés, la procédure ne peut pas compiler.

Suivez ce lien pour la syntaxe correcte de IF déclaration: http://dev.mysql.com/doc/refman/5.7/en/if.html

Utilisation EXIST opérateur au lieu de (SELECT count(*)... ) <=0,

lire ce lien pour connaître la raison: http://sqlblog.com/blogs/andrew_kelly/archive/2007/12/15/exists-vs-count-the-battle-never-ends.aspx

IF    EXISTS(  
               SELECT null FROM table1 
               WHERE IndentityNo=IdentityNo1 
                 AND clientNo=@clientNo
      )
THEN
              INSERT INTO table1 (Name,IdentityNO) VALUES (name,IdentityNO1);
              INSERT INTO table2 (Address,City) VALUES(address,city);
ELSE
              INSERT INTO tblexceptional(Name,Address,City,IdentityNo)
              VALUES(name,address,city,IdentityNo1);
END IF;

Je recommande l'utilisation de certains préfixes pour des arguments de procédure et les noms de variables pour éviter toute ambiguïté, par exemple, utiliser p_ pour les paramètres et les v_ pour les variables. Il est difficile de deviner, en regardant ce code, dont le nom est le nom d'une colonne, d'une variable ou d'un paramètre de la procédure. Cela peut conduire à des erreurs et des erreurs.

Évitez d'utiliser des SELECT * - ce code suivant ne fonctionnera pas si quelqu'un va modifier la structure de la table. Explicitement la liste des colonnes requises dans la déclaration de curseur:

declare cur1 cursor for 
        select name,Address,City,IdentityNO,clientNo
        from temptable;

Le corrigé procédure pourrait ressembler à ceci:

CREATE PROCEDURE `insertData`(In p_clientNo INT,In p_usedID INT)
BEGIN

declare v_name varchar(100);
declare v_address varchar(100);
declare v_city varchar(50);
declare v_IdentityNO varchar(20)
declare v_clientNo int


declare cur1 cursor for 
        select name,Address,City,IdentityNO,clientNo
        from temptable;
declare continue handler for not found set done=1;

    set done = 0;
    open cur1;
    igmLoop: loop
        fetch cur1 into v_name,v_Address,v_City,v_IdentityNO,v_clientNo;
        if done = 1 then leave igmLoop; end if;

        //If no record exists in some records table1,table2.
        IF  EXISTS(  SELECT 1 FROM table1 
                     WHERE IndentityNo = v_IdentityNo 
                       AND clientNo = v_clientNo)
              INSERT INTO table1 (Name,IdentityNO) VALUES (v_name,v_IdentityNO);
              INSERT INTO table2 (Address,City) VALUES(v_address,v_city);
        ELSE
              INSERT INTO tblexceptional(Name,Address,City,IdentityNo)
              VALUES(v_name,v_address,v_city,v_IdentityNo);
        END IF;
    end loop igmLoop;
    close cur1;
END