Il y a déjà un DataReader ouvert associé à cette Commande, qui doit d'abord être fermés

C'est le code que j'ai.

///<summary>
///Method calls stored procedure and fills DataSet of contacts associated with Lead
///</summary>
///<param name="leadID">The ID associated with a Lead</param>
///<returns>contacts list as DataSet</returns>
public static DataSet GetContactResultSetByLead(int leadID)
{
    SqlCommand Sqlmd = new SqlCommand("dbo.proc_contact");
    Sqlmd.CommandType = CommandType.StoredProcedure;
    Sqlmd.Parameters.Add("@LeadInfoID", SqlDbType.Int).Value = leadID;

    Sqlmd.Connection = m_ConStr;
    SqlDataAdapter da = new SqlDataAdapter(Sqlmd);

    DataSet data = new DataSet();
    try
    {
        da.Fill(data);
    }

    finally
    {
        m_ConStr.Close();
    }

    return data;
}
Qu'est-ce que m_ConStr?
modifiez cette ligne Sqlmd.Les paramètres.Add("@LeadInfoID", SqlDbType.Int).Valeur = leadID; Sqlmd.Les paramètres.AddWithValue("@LeadInfoID", leadID); Si vous voulez garder le Mondial de vous Connecter, Vérifiez l'État de la Connexion s'ouvrir et de la fermer avant de l'utiliser. RESTRUCTURER LE CODE ET L'UTILISATION SIGNIFICATIVE DES NOMS DE VARIABLES....! Enveloppez votre Connexion autour d'une aide() {}
Sqlmd sera exécuté au cours de la da.Remplir appel. m_ConStr semble être un mal nommée variable, il semble faire référence à la connexion en elle-même, et non pas la chaîne de connexion.

OriginalL'auteur Marcus3329 | 2012-01-26

  1. 9

    Votre problème est que vous ne semblent avoir qu'une seule instance de m_ConStr; si la méthode est appelé en même temps un seul d'entre eux sera en mesure d'utiliser la connexion et l'autre échouera avec l'exception que vous recevez.

    Utiliser ce modèle à la place:

    using (SqlConnection conn  = new SqlConnection())
{
    conn.Open();
    Sqlmd.Connection = conn;
    SqlDataAdapter da = new SqlDataAdapter(Sqlmd);
   //...etc
}

    En d'autres termes, ne pas définir les paramètres de connexion comme une variable globale à la classe.

    "ne pas définir les paramètres de connexion en tant que variable globale": mais alors, comment éviter de créer et d'ouvrir une connexion à chaque fois que j'ai pour exécuter un sp. Dans mon cas, j'ai d'exécuter beaucoup de sp, et sans l'instancier celle de la connexion, l'alternative est de créer et d'ouvrir à chaque fois. Tous les conseils?

    OriginalL'auteur Icarus

  2. 5

    Tous de votre de courte durée IDisposable objets sont dépourvus d'une "aide". Par extension, il est alors possible que vous avez fait quelque chose comme:

    var reader = anotherCommand.ExecuteReader();
...

    Mais cela ne jetez pas /fermer le lecteur. Si c'est le cas, ajouter "à l'aide":

    using(var reader = anotherCommand.ExecuteReader()) {
    ...
}

    Qui ferme le lecteur, indépendamment de la façon dont nous quitter. Les commandes, les connexions, les lecteurs et les transactions sont tous à usage unique et doit normalement utiliser "utiliser".

    OriginalL'auteur Marc Gravell

  3. 5

    je suggère que Vous pouvez à l'aide de bloc pour assurer une bonne élimination de sqlconnection.

    using (SqlConnection conn  = new SqlConnection())
{
    conn.Open();
    Sqlmd.Connection = conn;
    SqlDataAdapter da = new SqlDataAdapter(Sqlmd);
    Dataset ds = new Datasest
    da.Fill(ds)
}

    l'autre manière est que vous pouvez également définir des MARS de propriété en votre connexion, si vous avez besoin.

    SqlConnection m_ConStr;= new SqlConnection("Server= serverName;Database=yourDatabase;
        MultipleActiveResultSets=true;");
    La connexion doit être déclaré localement (toutes sortes d'effets secondaires qui va vous arriver si vous utilisez une instance partagée). Aussi, pourquoi ne pas vous venez d'employer une instruction d'utilisation pour gérer la durée de vie de la connexion?
    c'est également une bonne idée:) ok pour le modifier 🙂

    OriginalL'auteur Ravi Gadag

  4. 1

    Que vous essayez d'exécuter de multiples active des ensembles de résultats (aka MARS).

    Deux solutions me viennent à l'esprit:

    1. Ouvrir ouvrir une nouvelle connexion dans votre GetContractResultSetByLead
    2. Permettre de MARS sur votre serveur de base de données (décrit dans le lien ci-dessus).
    Vous pouvez aussi essayer d'ajouter ça dans le web.config dans la chaîne de connexion: connectionString="MultipleActiveResultSets=True;user=..."

    OriginalL'auteur RQDQ