procédure stockée avec if else sql server 2008

J'ai 3 tables, je dois vérifier si les grands-parents de la table id a des dossiers dans les petits-enfants de la table. Si oui, de retour oui, else return no. Voici ma procédure stockée. J'ai eu une mauvaise erreur de syntaxe. Je suis nouvelle procédure stockée. S'il vous plaît aider moi.

CREATE PROCEDURE P_Check
   @PKG_ID INT,
   @S_ID INT,
   @FLAG VCHAR(10) OUT

   DECLARE IDS CURSOR LOCAL FOR SELECT S_ID FROM T1 WHERE P_ID = @PKG_ID
   OPEN IDS
   FETCH NEXT FROM IDS into @S_ID

   WHILE @@FETCH_STATUS = 0
   BEGIN
      SELECT * FROM T2 WHERE S_ID = @S_ID

      IF @@ROWCOUNT<>0
         @FLAG = 'YES'
         RETURN 
      ELSE
         @FLAG = 'NO'

      FETCH NEXT FROM IDS into @S_ID
   END

   CLOSE IDS
   DEALLOCATE IDS

OriginalL'auteur jhon smith | 2013-04-11

  1. 4

    Quelques points à vérifier:

    1. Je ne pense pas qu'il y est un vchar type de données dans SQL Server, sauf si c'est votre type personnalisé. Donc le changer pour varchar
    2. Vous avez oublié le AS
    3. Vous pourriez vouloir joindre votre logique à l'intérieur de if entre begin et end

    Votre code peut être compilé:

    CREATE PROCEDURE P_Check
    @PKG_ID INT,
    @S_ID INT,
    @FLAG VARCHAR(10) OUT
AS
    DECLARE IDS CURSOR LOCAL FOR SELECT S_ID FROM T1 WHERE P_ID = @PKG_ID
    OPEN IDS
    FETCH NEXT FROM IDS into @S_ID
    WHILE @@FETCH_STATUS = 0

    BEGIN
      SELECT * FROM T2 WHERE S_ID = @S_ID
      IF @@ROWCOUNT<>0
      BEGIN
         SET @FLAG = 'YES'
         RETURN 
      END
      ELSE
      BEGIN
         SET @FLAG = 'NO'
         FETCH NEXT FROM IDS into @S_ID
      END
    END

    CLOSE IDS
    DEALLOCATE IDS

    Cependant, je pense que votre curseur ne sera pas fermé comme vous êtes de retour ici IF @@ROWCOUNT<>0. Je pense que ce que vous avez à faire est de changer cela:

    IF @@ROWCOUNT<>0
BEGIN
    SET @FLAG = 'YES'
    RETURN 
END

    :

    IF @@ROWCOUNT<>0
BEGIN
    SET @FLAG = 'YES'
    GOTO ON_EXIT
END

    puis à la fin de votre procédure comme ceci:

    ON_EXIT:
    CLOSE IDS
    DEALLOCATE IDS

    Alors pour répondre à votre question dans les commentaires, vous êtes "le retour" déjà dans un sens. Vous pouvez appeler et de tester votre procédure est comme ceci:

    declare @result varchar(10)
exec P_Check 1, 1, @result out
print @result
    Merci, ça fonctionne! comment puis-je retourner @drapeau que oui ou non?
    Vous êtes les bienvenus. Voir ma réponse mis à jour.
    Est la même pouvez obtenir par le biais de juste une jointure entre t1 et t2 par correspondance S_ID?
    Je vous suggère d'aller avec @marc_s de réponse. Votre question était à l'origine une erreur de syntaxe, et de l'optimisation, c'est autre chose. Marc_s réponse est une bonne alternative.

    OriginalL'auteur von v.

  2. 3

    C'est juste trop compliqué et à l'aide d'un curseur ici est totalement inutile et totalement inutile.

    Simplifier votre procédure:

    CREATE PROCEDURE P_Check
   @PKG_ID INT,
   @S_ID INT,
   @FLAG CHAR(1) OUT
AS BEGIN
   IF EXISTS (SELECT * FROM T2 
              INNER JOIN T1 ON T2.S_ID = T1.S_ID WHERE P_ID = @PKG_ID)
      SET @FLAG = 'Y'
   ELSE
      SET @FLAG = 'N'
END

    Lorsque l'on travaille sérieusement avec SQL Server, vous avez besoin de sortir de la procédure ligne-par-atroce-ligne la pensée de l'utilisation de curseurs et de boucles, et vous avez besoin pour commencer de la pensée dans des ensembles pour être efficace et productif.

    OriginalL'auteur marc_s