【问题标题】:Simpler way to write this code? [closed]编写此代码的更简单方法? [关闭]
【发布时间】:2014-03-18 09:22:36
【问题描述】:

这是我的名为“编辑”的按钮,当您在购物篮中有条目并单击该条目并单击“编辑”时,它会打开一个新窗口,允许您编辑条目、产品名称、数量或价格。这就是我所拥有的,它可以编译并运行良好,但是有没有更简单的方法来编写它?

private void btn_Edit_Click(object sender, EventArgs e)
{
    if (lst_Results.SelectedIndex >= 0)
    {
        // Want to edit the value of the Item
        Edit editbutton = new Edit();

        editbutton.NameOfItem =
        basket.Items[lst_Results.SelectedIndex].ItemName;
        editbutton.Quantity = basket.Items[lst_Results.SelectedIndex].Quantity;
        editbutton.ReplacementValue =
        basket.Items[lst_Results.SelectedIndex].Price;


        if (editbutton.ShowDialog() == DialogResult.OK)
        {
            basket.UpdateReplacementValue(basket.Items[lst_Results.SelectedIndex].ItemName, editbutton.Quantity, editbutton.ReplacementValue);
            RenderLibrary();
        }
    }
}

【问题讨论】:

  • 你能告诉我们这段代码的问题吗?
  • 这应该已经发布在 codereview.stackexchange.com
  • 这个问题属于codereview.stackexchange.com

标签: c# simplify


【解决方案1】:

至少,你可以写出重复的数组访问。

// Want to edit the value of the Item
Edit editbutton = new Edit();

var item = basket.Items[lst_Results.SelectedIndex];

editbutton.NameOfItem = item.ItemName;
editbutton.Quantity = item.Quantity;
editbutton.ReplacementValue = item.Price;

if (editbutton.ShowDialog() == DialogResult.OK)
{
    basket.UpdateReplacementValue(item.ItemName, editbutton.Quantity, editbutton.ReplacementValue);
    RenderLibrary();
}

此外,您可能只想将对象作为参数传递给控件,​​并在此处使用数据绑定对其进行编辑。

【讨论】:

  • 谢谢,这使它看起来更容易阅读:)
  • 抱歉,我该怎么做呢? ^^
猜你喜欢
  • 2010-12-08
  • 1970-01-01
  • 2012-05-08
  • 1970-01-01
  • 2018-05-30
  • 1970-01-01
  • 2021-02-14
  • 2011-11-19
  • 1970-01-01
相关资源
最近更新 更多