DataGrid et ViewStateRésolu

Question

Bonjour, 



J'ai un probl&#232;me pour cr&#233;er un datagrid qui utilise le viewstate, plus pr&#233;cisemment ce probl&#232;me l'&#233;v&#233;nement onItemCreated.

Dans celui-ci j'utilise un test pour savoir si l'Item qui est cr&#233;&#233; est un EditItem et pour faire des instructions :



if(e.Item.ItemType.ToString()=="EditItem")

 {

    ((TextBox)e.Item.FindControl("m_desc")).Text=ReaderTarifs["description"].ToString();

 }



jusque la pas de pb, ma textbox qui se trouve sur une ligne d'edition
se rempli avec la valeur voulu qui correspond &#224; une case du dataReader
qui n'est autre que la source du datagrid.

Par contre une fois que je clique sur le bouton qui est cens&#233;
enregistr&#233; ces modifications, j'ai un message d'erreur sur la ligne
d'instruction plus haut en me disant que l'objet n'existe pas. C'est
normal car le dataReader n'existe pas quand il recr&#233; le datagrid &#224;
partir du viewstate.



Comment faire pour r&#233;gler ce probl&#232;me ? (j'ai bien bidouiller avec un
try catch mais ca me semble pas tr&#232;s propre comme solution...)



Merci

jesusonline · Accepted Answer

j'aime pas les postbacks :p 

ce que tu peux faire c'est tester le datareader avant de binder tes trucs ... Sinon faudrais regarder en d&#233;tail ce qu'il se passe 

C'est assez logique en fait car quand tu cliques sur un autre bouton, le datagrid reste en edition, donc il va essayer de rebinder les controles, comme la page ne vient pas d'un itemcommand c'est &#224; dire d'un postback caus&#233; par le datagrid, la page ne passe par la m&#233;thode onItemCommand, donc ReaderTarifs est nulle, donc v&#233;rifie si ton reader existe, ou alors met un boolean a false qui passe a true dans l'itemcommand et test ce boolean dans onitemcreated y'a surement d'autres solutions, mais mieux vaut comprendre le probl&#232;me :) 


Est-ce le cas ? 



<HR>



Cyril - MVS - MCP ASP

jesusonline · Answer

<HR>
Cyril - MVS - MCP ASP

benjiiim94 · Answer

euh.... je sais bien que c'est l'intention qui compe mais la... ;-)

jesusonline · Answer

??? tient bizarre :s le probl&#232;me c'est que je sais plus ce que je disais :s 

Tu veux faire quoi ? 

t'as une ligne, avec des champs et un bouton editer ? quand tu cliques sur ce bouton ? tes champs se transforment en textbox, puis le bouton editer en sauvegarder ? 

quand t'appuies sur sauvegarder ca plante ? je comprend pas trop l'erreur :s


<HR>
Cyril - MVS - MCP ASP

benjiiim94 · Answer

Tu as tout bon, mais le probl&#232;me apparait pas que quand je cliques sur
le sauvegarder mais sur n'importe quel bouton. Quand je clique sur un
bouton (si j'ai bien compris tous le d&#233;roulement d'un postback) cela
recharge la page et entre autre charge les controles en viewstate.

C'est &#224; ce moment que ca plante, il me semble que l'&#233;venement
onItemCreated est d&#233;clench&#233;. Dans cet &#233;v&#232;nement je test si l'item en
cours de cr&#233;ation est en EditItem, si oui je remplis mes textboxs avec
le reader qui rempli normalement le datagrid. Mais lorsque le datagrid
se rempli en viewstate, ce reader n'existe pas, j'ai donc l'erreur
suivante. cet objet est le reader:




La r&#233;f&#233;rence d'objet n'est pas d&#233;finie &#224; une instance d'un objet.

Voici la trace de la pile :



