Class C# avec requéte SQL qui fonctionne pas

Version imprimable

Bonjour,
Je débute tout juste en programmation orientée objet et j'essaie de faire une class qui me permettra à chaque instanciation de faire une requête d'insertion dans ma base de données.
J'ai bien vérifié si la requête passai directement dans l'interface de gestion de la base de donnée et c'est le cas.
Je ne comprend pas pourquoi ça ne fonctionne pas, merci de m'éclairer.

Voici l'instanciation :

Code:

1
2
3
4
5
6
7
8
9
10
    protected void Button1_Click(object sender, EventArgs e)
    {
        string id = logiciel.SelectedValue;
        int access = 1;
        string titre = TextBoxTitre.Text;
        string comment = TextBoxComment.Text;
        DateTime date = DateTime.Now;
        string requete = "INSERT INTO [articles] ([id_produits_], [id_acces_], [id_titre_traduction_], [id_commentaire_traduction_], [titre], [commentaire], [type_fiche], [type_telechargement], [type_notice], [type_video], [type_faq], [date_publication], [date_maj]) VALUES (1, "+access+", NULL, NULL, '"+titre+"', '"+comment+"', 1, 0, 0, 0, 0, '"+date+"', NULL)";
        InsertionArticle InsertFiche = new InsertionArticle(requete);
    }

Voici ma class :

Code:

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
39
40
41
42
43
44
45
46
47
48
49
using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using System.Configuration;
using System.Data.SqlClient;
using System.Windows.Forms;
 
/// <summary>
/// Description résumée de InsertionArticle
/// </summary>
public class InsertionArticle
{   
    private string _StrConnexion;
    private string _requete;
 
	public InsertionArticle(string requete)
	{
        this._requete = requete;
        string _StrConnexion = ConfigurationManager.ConnectionStrings["supportConnectionString"].ConnectionString; 
	}
 
     public void Insert()
     {
         SqlConnection cx = new SqlConnection(this._StrConnexion);
         try
         {
             cx.Open();
             SqlCommand cmd = new SqlCommand(this._requete, cx);
             cmd.ExecuteNonQuery();
             cx.Close();
             Console.WriteLine("OK");
         }
         catch (Exception err)
         {
             Console.WriteLine(err.Message);
         }
         finally
         {
             cx.Close();
         }
     }
 
    public void Main() 
    {
        Insert();
    }
 
}

Merci d'avance pour votre aide.;)

01/04/2009, 17h20
Shivaneth

Bonjour,

si tu n'appelles pas InsertFiche.Insert() ta requête ne sera pas exécutée
01/04/2009, 17h21
lutecefalco

Et quel est le message d'erreur?
01/04/2009, 17h32
nasimpat7

Je n'ai justement aucun message d'erreur.
La méthode

Code:

public void Main(){...}

si je l'ai bien compris, s'exécute automatiquement à l'intanciation de la class.
Pourquoi devrai-je appeler InsertArticle.Insert() alors la méthode Main est censée se charger de cela. non:??
Je vais essayer d'enlever la méthode Main et d'appeler directement Insert()
01/04/2009, 17h34
lutecefalco

Ta méthode Main n'est pas appelée.
Tu confonds le constructeur d'une classe avec le Main d'un programme console par ex

J'ai alors fait quelques modifications

Mon instanciation

Code:

1
2
3
4
5
6
7
8
9
10
11
    protected void Button1_Click(object sender, EventArgs e)
    {
        string id = logiciel.SelectedValue;
        int access = 1;
        string titre = TextBoxTitre.Text;
        string comment = TextBoxComment.Text;
        DateTime date = DateTime.Now;
        string requete = "INSERT INTO [articles] ([id_produits_], [id_acces_], [id_titre_traduction_], [id_commentaire_traduction_], [titre], [commentaire], [type_fiche], [type_telechargement], [type_notice], [type_video], [type_faq], [date_publication], [date_maj]) VALUES (1, "+access+", NULL, NULL, '"+titre+"', '"+comment+"', 1, 0, 0, 0, 0, '"+date+"', NULL)";
        InsertionArticle InsertFiche = new InsertionArticle(requete);
        InsertFiche.Insert();
    }

