Discussion :

[eclepkens]

Bonjour à tous. j'ai effectué une recherche sur le forum au sujet de mon probleme et n'ayant rien trouvé je viens demander de l'aide directement ici. J'ai herite d'un site devellope a l'etranger (je suis en france) en asp.net c# . J'ai un probleme recurent sur n'importe quelle page de mon site. de façon tout a fait aleatoire (en apparence en tout cas) j'ai le message "
ExecuteReader requires an open and available Connection. The connection's current state is closed.

Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.

Exception Details: System.InvalidOperationException: ExecuteReader requires an open and available Connection. The connection's current state is closed.

Source Error:

Line 135:
Line 136: SqlCommand cmd_Extensions = new SqlCommand(requete, cconnection);
Line 137: StrDataReader = cmd_Extensions.ExecuteReader();
Line 138: while (StrDataReader.Read())
Line 139:
"
qui remonte. cependant en rafraichissant la page peut finalement apparaitre, ou pas. Quelqu'un a deja eu ce type de problemes ? merci.

[rad_hass]

Je suppose qu'il y a une classe qui gère la création d'une connexion ? C'est sous quel forme ? Peut être c'est une problème d'implémentation de cette classe ... cconnection est définie comment ?

[eclepkens]

oui je pense qu'il s'agit ce celle ci presente dans index.aspx.cx


{
string requete;

requete = "SELECT * from homepage";

string strConnection = System.Configuration.ConfigurationManager.ConnectionStrings["A***DBConnectionString"].ConnectionString;
SqlConnection cconnection = new SqlConnection(strConnection);

try
{
cconnection.Open();

SqlDataReader StrDataReaderx;

SqlCommand cmd_infosHome = new SqlCommand(requete, cconnection);
StrDataReaderx = cmd_infosHome.ExecuteReader();
while (StrDataReaderx.Read())
{
switch (StrDataReaderx["HOM_TYP"].ToString())
{
case "ACT":
lblTitreActualites.Text = Server.HtmlDecode(StrDataReaderx["HOM_PHO"].ToString());
lblTexteActualites.Text = Server.HtmlDecode(StrDataReaderx["HOM_ACT"].ToString());
break;
case "OFR":
//imgOffre.ImageUrl = StrDataReader["HOM_PHO"].ToString();
//HyperLinkOffre.NavigateUrl = StrDataReader["HOM_URL"].ToString();
break;
}
}
StrDataReaderx.Close();
}
finally
{
cconnection.Close();
cconnection.Dispose();
}

}

[rad_hass]

Utilise la balise code, parce que c'est difficile de lire ainsi du code ...
Il n y a pas de catch, essaie de mieux traiter tes exceptions ...

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
{
            string requete;
 
            requete = "SELECT * from homepage";
 
            string strConnection = System.Configuration.ConfigurationManager.ConnectionStrings["A***DBConnectionString"].ConnectionString;
            SqlConnection cconnection = new SqlConnection(strConnection);
 
            try
            {
                cconnection.Open();
 
                SqlDataReader StrDataReaderx;
 
                SqlCommand cmd_infosHome = new SqlCommand(requete, cconnection);
                StrDataReaderx = cmd_infosHome.ExecuteReader();
                while (StrDataReaderx.Read())
                {
                    switch (StrDataReaderx["HOM_TYP"].ToString())
                    {
                        case "ACT":
                            lblTitreActualites.Text = Server.HtmlDecode(StrDataReaderx["HOM_PHO"].ToString());
                            lblTexteActualites.Text = Server.HtmlDecode(StrDataReaderx["HOM_ACT"].ToString());
                            break;
                        case "OFR":
                            //imgOffre.ImageUrl = StrDataReader["HOM_PHO"].ToString();
                            //HyperLinkOffre.NavigateUrl = StrDataReader["HOM_URL"].ToString();
                            break;
                    }
                }
                StrDataReaderx.Close();
            }
            finally
            {
                cconnection.Close();
                cconnection.Dispose();
            }
 
        }

