2017-03-24 20 views
1

動的に生成されたユーザコントロールで、選択した値を保持するのに問題があります。いくつかのフォームフィールドを持つusercontrolを作成しました。メインページには、私のusercontrolをプレースホルダに配置するボタンがあります。ユーザーは、必要な数だけこれらを作成できます。私はまた、ユーザーが任意のコントロールを削除することができますコントロール自体にボタンがあります。この機能は正常に動作しているようです。ポストバックで動的に作成されたusercontrolsの奇妙な動作

しかし、すごみは、次のシナリオですさまじい:ページ上のユーザーコントロールを作成する

  1. クリックボタン。フォームの に記入してください。
  2. ボタンをもう一度クリックして、2番目のUCを作成し、フォーム のフィールドに記入します。この時点ですべてがうまくいきます。 UC#1の値は、 の値を保持していました。
  3. ボタンをクリックすると3番目のUCが作成され、選択された値はすべてUC#1と#2から抹消された です。
  4. すべてのUCのフィールドをリフィルしてからボタンをクリックして4番目のUCを作成すると、UC#1とUC#3はその値を保持しますが、UC#2はその値を失います。

大変お手伝いをいたします。私はこれを理解しようとしているバグに目をつぶっています。これは動的なユーザコントロールへの私の最初の進出であり、これまでのところ私のお尻を蹴っています。そして、私はまだこれらのUCにdbの値を設定する方法を理解しなければならないので、ユーザーはフォームを戻って編集することができますが、一度に1つのことを行うことができます。

ASPX:

<asp:PlaceHolder ID="placeholderOffenseCodes" runat="server"> </asp:PlaceHolder> 
    <asp:Button ID="btnAddOffense" runat="server" Text="Add an Offense" CausesValidation="false" OnClick="btnAddOffense_Click" /> 
<!--The text value determines how many items are initially displayed on the page--> 
    <asp:Literal ID="ltlCount" runat="server" Text="0" Visible="false" /> 
    <asp:Literal ID="ltlRemoved" runat="server" Visible="false" /> 

aspx.cs:

protected void Page_Load(object sender, EventArgs e) 
{ 
    AddAndRemoveDynamicOffenseControls(); 
} 

private void AddAndRemoveDynamicOffenseControls() 
{ 
    //Determine which control fired the postback event. 
    Control c = GetPostBackOffenseControl(Page); 

    if ((c != null)) 
    { 
     //If the add button was clicked, increase 
     //the count to let the page know we want 
     //to display an additional user control 
     if (c.ID.ToString() == "btnAddOffense") 
     { 
      ltlCount.Text = Convert.ToString(Convert.ToInt16(ltlCount.Text) + 1); 
     } 
    } 

    //Be sure everything in the placeholder control is cleared out 
    placeholderOffenseCodes.Controls.Clear(); 

    int ControlID = 0; 

    //Re-add controls every time the page loads. 
    for (int i = 0; i <= (Convert.ToInt16(ltlCount.Text) - 1); i++) 
    { 
     IncidentGroupA_Offenses uc = (IncidentGroupA_Offenses)LoadControl("IncidentGroupA_Offenses.ascx"); 

     //If this particular control id has been deleted 
     //from the page, DO NOT use it again. If we do, it will 
     //pick up the viewstate data from the old item that 
     //had this control id, instead of generating 
     //a completely new control. Instead, increment 
     //the control ID so we're guaranteed to get a "new" 
     //control that doesn't have any lingering information in the viewstate. 
     while (InDeletedOffenseList("offense" + ControlID) == true) 
     { 
      ControlID += 1; 
     } 

     //Note that if the item has not been deleted from the page, 
     //we DO want it to use the same control id 
     //as it used before, so it will automatically maintain 
     //the viewstate information of the user control 
     //for us. 
     uc.ID = "offense" + ControlID; 

     //Add an event handler to this control to raise 
     //an event when the delete button is clicked 
     //on the user control 
     uc.RemoveOffenseUC += this.HandleRemoveOffenseUserControl; 

     //Add the user control to the panel 
     placeholderOffenseCodes.Controls.Add(uc); 

     //Add Offense number to label on usercontrol 
     int OffenseNum = i + 1; 
     uc.OffenseNumber = "Offense " + OffenseNum; 

     //Increment the control id for the next round through the loop 
     ControlID += 1; 
    } 
} 