[NullReferenceException: La r&#233;f&#233;rence d'objet n'est pas d&#233;finie &#224; une instance d'un objet.]
   Alpha_Intra.Administration.bateau.onItemCreated(Object sender, DataGridItemEventArgs e) in c:\inetpub\wwwroot\alpha_intra\administration\bateau.aspx.cs:296
   System.Web.UI.WebControls.DataGrid.OnItemCreated(DataGridItemEventArgs e) +110
   System.Web.UI.WebControls.DataGrid.CreateItem(Int32 itemIndex, Int32 dataSourceIndex, ListItemType itemType, Boolean dataBind, Object dataItem, DataGridColumn[] columns, TableRowCollection rows, PagedDataSource pagedDataSource) +111
   System.Web.UI.WebControls.DataGrid.CreateControlHierarchy(Boolean useDataSource) +1408
   System.Web.UI.WebControls.BaseDataList.CreateChildControls() +60
   System.Web.UI.Control.EnsureChildControls() +100
   System.Web.UI.WebControls.BaseDataList.get_Controls() +12
   System.Web.UI.Control.LoadViewStateRecursive(Object savedState) +292
   System.Web.UI.Control.LoadViewStateRecursive(Object savedState) +414
   System.Web.UI.Control.LoadViewStateRecursive(Object savedState) +414
   System.Web.UI.Page.LoadPageViewState() +306
   System.Web.UI.Page.ProcessRequestMain() +447

J'&#233;sp&#232;re que j'ai &#233;t&#233; assez clair maintenant.



Pour contourner l'erreur j'ai mis un try autour du remplissage de mes
texbox et un catch vide. Mais je ne pense pas qu'on ai "le droit"
d'utiliser les try catch comme ca...



Merci beaucoup

jesusonline · Answer

Le try c'est "pas mal" ;) 

pourquoi ne testerais tu pas si ton datareeader existe ? 

fais voir ton code ? je verrais ce qui est faisable :)


<HR>
Cyril - MVS - MCP ASP

benjiiim94 · Answer

Cool j'avais peur d'avoir fait encore une erreur dans l'enchenement des flashbacks (euh des postbacks...)

Je n'avais pas pens&#233; a regarder pour le DataReader, c'est vrai que c'est mieux que le try...



Ya pas de choses compliqu&#233;, c'est pour ca que je n'ai pas tri&#233; l'essentiel.

Merci





        private void
