Langage C#
  1. Langage C#
  3. Je continue à avoir cette erreur: «tentative d’appel invalide lire quand le lecteur est fermé»
Intereting Posts
Ajouter un horodatage à Trace.WriteLine () Protobuf-net: les instances de classes abstraites ne peuvent pas être créées Méthode prioritaire et parameters facultatifs Problème avec HTMLAgilityPack lors de l’parsing de HTML à l’aide de C # Comment appliquer mon propre thème à mon application Windows Forms? Effet d’ombre de la fenêtre WPF Annulation des tâches en arrière-plan Comment remplir une zone fermée dans l’object Bitmap avec une couleur Afficher les fichiers word / pdf / excel, etc. dans le navigateur. (Asp.net, C # .net 2008) Effectuer une recherche de plage en C # – comment implémenter La classe de type (dans app_Code) existe à la fois dans «solution.dll» et dans «app_code.wxdafd.dll». Le déploiement ClickOnce laisse plusieurs versions (oui, plus de deux) C # Obtenir efficacement les données de pixel à partir de System.Drawing.Bitmap Calculer le nombre de jours de la semaine entre deux dates en C # En attente d’une méthode IAsyncResult qui attend une autre IAsyncResult (Chaining)

Je continue à avoir cette erreur: «tentative d’appel invalide lire quand le lecteur est fermé»

Voici mon code, je ferme et ouvre le lecteur et cela ne fonctionne toujours pas. Quelques threads peuvent accéder à cette fonction simultanément, mais il y a un verrou. Cela fonctionne plusieurs fois au début, mais tôt ou tard, l’exception “Tentative d’appel non valide Lire, lorsque le lecteur est fermé” à 

private IList GetUsers(SqlCommand cmd) { IList users = new List(); User user; lock (thisLock) { SqlDataReader dr = null; try { Conn.Open(); dr = cmd.ExecuteReader(CommandBehavior.CloseConnection); while (dr.Read()) { user = new User { UserId = Convert.ToInt32(dr["WorkerNum"]), CompanyName = dr["CompanyName"].ToSsortingng(), WorkerName = dr["WorkerFirstName"] + " " + dr["WorkerFamilyName"], Phone = dr["Phone"].ToSsortingng() }; if (dr["QueueNum"] != null && dr["QueueNum"] != DBNull.Value) { user.Queue = new Queue { HasAlreadyEntered = dr["flgAppear"] != null && dr["flgAppear"].ToSsortingng() == "Y", IsFromWebsite = dr["TookFrom"].ToSsortingng() == "1", IsMelutash = dr["IsMelutash"].ToSsortingng() == "1", TimeOrdered = DateTime.Parse(dr["DateTime1"].ToSsortingng()), QueueNum = Convert.ToInt32(dr["QueueNum"]), SMS = dr["SMSCode"].ToSsortingng() }; } users.Add(user); } } catch (Exception e) { throw e; } finally { if (dr != null) { dr.Close(); dr.Dispose(); } } return users; } }

Ce qui donne?

Essayons d’améliorer un peu ce code: 

 private IEnumerable GetUsers() { using (var conn = new SqlConnection(SomeConnectionSsortingng)) using (var cmd = conn.CreateCommand()) { conn.Open(); cmd.CommandText = "SELECT WorkerNum, CompanyName, ... FROM Users"; using (var reader = cmd.ExecuteReader()) { while (reader.Read()) { var user = new User { UserId = reader.GetInt32(reader.GetOrdinal("WorkerNum")), CompanyName = reader.GetSsortingng(reader.GetOrdinal("CompanyName")), // TODO: complete other fields }; // TODO: do the tests and complete the complex properties yield return user; } } } }

Maintenant, ce code est parfaitement réentrant et thread-safe. Vous n’avez besoin d’aucun locking.