protected void btnAddOffense_Click(object sender, EventArgs e) 
{ 
    //handled in page_load 
} 

private bool InDeletedOffenseList(string ControlID) 
{ 
    //Determine if the passed in user control ID 
    //has been stored in the list of controls that 
    //were previously deleted off the page 
    string listvalues = ltlRemoved.Text; 
    string[] stringSeparators = new string[] { "|" }; 
    string[] DeletedList = listvalues.Split(stringSeparators, StringSplitOptions.RemoveEmptyEntries); 
    for (int i = 0; i <= DeletedList.GetLength(0) - 1; i++) 
    { 
     if (ControlID == DeletedList[i]) 
     { 
      return true; 
     } 
    } 
    return false; 
} 

public void HandleRemoveOffenseUserControl(object sender, EventArgs e) 
{ 
    //This handles delete event fired from the user control 

    //Get the user control that fired this event, and remove it 
    LinkButton linkBtn = sender as LinkButton; 
    IncidentGroupA_Offenses uc = (IncidentGroupA_Offenses)linkBtn.Parent; 

    if (uc != null) 
    { 
     placeholderOffenseCodes.Controls.Remove(uc); 
    } 

    //Keep a pipe delimited list of which user controls were removed. This will increase the 
    //viewstate size if the user keeps removing dynamic controls, but under normal use 
    //this is such a small increase in size that it shouldn't be an issue. 
    ltlRemoved.Text += uc.ID.ToString() + "|"; 

    //Also, now that we've removed a user control decrement the count of total user controls on the page 
    ltlCount.Text = Convert.ToString(Convert.ToInt16(ltlCount.Text) - 1); 
} 

public Control GetPostBackOffenseControl(Page page) 
{ 
    Control control = null; 

    string ctrlname = page.Request.Params.Get("__EVENTTARGET"); 
    if ((ctrlname != null) & ctrlname != string.Empty) 
    { 
     control = page.FindControl(ctrlname); 
    } 
    else 
    { 
     foreach (string ctl in page.Request.Form) 
     { 
      Control c = page.FindControl(ctl); 
      if (c is System.Web.UI.WebControls.Button) 
      { 
       control = c; 
       break; 
      } 
     } 
    } 
    return control; 
} 

.ascx.cs:

public event EventHandler RemoveOffenseUC; 

protected void btnRemoveOffense_Click(object sender, EventArgs e) 
{ 
    //Raise this event so the parent page can handle it 
    if (RemoveOffenseUC != null) 
    { 
     RemoveOffenseUC(sender, e); 
    } 
} 

public string OffenseNumber 
{ 
    get { return lblOffenseNumber.Text; } 
    set { lblOffenseNumber.Text = value; } 
} 

答えて

0

私はいつもあなたが正しいことを確認するためにPage_Initで動的にコントロールを追加すると思っていましたViewStateの読み込み。多分簡単な変更はあなたの問題を解決するでしょうか?

それ以外の場合は、リピータを使用してダイナミックコントロールを回避することで過去に運が良かったと思います。ダイナミックなコントロールを追加するのではなく、Webフォームがあまり良くないのではなく、リストやADO DataTableのようなデータ構造にデータを追加し、それをあなたが望むコントロールを持つasp:Repeaterにバインドします。ダイナミックなコントロールでうんざりしないでください。

運が良かった!

+0

ありがとうございました。私のコードをPage_Initに移動しようとしています。 – Wilock