Discussion :

[UbiCoder]

Bonjour,

Je développe une application de suivi de stock. en ASP.NET (C#)
Au premier affichage, j'affiche tout le stock dans un GridView paginé.
La page présente des DropDownList pour présenter les critères de filtrage à la disposition de l'utilisateur.
J'ai mis un Button "Actualiser" qui permet de réafficher la GridView selon les critères sélectionnés par l'utilisateur.

Mon problème est le suivant : Aucune grille n'apparet à l'écran après avoir clicqué sue "Actualiser".

Voici le code behind de la modification du datasource :
commerce : est le DataSource
GridViewPoisson : est le GridView à actualiser

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
 
protected void ButtonActualiser_Click(object sender, EventArgs e)
{
	if (selectParams != 0)
	{
		for (int i = 0; i < selectParams; i++)
			commerce.SelectParameters.RemoveAt(0);
	}
	commerce.SelectParameters.Add("@eau", poissonEauCritere);
	commerce.SelectParameters.Add("@origine", poissonOrigineCritere);
	commerce.SelectParameters.Add("@famille", poissonFamilleCritere);
	commerce.SelectParameters.Add("@acclim", poissonAcclimCritere);
	selectParams = commerce.SelectParameters.Count;
	commerce.SelectCommandType = SqlDataSourceCommandType.Text;
	commerce.SelectCommand = String.Format(@"poisson_get_by_filter");
	commerce.DataBind();
	GridViewPoisson.DataSourceID = "commerce";
	GridViewPoisson.DataBind();
 
}

[hmimoud]

Essaye d'expliquer plus ton code (if, for et quelques methodes ...) , je ne comprend pas l'utilité de certaines choses dans ton code !

[The zxeno prophet]

Si tu veux filtrer avec des drop lists, il vaut mieux qu'elles fassent des postback (AutoPostBack="True") et que tu fasses des paramètres de type ControlParameter qui sont justement fait pour ça, en utilisant la propriété SelectedValue des drop lists.
Du coup avec l'avec le post back plus besoins de bouton Actualiser, la grid ce met à jour toute seule à chaque changement sur la drop list. Ou si tu veux un bouton actualiser, tu retire le AutoPostBack de la drop list, et tu laisse ton bouton actualiser, il feras le post back tout seul.

Si tu as de bonne raison de garder ce code je vois ça :

Code :

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

commerce.SelectCommand = String.Format(@"poisson_get_by_filter");

C'est pas le @ qui est tout simplement à l'extérieur des double quotes ?
Et c'est quoi ce poisson_get_by_filter ?

Aussi ton if ne sert à rien, si selectParams est à zéro, le for ne boucleras tout simplement pas.

De plus tu peux vider ta liste avec de paramètres avec

Code :

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

commerce.SelectParameters.Clear();

Edit:Non en fait tu dois vider toutes liste comme ça

[hmimoud]

Exacte !

[The zxeno prophet]

En fait plus je le regardes et moins je le comprends ce code, si tu veux actualiser tes paramètres pas besoins de les supprimer et de les récréer, tu met littéralement les objets paramètre à jour.
De plus tu n'a pas besoins d'écraser la commande SQL en elle même, ça enlève tout l'intérêt des paramètres.
Dans le même genre pas besoins de redire à la grid view que ça source c'est commerce elle le sait déjà