定义方法的好方法

Question

实现方法调用的最佳/好方法是什么？

例如：以下内容通常被认为是最佳实践。如果两者都不好，那么什么被认为是最佳实践。

选项 1：

   private void BtnPostUpdate_Click(object sender, EventArgs e)
    {
        getValue();
    }

    private void getValue()
    {
        String FileName = TbxFileName.Text;
        int PageNo = Convert.ToInt32(TbxPageNo.Text);

        // get value from Business Layer
        DataTable l_dtbl = m_BLL.getValue(FileName, PageNo);

        if (l_dtbl.Rows.Count == 1)
        {
            TbxValue.Text = Convert.ToInt32(l_dtbl.Rows[0]["Value"]);
        }
        else
        {
            TbxValue.Text = 0;
        }
    }

选项 2：

    private void BtnPostUpdate_Click(object sender, EventArgs e)
    {
        String FileName = TbxFileName.Text;
        int PageNo = Convert.ToInt32(TbxPageNo.Text);

        int Value = getValue(FileName, PageNo);

        TbxValue.Text = Value.ToString();

    }

    private int getValue(string FileName, int PageNo)
    {
        // get value from Business Layer
        DataTable l_dtbl = m_BLL.getValue(FileName, PageNo);

        if (l_dtbl.Rows.Count == 1)
        {
            return Convert.ToInt32(l_dtbl.Rows[0]["Value"]);
        }
        return 0;
    }

我知道我们可以直接传递参数而不分配给局部变量...我的问题更多是关于方法定义及其处理方式。

Answer 1

如果您自动订阅事件，我认为拥有一个带有事件处理程序签名的方法并不是特别糟糕，该方法仅委托给具有您需要的“真实”签名的方法（在本例中，没有参数） 。

如果您手动订阅，则可以改用 lambda 表达式：

postUpdateButton.Click += (sender, args) => PostUpdate();

然后在 PostUpdate 中完成工作。是否将 PostUpdate 拆分为两种方法，一种处理 UI 交互，一种处理 BLL 交互，这取决于您。在这种情况下，我认为这并不重要。

不过，如何构建 UI 逻辑以使其可测试是完全不同的事情。我最近成为 MVVM 模式的粉丝，但我不知道它对您的特定场景有多大的适用性（它实际上是围绕 Silverlight 和 WPF 设计的）。

不过还有其他一些评论：

• 按照惯例，参数应该采用驼峰命名法，而不是帕斯卡命名法。
• 您真的相信您可以从在局部变量中添加 l_ 前缀中获益吗？这不是很明显他们是本地人吗？就我个人而言，我并不热衷于此处显示的大多数变量名称 - 考虑根据变量的含义而不是其类型来命名。
• 使用DataTable 返回信息是一种容易出错的方法。为什么 BLL 不能返回一个 int？ 来指示该值（或缺少值）？

Answer 2

如果我不实现 mvc，这就是我喜欢的。我假设这里有网络。

我首先执行选项 2，但不是让按钮代码设置文本 id，而是创建一个属性来设置文本框值。

我这样做是因为如果其他东西设置了文本框值，那么您将重复代码。如果更改名称或控件类型，情况会很糟糕。

Answer 3

根据您的示例，选项 2 是最佳选择。选项 1 了解您的表单以及如何在表单上显示数据，这违反了 SRP。