[Kelpan]

Pourtant le message semble clair...

"La connexion n'est pas ouverte."

Il te manque l'instruction suivante entre la ligne 136 et 137

Code :

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

1
2
 
cconnection.Open();

[rad_hass]

Pourtant le message semble clair...

"La connexion n'est pas ouverte."

Il te manque l'instruction suivante entre la ligne 136 et 137

Code :

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

1
2
 
cconnection.Open();

Après le try il fait bien

Code :

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

1
2
 
cconnection.Open();

Donc ce n'est pas le souci ...

En plus, si je ne m'abuse il dit que ça ne marchait pas que de manière aléatoire ...

[Kelpan]

La source avec le "try" ne correspond pas à la source qui provoque l'erreur apparemment.

Pour moi je ne vois pas beaucoup de possibilité. Le message précise bien que la connexion est fermée au moment de l'appel. Cela veut dire qu'il n'y a pas d'ouverture de connexion avant le lancement de la commande.

Ensuite, si cela marche aléatoirement, cela vient surement d'une mauvaise gestion de la femeture de connexion dans certaines procédures qui fait que la procédure peut recevoir une connexion ouverte.

[eclepkens]

Merci rad_hass, en effet c'est tres etrange comme comportement. le contenu des pages remonte ou pas. je n'arrive pas a savoir d'ou cela peut venir. du code ? du sqlerver ? des deux ? n'etant pas le devellopeur et n'ayant pas participé au devellopement je tatonne pas mal.
Quand aux lignes 136 et 137 ça depend de la page. là je n'avais pas penser a la noter. apres une dizaine de refresh sur la meme page j'ai reussi a la replanter et là j'ai :

ExecuteReader requires an open and available Connection. The connection's current state is closed.
Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.

Exception Details: System.InvalidOperationException: ExecuteReader requires an open and available Connection. The connection's current state is closed.

Source Error:


Line 23: string requete = "select * from ConditionsDeVente where obj_id in(" + Request.QueryString["sej_id"].ToString() + "," + Request.QueryString["hot_id"].ToString() + ")";
Line 24: SqlCommand cmd_conditions = new SqlCommand(requete, cconnection);
Line 25: StrDataReader = cmd_conditions.ExecuteReader();
Line 26: lblConditionsHotel.Text = "";
Line 27: while (StrDataReader.Read())


Source File: c:\Inetpub\Amplitudes.com\Sejours\PrixEtConditions.aspx.cs Line: 25

Stack Trace:


[InvalidOperationException: ExecuteReader requires an open and available Connection. The connection's current state is closed.]
System.Data.SqlClient.SqlConnection.GetOpenConnection(String method) +857028
System.Data.SqlClient.SqlConnection.ValidateConnectionForExecute(String method, SqlCommand command) +9
System.Data.SqlClient.SqlCommand.ValidateCommand(String method, Boolean async) +56
System.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, String method, DbAsyncResult result) +73
System.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, String method) +32
System.Data.SqlClient.SqlCommand.ExecuteReader(CommandBehavior behavior, String method) +122
System.Data.SqlClient.SqlCommand.ExecuteReader() +84
AmplitudesV2.Sejours.PrixEtConditions.Page_Load(Object sender, EventArgs e) in c:\Inetpub\Amplitudes.com\Sejours\PrixEtConditions.aspx.cs:25
System.Web.Util.CalliHelper.EventArgFunctionCaller(IntPtr fp, Object o, Object t, EventArgs e) +15
System.Web.Util.CalliEventHandlerDelegateProxy.Callback(Object sender, EventArgs e) +34
System.Web.UI.Control.OnLoad(EventArgs e) +99
System.Web.UI.Control.LoadRecursive() +47
System.Web.UI.Page.ProcessRequestMain(Boolean includeStagesBeforeAsyncPoint, Boolean includeStagesAfterAsyncPoint) +1061