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 ?

[Aeronia]

Je suis d'accord kheironn.
C'est peut être un tort de ma part, mais j'ai toujours pris pour habitude de mettre un constructeur par défaut dans toutes mes classes

Ça rend pour moi le code plus lisible ...

Si c'était un problème de ; dans le string de la requête, ne serait plutôt pas un problème à l’exécution plutôt qu'à la compilation ?

[kheironn]

Si c'était un problème de ; dans le string de la requête, ne serait plutôt pas un problème à l’exécution plutôt qu'à la compilation ?

Je ne sais pas, je n'ai jamais eu cette erreur... je suis parti dans les suppositions sur une connerie à 2 balles...

[binoo]

Salut,

Une remarque peut-être bête, mais si tu double cliques sur ta ligne d'erreur, ça doit t'envoyer vers la ligne en question soit la ligne 334 (à en voir ton imprim écran) et sur cette ligne, tu dois avoir oublié un ';'

[Nathanael Marchand]

Je viens de voir ca:
Depuis quand new possède une majuscule?

[calagan99]

Je viens de voir ca:
Depuis quand new possède une majuscule?

Punaise, tu as l'oeil aujourd'hui

[Nathanael Marchand]

Punaise, tu as l'oeil aujourd'hui

Toujours! L'habitude du boulot

[Aeronia]

Comme quoi on va toujours regarder les détails alors que c'est souvent un truc simple

[drissi]

Olala, bravo à Nathanael...
J'ai loupé ceci...
Merci à toi... dire qu'en développement, il faut toujours avoir scanner de haut technologie...
Sinon pour revenir à vos remarques, @binoo en cliquant sur la ligne d'erreur, on m’envoie vers la classe souligné comme indiqué dans mon premier post...
@calagan99, exactement, je me suis trompé en mettant close() après le return...
D'autre part, quel est l'alternative de MessagBox() dans ce cas.
Merci à vous tous...

[kheironn]

Je viens de voir ca:
Depuis quand new possède une majuscule?

depuis aujourd'hui, 12h23 et un peu avant...

J'aurais jamais imaginé avoir ce messae d'erreur pour cette erreur... du coup, ça ne me parait pas clair.

[Bluedeep]

J'aurais jamais imaginé avoir ce messae d'erreur pour cette erreur... du coup, ça ne me parait pas clair.

C'est assez logique : le compilateur tombe sur un mot non réservé, non suivi d'une parenthèse indiquant qu'il s'agit d'un appel à une méthode, donc il le considére comme une variable; avant d'aller voir si il peut résoudre cette variable, il parse l'instruction et ne trouve pas de ";".

[kheironn]

C'est assez logique : le compilateur tombe sur un mot non réservé, non suivi d'une parenthèse indiquant qu'il s'agit d'un appel à une méthode, donc il le considére comme une variable; avant d'aller voir si il peut résoudre cette variable, il parse l'instruction et ne trouve pas de ";".

En même temps, je n'ai jamais eu cette erreur... Mais merci quand même... et effectivement en y refléchissant, c'est logique J'aurais mieux fait de refléchir avant de l'ouvrir. Ca m'apprendra !

[drissi]

Sinon, dites moi, pourquoi le fait d'utiliser MessageBox en mode Web est n'importe quoi?

[Bluedeep]

Sinon, dites moi, pourquoi le fait d'utiliser MessageBox en mode Web est n'importe quoi?

Par ce que afficher une MessageBox sur le serveur, si ça c'est pas "n'importe quoi", il faudra qu'on m'explique ce que c'est.

[Nathanael Marchand]

Par ce que afficher une MessageBox sur le serveur, si ça c'est pas "n'importe quoi", il faudra qu'on m'explique ce que c'est.

En effet, le compte qui fait tourner un serveur web n'est pas une personne réelle. Ainsi, ouvrir une popup, n'a pas de sens puisque soit elle ne s'ouvrira pas soit elle s'ouvrira sur un espace que personne ne maitrise (une popup perdue dans les limbes )