Discussion :

[tito55]

Bonjour,

Je veux insérer une ligne avec les paramètres en entrées et un paramètre qui sera la clé au retour.

Donc voici ma proc en SQL Server 2005

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
 
BEGIN 
    SET NOCOUNT ON 
    INSERT INTO TESTSCOPE 
    ( 
        NOM,
        PRENOM 
    ) 
    VALUES 
    ( 
        @param1,
        @param2
    ) 
    SELECT NEWID = SCOPE_IDENTITY() 
END

Voici mon code vb.net

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
20
21
22
23
24
25
26
27
28
29
30
31
 
Dim connectString As String = "Data Source=XXX;Initial Catalog=MaBd;User ID=MonUser;Password=MyPWD"
 
Dim connection As SqlConnection = New SqlConnection(connectString)
        Try
 
            connection.Open()
            Dim cmd As SqlCommand = New SqlCommand()
            cmd.CommandType = CommandType.StoredProcedure
            cmd.CommandText = "ProcidTest"
            With cmd.Parameters
                .Add(New SqlParameter("@Param1", TextBox2.Text))
                .Add(New SqlParameter("@Param2", TextBox1.Text))
                .Add(New SqlParameter("@Lastid", SqlDbType.Int))
            End With
            cmd.Parameters("@Param1").Direction = ParameterDirection.Input
            cmd.Parameters("@Param2").Direction = ParameterDirection.Input
            cmd.Parameters("@Lastid").Direction = ParameterDirection.ReturnValue
 
            cmd.Connection = connection
 
            cmd.ExecuteNonQuery()
            TextBox3.Text = cmd.Parameters("@Lastid").Value
 
        Catch ex As Exception
 
        Finally
            connection.Close()
            connection.Dispose()
            connection = Nothing
        End Try

Mon problème est que je ne reçois jamais la valeur du identity dans TextBox3.Text du paramètre @Lastid.

Quelqu'un peut m'aider ?

Merci d'avance

[narutobaka]

Bonjour,

Je pense qu'il te manque un curseur pour lire la réponse de ta requête.

Je ne sais pas où tu crées ton paramètre @last.

essaye cela :

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
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
Dim connectString As String = "Data Source=XXX;Initial Catalog=MaBd;User ID=MonUser;Password=MyPWD"
 
Dim connection As SqlConnection = New SqlConnection(connectString)
        Try
 
            connection.Open()
            Dim cmd As SqlCommand = New SqlCommand()
            cmd.CommandType = CommandType.StoredProcedure
            cmd.CommandText = "ProcidTest"
            With cmd.Parameters
                .Add(New SqlParameter("@Param1", TextBox2.Text))
                .Add(New SqlParameter("@Param2", TextBox1.Text))
                .Add(New SqlParameter("@Lastid", SqlDbType.Int))
            End With
            cmd.Parameters("@Param1").Direction = ParameterDirection.Input
            cmd.Parameters("@Param2").Direction = ParameterDirection.Input
            cmd.Parameters("@Lastid").Direction = ParameterDirection.ReturnValue
 
            cmd.Connection = connection
 
dim curseur as SqlDataReader = cmd.executeReader()
 
If curseur.read() then
TextBox3.Text = (curseur.GetInt32(0).tostring())
end if
 
curseur.close()
 
            'cmd.ExecuteNonQuery()
            'TextBox3.Text = cmd.Parameters("@Lastid").Value
 
        Catch ex As Exception
 
        Finally
            connection.Close()
            connection.Dispose()
            connection = Nothing
        End Try

[cybermaxs]

Salut,

Il faut que tu fasses un return avec l'élément dans la procédure stockée sinon tu ne recevra rien. Ne pas confondre valeur de retour (Return) et valeur de sortie (OUTPUT).

[Pol63]

le return n'est pas obligatoire, le plus simple est de finir par un select comme c'est fait et dans ce faire executereader ou executescalar à la place de executenonquery

[cybermaxs]

le return n'est pas obligatoire, le plus simple est de finir par un select comme c'est fait et dans ce faire executereader ou executescalar à la place de executenonquery

et vis à vis de la déclaration de ces paramètres ?
sinon je suis d'accord autant faire un executescalar, c'est tout de même fait pour ça.

[Pol63]

oui j'avais pas regardé tout le code

soit finir par un select / executescalar et ne pas déclarer de parametre pour l'identity créé

soit déclarer le paramètre de sortie avec le mot clé out
on ne voit pas la déclaration des paramètres dans la procédure stockée donc on ne peut pas dire si c'est là qu'il a fait une erreur (surement le cas, on ne voit d'autre paramètre @ dans le code)
et dans ce cas un executenonquery fonctionne sans qu'il y ait un select dans la SP (un SET suffit)

et le parameters dans .net doit etre de type inputouput je crois

m'enfin avec les paramètres c'est quand même moins simple à écrire