Discussion :

[yassin123441]

salut tt le monde ,

j ai essai de récupéré une valeur de la table du base de donnée et la mettre dans un autre table et j ai eu cette Erreur Un DataReader associé à cette Command est déjà ouvert. Il doit d'abord être fermé.
voile le 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
38
39
40
41
42
43
44
{
            int idfichier = 0;
            int rows = e.RowIndex;
            string urlfichier;
 
            urlfichier = this.dataGridViewdossier[1, rows].Value.ToString();
 
 
            tempsProcessus pp = new tempsProcessus();
            pp.PrintDoc(urlfichier);
            SqlCommand verif = new SqlCommand("SELECT id FROM dossier where nom ='" + urlfichier + "' ", con);
 
                SqlDataReader r = verif.ExecuteReader();
                while (r.Read())
                {
                    idfichier = (int)r["id"];
                }
 
            MessageBox.Show(idfichier.ToString());
            DateTime x = pp.getProcessStartTime();
            DateTime y = pp.getProcessEndTime();
            TimeSpan xy = x - y;
            SqlCommand sqlComm = new SqlCommand();
            sqlComm = con.CreateCommand();
 
            sqlComm.CommandText = @"INSERT INTO compture (dateouverture,datefermeture,tempstravail,id_personne,id_ficher) VALUES (@x,@y,@xy,@idpersonne,@idfichier)";
 
            sqlComm.Parameters.Add("@x", SqlDbType.DateTime);
            sqlComm.Parameters.Add("@y", SqlDbType.DateTime);
            sqlComm.Parameters.Add("@xy", SqlDbType.DateTime);
            sqlComm.Parameters.Add("@idpersonne", SqlDbType.Int);
            sqlComm.Parameters.Add("@idfichier", SqlDbType.Int);
 
            sqlComm.Parameters["@x"].Value = x;
            sqlComm.Parameters["@y"].Value = y;
            sqlComm.Parameters["@xy"].Value = xy;
            sqlComm.Parameters["@idpersonne"].Value = idpersonne;
            sqlComm.Parameters["@idfichier"].Value = idfichier;
 
 
            sqlComm.ExecuteNonQuery();
 
 
        }

merci d avance

[kheironn]

L'erreur se produit sur

Code :

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

sqlComm.ExecuteNonQuery();

?

Il faut savoir que dont Reader est conosmmé lorsqu'il est lu (méthode Read()) et donc, arrivé en bout de course, tu te trouves à la dernière ligne... et pas possible de le réutiliser.

Par contre, je ne comprends pas :

Code :

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

1
2
3
4
while (r.Read())
{
    idfichier = (int)r["id"];
}

tu vas systématiquement utiliser le dernier retourné !
Si ta requête ne retourne qu'une ligne, alors inutile de faire un while !!!

Code :

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

1
2
If(r.Read())
    idfichier = (int)r["id"];

Ensuite tu fais

Code :

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

SqlCommand verif = new SqlCommand("SELECT id FROM dossier where nom ='" + urlfichier + "' ", con);

, c'est mal !
Utilise ici aussi une requête paramétrée comme tu as fais en dessous.

Enfin,

Code :

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

tempsProcessus pp = new tempsProcessus();

Il est d'usage, en C#, de mettre une majuscule en Début de nom de classe.

[yassin123441]

merci de votre commentaire,
alors je fait quoi dans ce cas la ???

[kheironn]

quelle est la ligne à l'origine de l'erreur ?

j'ai bien mon idée, mais le but est de t'aider à la trouver, pas de donner la réponse pour un problème si banal...

[Bluedeep]

salut tt le monde ,

j ai essai de récupéré une valeur de la table du base de donnée et la mettre dans un autre table et j ai eu cette Erreur Un DataReader associé à cette Command est déjà ouvert. Il doit d'abord être fermé.

C'est tout à fait normal, le reader travaillant en mode connecté, il y a un appel au serveur à chaque lecture. De ce fait la connexion ne peut pas être utilisée pour une autre commande en même temps.

Plusieurs possibilités :

- utiliser deux connexions : une pour lire, une pour écrire.

- travailler en mode "déconnecté" : lire dans un DataTable puis écrire la transfo de données en vrac depuis les données du DataTable. (attention le mode déconnecté ici est relatif à la commande, pas besoin de fermer la connexion entre la lecture et l'écriture.

- la troisième possibilité consiste à traiter dans une même requête la lecture et l'écriture.

[yassin123441]

voila ,

Code :

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

 sqlComm.ExecuteNonQuery();

[kheironn]

C'est tout à fait normal, le reader travaillant en mode connecté, il y a un appel au serveur à chaque lecture. De ce fait la connexion ne peut pas être utilisée pour une autre commande en même temps.

Plusieurs possibilités :

- utiliser deux connexions : une pour lire, une pour écrire.

- travailler en mode "déconnecté" : lire dans un DataTable puis écrire la transfo de données en vrac depuis les données du DataTable. (attention le mode déconnecté ici est relatif à la commande, pas besoin de fermer la connexion entre la lecture et l'écriture.

- la troisième possibilité consiste à traiter dans une même requête la lecture et l'écriture.

Je pensais à la première sollution, mais je voulais le laisser chercher un peu...

[yassin123441]

j ai utilise deux connexion merci bcp

[Bluedeep]

Ne pas oublier le bouton alors.