当前位置：文江博客话题详情

外部更新的 Winform 的最佳实践

发布于 2024-09-27 20:08:42 字数 65 浏览 2 评论 0原文

我想知道将标签公开以便其他类可以更改它们并获得它们的值是否是一个好主意。这是个好主意吗？如果不是的话，那该怎么办呢？

原文

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

卸妝后依然美 2024-10-04 20:08:42

我不会公开这个标签。

最好添加一个特定于标签显示内容的公共方法，并让它更新标签。

例如，如果您的标签是“系统状态”标签，您可能需要添加（到您的表单/用户控件）：

public void SetStatusInformation(string currentStatus)
{
     this.statusLabel.Text = currentStatus;
}

这允许您稍后更改此信息的显示方式（如果您想使用不同的控件）），并且还简化了您的 API，因为公共方法对用户来说非常清楚。

I wouldn't make the label public.

It would be better to add a public method that was specific to what the label was displaying, and have it update the label.

For example, if your label was a "System status" label, you might want to add (to your Form/UserControl):

public void SetStatusInformation(string currentStatus)
{
     this.statusLabel.Text = currentStatus;
}

This allows you, later, to change how this information is displayed (in case you want to use a different control), and also simplifies your API, since the public methods are very clear to the user.

回复收藏 0 原文

滴情不沾 2024-10-04 20:08:42

这是一个坏主意。 WinForms 留下了许多“做 X 的最佳方法是什么？”的问题。问题开放；最好的答案是遵循既定的模式和实践（这不是 WinForms 特有的）。

阅读 MVP 或 MVC 模式。它们都是高级模式，专注于将特定于 UI 的代码与业务逻辑分开。如果没有这种分离，您的应用程序很快就会成为维护噩梦，并且本应简单的事情会变得更加复杂。

对于您的特定场景，您可能最终会得到一个模型（您的业务逻辑），它使用数据绑定在 WinForms 屏幕上显示其数据。当 UI 发生更改时，模型将接收更改，并且该更改将通过数据绑定传播到 UI。

回复收藏 0 原文

╰沐子 2024-10-04 20:08:42

我建议包装在 setter 属性或方法中，因为如果调用者来自另一个线程，您很可能必须在窗口的主线程上添加日志记录或重新调用之类的操作。我发现在公开允许客户端更新任何图形的功能时，始终使用如下所示的代码会更容易。

public void SetStart()
    {
        if (this.InvokeRequired)
        {
            this.Invoke((MethodInvoker)delegate()
            {
                this.SetStart();
            });
        }
        else
        {
            progressBar1.Value = 0;
            progressBar1.Visible = true;
        }
    }

I would suggest wrapping in a setter property or method because it's very possible you'll have to do something like add logging or re-call on window's main thread if the caller is from another one. I found it easier to just always use code like the following when exposing functionality that lets clients update anything graphical.

public void SetStart()
    {
        if (this.InvokeRequired)
        {
            this.Invoke((MethodInvoker)delegate()
            {
                this.SetStart();
            });
        }
        else
        {
            progressBar1.Value = 0;
            progressBar1.Visible = true;
        }
    }

回复收藏 0 原文

~没有更多了~