et ma class

Code:

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
39
40
41
42
43
44
using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using System.Configuration;
using System.Data.SqlClient;
using System.Windows.Forms;
 
/// <summary>
/// Description résumée de InsertionArticle
/// </summary>
public class InsertionArticle
{   
    private string _StrConnexion;
    private string _requete;
 
	public InsertionArticle(string requete)
	{
        this._requete = requete;
        string _StrConnexion = ConfigurationManager.ConnectionStrings["supportConnectionString"].ConnectionString; 
	}
 
     public void Insert()
     {
         SqlConnection cx = new SqlConnection(this._StrConnexion);
         try
         {
             cx.Open();
             SqlCommand cmd = new SqlCommand(this._requete, cx);
             cmd.ExecuteNonQuery();
             cx.Close();
             Console.WriteLine("OK");
         }
         catch (Exception err)
         {
             Console.WriteLine(err.Message);
         }
         finally
         {
             cx.Close();
         }
     }
 
}

Mais malheuresement cela ne fonctionne toujours pas.:(

Salut,

Plutôt
Code:

1 2 3 4 5 6 7 8 private string _StrConnexion; private string _requete; public InsertionArticle(string requete) { this._requete = requete; this._StrConnexion = ConfigurationManager.ConnectionStrings["supportConnectionString"].ConnectionString; }
Si tu fais de l'ASP il ne faut pas utiliser "Console.WriteLine":

Code:

Console.WriteLine(err.Message);

mais

Code:

Response.Write(err.Message);

A+

01/04/2009, 19h35
AJemni

Salut,
peut être que le pb provient de ta requête.
a tu vérifie la compatibilité entre les types de données?
donne nous le script de création de la table articles
01/04/2009, 20h01
nasimpat7

Le problème est résolu, merci à tous et particulièrement à toi Immobilis pour ton oeil de lynx. Erreur bette de ma part, c'était un this._StrConnexion et non pas un string _StrConnexion qu'il fallait mettre. Ce qui est logique car private string _StrConnexion restait à null. Les requêtes se font bien mais pas de message dans la console? J'utilise Firebug.
Merci encore à tous.:king:
01/04/2009, 20h25
Immobilis

Citation:

mais pas de message dans la console

NOrmal, comme je te l'ai dit tu es en web la console n'est pas disponible. Je me demande pourquoi ça compile d'ailleurs.

A+
01/04/2009, 20h35
nasimpat7

J'avais bien remplacé les Console.WriteLine() par Console.Write() comme tu me l'avais conseillé (j'ai oublié de le préciser). Toujours pas de message dans la console de Firebug mais bon c'est pas très important quoique ça m'intrigue quand même.
Je précise aussi que je développe sous .net Framework 3.5 SP1.
01/04/2009, 21h26
Immobilis

RRaaahhh

Citation:

Envoyé par Immobilis

Si tu fais de l'ASP il ne faut pas utiliser "Console.WriteLine":

Code:

Console.WriteLine(err.Message);

mais

Code:

Response.Write(err.Message);

A+

:arf:

;)
01/04/2009, 23h32
nasimpat7

Désolé et merci encore Immobilis.:mouarf:
Response.Write() ne passe pas dans une class (erreur de compilation).

Seul Consol.Write() ou Console.WriteLine() avec lesquels ça compile => équivalent je pense à Console.log() en JavaScript (Je me trompe peut être).
De toute façon c'est pas grave, le principal est que les requêtes passent, je trouverai bien un moyen de retourner un message.

A+;)
01/04/2009, 23h58
Immobilis

Citation:

Envoyé par nasimpat7

Response.Write() ne passe pas dans une class (erreur de compilation).

Il faut importer la référence system.web.

Après c'est pas forcement logique de faire un response.write ou console.write en dehors d'une interface homme machine. Parce que ainsi, tu couple fortement ta clase avec l'IHM. Dans ce cas tu ne pourras pas l'utiliser en dehors de cette IHM. Il faut éventuellement utiliser la trace ou bien que la méthode renvoit une chaine ou bien passer une variable par référence ou encore utiliser une propriété assignée au cours du processus.

A+