onItemCommandDataGrid(object source,
System.Web.UI.WebControls.DataGridCommandEventArgs e)

        {

            if(e.CommandName=="New")

            {

           
    string A =
((TextBox)e.Item.FindControl("n_sem_A")).Text;

           
    string B =
((TextBox)e.Item.FindControl("n_sem_B")).Text;

           
    string C =
((TextBox)e.Item.FindControl("n_sem_C")).Text;

           
    string D =
((TextBox)e.Item.FindControl("n_sem_D")).Text;

           
    string E =
((TextBox)e.Item.FindControl("n_sem_E")).Text;



           
    string wA =
((TextBox)e.Item.FindControl("n_week_A")).Text;

           
    string wB =
((TextBox)e.Item.FindControl("n_week_B")).Text;

           
    string wC =
((TextBox)e.Item.FindControl("n_week_C")).Text;

           
    string wD =
((TextBox)e.Item.FindControl("n_week_D")).Text;



                if(A=="") A="0";

                if(B=="") B="0";

                if(C=="") C="0";

                if(D=="") D="0";

                if(E=="") E="0";



                if(wA=="") wA="0";

                if(wB=="") wB="0";

                if(wC=="") wC="0";

                if(wD=="") wD="0";



           
    ModifCommand.CommandText="Insert Into bateaux_tarifs
(bateau_id, description, prix_A, prix_B, prix_C, prix_D, prix_E,
prix_weekend_A, prix_weekend_B, prix_weekend_C, prix_weekend_D) values
(" + Request.QueryString.GetValues("id")[0].ToString() + ",'" +
((TextBox)e.Item.FindControl("n_desc")).Text.Replace("'","''") + "'," +
A + "," + B + "," + C + "," + D + "," + E + "," + wA + "," + wB + "," +
wC + "," + wD + ")";

                ModifCommand.ExecuteNonQuery();



                Binding();

            }

            else if(e.CommandName=="t_edition")

            {

                DataGridTarifs.EditItemIndex=e.Item.ItemIndex;

                Binding();



            }

            else if(e.CommandName=="t_suppression")

            {

           
    ModifCommand.CommandText="Delete * from
bateaux_tarifs where id = " + e.CommandArgument;

                ModifCommand.ExecuteNonQuery();

                Binding();

            }

            else if(e.CommandName=="t_annulation")

            {

                DataGridTarifs.EditItemIndex=-1;



                Binding();

            }

            else if(e.CommandName=="t_modification")

            {



           
    string A =
((TextBox)e.Item.FindControl("m_sem_A")).Text;

           
    string B =
((TextBox)e.Item.FindControl("m_sem_B")).Text;

           
    string C =
((TextBox)e.Item.FindControl("m_sem_C")).Text;

           
    string D =
((TextBox)e.Item.FindControl("m_sem_D")).Text;

           
    string E =
((TextBox)e.Item.FindControl("m_sem_E")).Text;



           
    string wA =
((TextBox)e.Item.FindControl("m_week_A")).Text;

           
    string wB =
((TextBox)e.Item.FindControl("m_week_B")).Text;

           
    string wC =
((TextBox)e.Item.FindControl("m_week_C")).Text;

           
    string wD =
((TextBox)e.Item.FindControl("m_week_D")).Text;



                if(A=="") A="0";

                if(B=="") B="0";

                if(C=="") C="0";

                if(D=="") D="0";

                if(E=="") E="0";



                if(wA=="") wA="0";

                if(wB=="") wB="0";

                if(wC=="") wC="0";

                if(wD=="") wD="0";



           
    ModifCommand.CommandText="Update bateaux_tarifs set
description='" +
((TextBox)e.Item.FindControl("m_desc")).Text.Replace("'","''") + "',
prix_A=" + A + ", prix_B=" + B + ", prix_C=" + C + ", prix_D=" + D + ",
prix_E=" + E + ", prix_weekend_A=" + wA + ", prix_weekend_B=" + wB + ",
prix_weekend_C=" + wC + ", prix_weekend_D=" + wD + " where Id=" +
e.CommandArgument;

                ModifCommand.ExecuteNonQuery();



                DataGridTarifs.EditItemIndex=-1;

                Binding();

            }

    }





        private void Binding()

        {



           
CmdTarifs.CommandText="Select * from bateaux_tarifs where bateau_id=" +
Request.QueryString.GetValues("id")[0].ToString();



            ReaderTarifs = CmdTarifs.ExecuteReader();

            DataGridTarifs.DataSource=ReaderTarifs;

            DataGridTarifs.DataBind();

            ReaderTarifs.Close();

        }





private void onItemCreated(object sender, System.Web.UI.WebControls.DataGridItemEventArgs e)

        {

            if(e.Item.ItemType.ToString()=="EditItem")

            {

                try

                {

           
       
((TextBox)e.Item.FindControl("m_desc")).Text=ReaderTarifs["description"].ToString();

               
   //plus plein d'autres remplissages de textboxs dans le
meme genre

                }

                catch

                {

                }

                

            }

        

        }

benjiiim94 · Answer

Ouai j'avais bien compris "le ph&#233;nom&#232;ne", je vais donc tester
l'existence du Reader avant de continuer. Je pensais que je faisais une
grosse erreur de "conception" qui impliquait cela.



J'ai &#233;galement compris au fil de tes posts que tu n'aimais pas les
postbacks, il faudrait moi aussi que j'utilise plus javascript (et du
ClientCallBack mais juste quand cela est n&#233;cessaire ! ;-) ). As-tu
quelques liens de r&#233;f&#233;rence sur le javascript "utile" qui trainent sous
la main ?



Merci

jesusonline · Answer

non, il ne faut pas remplacer les postback par les clientCalllBack la est le meme probl&#232;me : les robots ne pourront pas suivre les liens, je pense surtout &#224; la pagination d'un datagrid ... je fais seulement du web grand public, donc google est TRES important, donc les potback sont banni sauf pour certaines choses, dans ce cas par exemple pour l'edition pourquoi pas, ca permet d'aller plus vite ..

L'autre probl&#232;me c'est que les postbacks sont difficile &#224; comprendre et la plupart des personnes l'utilisent sans meme savoir que ca utilise javascript, donc inaccessible aux robots :( 

Ensuite pour javascript, oui il faut s'y interesser :):) javascript c'est ENORME mais malheureusement c'est un langage d'incompris ... pour apprendre javascript ce que j'ai fait c'est lire du code orient&#233; objet ... notament la librairie prototype et script.aculo.us et tester des trucs ... 

Je vais bientot essayer de prendre un peu de temps pour parler des bases de la programmation javascript sur mon blog, classe, h&#233;ritage, namespace etc... 


<HR>
Cyril - MVS - MCP ASP

benjiiim94 · Answer

Merci pour ces pr&#233;cisions, je suivrai ton blog avec impatience...

jesusonline · Answer

Au fait je viens d'y repenser mais ca : 

CmdTarifs.CommandText="Select * from bateaux_tarifs where bateau_id=" + Request.QueryString.GetValues("id")[0].ToString(); 

dis moi que c'etait une blague ??? 

car si je vais sur le site et je tape 

mapage.aspx?id=2;shutdown with no wait-- 

ou mapage.aspx?id=2; EXEC master..xp_cmdshell 'ECHO coucou > hacked.txt'

>> http://www.evilznet.com/blog.aspx?EntryID=31 


<HR>
Cyril - MVS - MCP ASP

benjiiim94 · Answer

Euh..., en effet, je ne me suis pas beaucoup occup&#233; de la s&#233;curit&#233; dans
ce projet. C'est un intranet qui ne poss&#232;de aucun backlink et les
utilisateurs doivent se logguer avant d'y acc&#233;der donc ca limite les
risques d'attaque. Mais tu as raison, faut que je fasse atention &#224; ca...



Merci

DataGrid et ViewState

12 réponses

Votre réponse

Discussions similaires