asp.net редактирование данных

Question

Я читаю данные из базы данных и показываю их на странице для редактирования:

    <h2>Create new topic: 
    <asp:Label ID="_lblTopicName" runat="server" Text=""></asp:Label></h2>
      <p>
      Edit Level:
      <br/>
      <asp:DropDownList ID="_dtlEditRole" runat="server"></asp:DropDownList>
      <br/>
      View Level:
      <br/>
      <asp:DropDownList ID="_dtlViewRole" runat="server"></asp:DropDownList>
      <br/>
      <asp:TextBox ID="_tbxTopicText" TextMode="MultiLine" runat="server" Height="204px" 
        Width="885px"></asp:TextBox>
    </p>
  <asp:Button ID="_btnSaveTopic" runat="server" Text="Save" onclick="_btnSaveTopic_Click" />

Я заполняю поля в Page_PreRender () примерно так:

    private string _topicString;
    private Topic _topic = null;
    private Topics_GetTopicByTopicResult _findTopicResults = null;

    protected void Page_PreRender(object sender, EventArgs e)
    {
        // Load the User Roles into checkboxes.
        _dtlEditRole.DataSource = Roles.GetAllRoles();
        _dtlEditRole.DataBind();
        _dtlViewRole.DataSource = Roles.GetAllRoles();
        _dtlViewRole.DataBind();

        _topicString = Request.QueryString["Topic"];

        if (String.IsNullOrEmpty(_topicString))
        {
            Response.Redirect("~/Default.aspx");
        }
        else
        {
            _topic = new Topic();
            _findTopicResults = _topic.FindTopic(_topicString);

            if (_topic != null)
            {
                // Check if the user has permission to access
                if (RoleHelper.IsEditAllowed(_findTopicResults.ViewRoleName))
                {
                    _lblTopicName.Text = _findTopicResults.Topic;
                    _tbxTopicText.Text = _findTopicResults.Text;

                    _dtlEditRole.SelectedValue = _findTopicResults.EditRoleName;
                    _dtlViewRole.SelectedValue = _findTopicResults.ViewRoleName;
                }
                else
                {
                    Response.Redirect("~/Error.aspx?ReturnUrl=" + HttpUtility.UrlEncode(Request.RawUrl));
                }
            }
            else
            {
                Response.Redirect("~/CreateTopic.aspx?Topic=" + _topicString);
            }
        }
    }

Но теперь, когда я нажимаю кнопку _btnSaveTopic, поля:

    private string _topicString;
    private Topic _topic = null;
    private Topics_GetTopicByTopicResult _findTopicResults = null;

Все они равны NULL, и я не могу что-либо обновить.

Вот мое событие нажатия кнопки:

    protected void _btnSaveTopic_Click(object sender, EventArgs e)
    {
            _topic.UpdateTopic(_findTopicResults.ID, _findTopicResults.Topic, _tbxTopicText.Text,
                               _dtlViewRole.SelectedItem.Text, _dtlEditRole.SelectedItem.Text);
            Response.Redirect("~/ViewPage.aspx?Topic=" + _topicString);
    }

Что бы правильно это делать?

Answer 1

Жизненный цикл страницы ASP.NET указывает, что Page_Init следует использовать для «инициализации свойств элемента управления», что похоже на то, что вы делаете.

Кроме того, обычно хорошей практикой является разбиение таких больших участков кода на более мелкие рефакторированные методы. Постарайтесь свести к минимуму количество кода, непосредственно размещаемого в обработчиках событий.

Вы можете начать, щелкнув правой кнопкой мыши раздел выделенного кода в Visual Studio -> Refactor -> Извлечь метод

Кроме того, если вам нужна дополнительная помощь в понимании того, как улучшить код, вам следует задать вопрос, указывающий на этот вопрос, на сайте обзора кода: здесь

Answer 2

Вы повторно связываете выпадающий список (и, следовательно, стираете 'SelectedValue') в вашем методе Page_PreRender.Оберните метод в

protected void Page_PreRender(object sender, EventArgs e)
{
  if( !IsPostBack){
    //your current code
  }
}

, и он должен работать.