Discussion :

[drissi]

Salut à tous,
J'ai une erreur de compilation bizarre,
Voila mon code:

Code :

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

1
2
Execute_update_insert_delete Exec_proc = New Execute_update_insert_delete();
Exec_proc.Execute(Stringreq, Prenom_TextBox.Text.ToString(), Nom_TextBox.Text.ToString(), v_sexe, v_date_naissance, Pays_TextBox.Text.ToString(), GetMD5(Passe_TextBox.Text.ToString()), Email_TextBox.Text.ToString(), DateTime.Now, Adresse_Textbox.Text.ToString());

Ma classe je l'ai crée dans app_code:

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
using System;
using System.Collections;
using System.Configuration;
using System.Data.SqlClient;
using System.Security.Cryptography;
using System.Text;
using System.Text.RegularExpressions;
using System.Web.Mail;
using System.Windows.Forms;
 
/// <summary>
/// Summary description for Execute_update_insert_delete
/// </summary>
public class Execute_update_insert_delete
{
    public int Execute(string requeteString, string p_prenom, string p_nom, string p_sexe, DateTime p_date_naissance, string p_pays, string p_passe, string p_email, DateTime p_date_creation, string p_adresse)
    {
        var connString = ConfigurationManager.ConnectionStrings["XXOA_MAILS_ConnectionString"].ConnectionString;
        SqlConnection conn = new SqlConnection(connString);
        conn.Open();
        SqlCommand cmd = new SqlCommand(requeteString, conn);
        SqlParameter param = new SqlParameter();
        cmd.Parameters.Add(new SqlParameter("@PRENOM", p_prenom));
        cmd.Parameters.Add(new SqlParameter("@NOM", p_nom));
        cmd.Parameters.Add(new SqlParameter("@SEXE", p_sexe));
        cmd.Parameters.Add(new SqlParameter("@DATE_NAISSANCE", p_date_naissance));
        cmd.Parameters.Add(new SqlParameter("@PAYS", p_pays));
        cmd.Parameters.Add(new SqlParameter("@PASSE", p_passe));
        cmd.Parameters.Add(new SqlParameter("@EMAIL", p_email));
        cmd.Parameters.Add(new SqlParameter("@DATE_CREATION", p_date_creation));
        cmd.Parameters.Add(new SqlParameter("@ADRESSE", p_adresse));
        int countLigne = 0;
        countLigne = cmd.ExecuteNonQuery();
        return countLigne;
        conn.Close();
    }
}

L'erreur est souligné avec Execute_update_insert_delete comme dans le premier code avec un affichage " ';' attendu."

[calagan99]

Je ne sais pas d'où vient ton erreur, mais en tout cas une chose est sure : tu ne fermes jamais ta connexion puisque tu fais

Code :

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

conn.Close();

APRES ton return.

[drissi]

Je n'ai jamais clôturé ma connexion, mais un consultant Microsoft nous a recommandé ça pour ne pas encombré la trafic réseau de la base de donnée...
Donc ils nous a dit soit, fermer la connexion soit, utiliser using { }...

Voici une capture d'écran en pj

[calagan99]

Plusieurs choses :
- un consultant Microsoft te recommandera plutôt de fermer ta connexion avant de faire un return. Parce que là, tu peux supprimer ta dernière ligne, elle ne sert à rien.
- En effet, une clause Using est probablement la plus adaptée.

Sinon :
- as-tu un ";" à la fin de la ligne dans laquelle tu construis ta requête ?

D'autre part :
- MessageBox ? En mode web ? Ca, c'est une vilaine bidouille...
- Comme te l'indique VisualStudio, "System.Web.Mail" est déprécié. Utilise plutôt "System.Net".

[Aeronia]

Bonjour,
bizarrement, je ne vois aucun constructeurs dans ta classe.

Un oubli lors du c/c du code dans ton post ?

[kheironn]

A défaut de using, c'est typiquement le genre de code où un Try/Finally est indiqué, avec le close() dans le finally... C'est un minimum.

@Aeronia : il doit utiliser le constructeur par défaut hérité d'Object.

Vu qu'on ne le voit pas : y-a-t'il un point-virgule à la fin de ta ligne de déclaration de stringreq ?