Discussion :

[flogreg]

Bonjour,

J'ai déjà créé ce sujet sur le forum sql server mais on m'a dit que les procédures étaient correctes donc je me permet de venir poster ici. http://www.developpez.net/forums/sho...d.php?t=592228

J'ai cette procédure stocké :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
CREATE PROCEDURE [dbo].[proc1]
As
BEGIN
     insert into matable(champs1) values('pouet')
     select @@identity as resultat1
END

maintenant, j'ai une deuxieme procédure qui appelle la premier :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
CREATE PROCEDURE [dbo].[proc2]
As
BEGIN
     exec proc1
     insert into matable2(champs1) values('pouet2')
     select @@identity as resultat2
END

Mon problème est que quand j'appelle proc2, je recois en retour le résultat de proc1 et je ne veux récupérer que le résultat de proc2.

Voici la partie C#

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
                string connectString = ConfigurationManager.ConnectionStrings["test"].ToString();
                SqlConnection conn = new SqlConnection(connectString);
                conn.Open();
 
                SqlCommand cmd = new SqlCommand("proc2", conn);
                cmd.CommandType = CommandType.StoredProcedure;
                SqlDataReader r = cmd.ExecuteReader();
 
                if (r.Read())
                {
                    int test = r.GetInt32(0);
                }
                r.Close();
                conn.Close();

Si quelqu'un pouvait m'expliquer ce qui ne va pas. Merci d'avance.

[elbj]

Bonjour

Lorsque tu executes tes SP sur le Query Analyser ou SQL Server Management Studio l'identity retourné est-il le bon ?

Sinon, au lieu de passer par un Reader, fait un ExecuteScalar.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
string connectString = ConfigurationManager.ConnectionStrings["test"].ToString();
SqlConnection conn = new SqlConnection(connectString);
SqlCommand cmd = new SqlCommand("proc2", conn);
cmd.CommandType = CommandType.StoredProcedure;
object resultat = null;
try
{
  cmd.Connection.Open();
  resultat = cmd.ExecuteScalar();
}
catch (Exception ex)
{
  throw ex;
}
finally
{
  cmd.Connection.Close();
}
return resultat;

Cordialement

[flogreg]

Bonjour,

Je viens de tester avec la méthode executeScalar(). J'obtiens le résultat de proc1 au lieu d'obtenir le résultat de proc2...


Si j'exécute dans management studio, voici le résultat en piece jointe

[elbj]

OK !
Voilà comment j'interprête le problème. La requête te retourne 2 résultats mais ton code ne prend que le premier.

La solution que je te conseille est de ne pas imbriquer ces SP et de les appeler successivement dans ton code client.

Cordialement

[flogreg]

non elle ne me retourne pas les 2 résultats, elle ne m'en renvoit qu'un seul.
J'ai testé avec un while(r.Read()) et je n'ai qu'une seule valeur.

[Pol63]

c'est normal, studio management affiche successivement les résultats des différentes requetes passée
alors qu'un reader ne retourne que le dernier select

c'est comme ca, tu ne peux rien y faire, à part modifier ce que tu fais

[flogreg]

c'est normal, studio management affiche successivement les résultats des différentes requetes passée
alors qu'un reader ne retourne que le dernier select

c'est comme ca, tu ne peux rien y faire, à part modifier ce que tu fais

justement ! j'aimerais qu'il me renvois le dernier select. La il me renvoit le select de la procédure appelé dans la procédure.

[Pol63]

oui enfin je voulais dire le premier je pense

[flogreg]

y aurait il un moyen pour quel l'exec proc1 ne retourne pas de résultat ?

[elbj]

Retirer le select @@identity qui s'y trouve OU ne pas imbriquer ces SP et de les appeler successivement dans ton code client.

[Pol63]

ou alors mettre un paramètre booléen en entrée qui permet de dire si tu veux ou non que proc1 retourne son résultat (via un if)

[SaumonAgile]

Essaie avec un :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
while (reader.NextResult())
{
    while (reader.Read())
    {
        ...
    }
}

Tu verras déjà s'il te renvoie un ou deux resultsets.

[fmoreau73]

Sinon, il suffit de passer au résultat suivants dans ton objet SqlDataReader après la première lecture :

r.NextResult();

Edit : Zut, SaumonAgile m'a grillé ...

[Pol63]

si ca fait ce qu'il veut, c'est bien pratique !
meme pour les compute d'une requete

[flogreg]

Essaie avec un :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
while (reader.NextResult())
{
    while (reader.Read())
    {
        ...
    }
}

Tu verras déjà s'il te renvoie un ou deux resultsets.

Effectivement, il me renvoit 2 resultsets... Mais si ma proc 2 appelle plusieurs fois proc1 comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
CREATE PROCEDURE [dbo].[proc2]
As
BEGIN
     exec proc1
     exec proc1
     exec proc1
     exec proc1
     insert into matable2(champs1) values('pouet2')
     select @@identity as resultat2
END

Comment puis-je savoir quel resultset utiliser ? Il n'y a pas de méthode LastResult()

ou alors mettre un paramètre booléen en entrée qui permet de dire si tu veux ou non que proc1 retourne son résultat (via un if)

Effectivement cette solution pourrait convenir, mais je la trouve bien bourrin ! Je pensais que ce genre de problématique se posait à pas mal de monde et qu'il y avait une méthode pour ce genre de chose. Mais j'en demande peut etre trop

[Pol63]

je trouve pas ca bourrin


et puis tu peux aussi demander ton identity autre part que dans ta procédure stockée, il n'y a pas que @@identity, y a des fonctions ou tu peux passer le nom de la table je crois ou scopeidentity est ptete bien aussi

[flogreg]

Ca y est ! J'ai trouvé ! Merci le DBA

En fait, il suffit de définir un paramètre de type output dans proc1 comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
ALTER PROCEDURE [dbo].[proc1]
@test  int output 
AS
BEGIN
     INSERT INTO matable(champs1) VALUES('pouet')
     SELECT @test = @@identity
END

puis dans proc2 :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
ALTER PROCEDURE [dbo].[proc2]
AS
BEGIN
    declare @t int;
    exec proc1 @t output
    INSERT INTO matable2(champs2) VALUES('pouet2')
    SELECT @@identity AS resultat2
END

Merci beaucoup pour votre aide