组合方法

Question

在我的应用程序中，我有很多完全相同的复制和粘贴代码，并且执行完全相同的功能（按钮单击事件等）。这些冗余代码存在于我的许多页面的代码隐藏中。因此，我决定减少代码重复，并将这些方法移至类文件中，并且仅从代码隐藏页面调用它们。

以下是我的代码中调用类文件中的方法后的按钮单击事件的示例：

#region DELETE selected users - button

protected void btnDeleteSelected_Click(object sender, EventArgs e)
{
    try
    {
        UserGvUtil.DeleteSelectedUsersAndProfiles(GridView1, Msg);
    }
    catch (Exception ex)
    {
        UserGvUtil.ExceptionErrorMessage(Msg, ex);
    }
    finally
    {
        UserGvUtil.RefreshGridView(GridView1);
    }
}

#endregion

我可以将此 try/catch 块合并到另一个方法中并将其移动到同一个类文件中吗？因此，我在点击事件中唯一拥有的就是一行代码。

这样做有意义吗？不知道为什么，但我希望文件背后的代码尽可能干净和简单，这样我就可以在一个地方进行所有编辑。

抱歉，如果我没有道理。我刚刚学习了课程和方法，很多想法涌入我的脑海。

Answer 1

您可以将 try 块内的内容移动到匿名委托中，然后将其传递给具有 try/catch 的共享方法。不过，您确实不需要将刷新放入最后。事实上，我认为您只想在 try 块成功时运行它。

Answer 2

您可以手动连接事件处理程序。

btnDeleteSelected1.Click += Events.BtnDeleteSelected_Click;
btnDeleteSelected2.Click += Events.BtnDeleteSelected_Click;
...
btnDeleteSelected3.Click += Events.BtnDeleteSelected_Click;

public static class Events
{
  public static BtnDeleteSelected_Click(object sender, EventArgs e)
  {
     ...
  }
}

编辑（对于投反对票的人：？？？）
该代码将为您提供一个单行代码，当自定义事件全部相同时，您不必担心编写自定义事件。

另外，如果实用程序方法具有相同的签名，您可以拥有一个通用方法：

public void ExecuteGvMethod(Action<GridView, string> gvMethod, GridView gv, string msg)
{
    try
    {
        gvMethod(gv, msg);
    }
    catch (Exception ex)
    {
        UserGvUtil.ExceptionErrorMessage(msg, ex);
    }
    finally
    {
        UserGvUtil.RefreshGridView(GridView1);
    }
}

在代码中：

public static class Events
{
  public static BtnDeleteSelected_Click(object sender, EventArgs e)
  {
    ExecuteGvMethod(UserGvUtil.DeleteSelectedUsersAndProfiles, (GridView)sender, "hi of whatever");
  }
}

Answer 3

我冒着投反对票的风险，但这里是我的 2 美分：

不要使用 try/catch，而是使用返回状态代码的方法。
例如（只是一个想法，您可以使用更复杂的类，而不是使用枚举）：

public enum StatusCode
{
    Success = 1,
    Error =2
}

public class UserGvUtil
{
    public StatusCode getStatusAfterDelete(GridView GridView1, string Msg) 
    {
        try
        {
            DeleteSelectedUsersAndProfiles(GridView1, Msg);
            Return StatusCode.Success;
        }
        catch (Exception ex)
        {
            UserGvUtil.ExceptionErrorMessage(Msg, ex);
            Return StatusCode.Error;
        }
    }

//your other methods here
}

然后在后面的代码中：

protected void btnDeleteSelected_Click(object sender, EventArgs e)
{
    StatusCode sc = UserGvUtil.getStatusAfterDelete(GridView1, Msg);

    //then do something with the status code if you have to:
    if (sc==StatusCode.Error) throw new Exception("Error deleting users and profiles");
    else UserGvUtil.RefreshGridView(GridView1);

}

这样，如果您认为它会影响性能等，您可以稍后更改您的 try/catch...
希望有帮助。

Answer 4

我有很多复制和粘贴代码
这是完全相同的......（按钮
点击事件等）

只需将多个单击处理程序后面重复的所有代码移至单独类中的方法中，并传递所需的任何内容（在本例中为 GridView 和 MSG 对象）作为参数。如果它可以节省大量重复，那么这样做是有意义的。 DRY（不要重复自己）是一条有效的原则。