Discussion :

[nasimpat7]

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 :

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

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 :

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
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.

[Shivaneth]

Bonjour,

si tu n'appelles pas InsertFiche.Insert() ta requête ne sera pas exécutée

[lutecefalco]

Et quel est le message d'erreur?

[nasimpat7]

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

Code :

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

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()

[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

[nasimpat7]

J'ai alors fait quelques modifications

Mon instanciation

Code :

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

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 :

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
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.

[Immobilis]

Salut,

Plutôt

Code :

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

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 :

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

Console.WriteLine(err.Message);

mais

Code :

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

Response.Write(err.Message);

A+

[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

[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.

[Immobilis]

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+

[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.

[Immobilis]

RRaaahhh

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

Code :

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

Console.WriteLine(err.Message);

mais

Code :

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

Response.Write(err.Message);

A+

[nasimpat7]

Désolé et merci encore Immobilis.
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+

[Immobilis]

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+