正在将对象引用存储在控件标记属性中，正常

发布于 2024-09-27 11:46:23 字数 506 浏览 2 评论 0原文

我正在为列表中的每个对象创建一组表单控件，是否可以在控件 Tag 属性中存储对该对象的引用？

我这样做是为了在控件上有一个通用的 Click 事件，这样当它们被单击时，我可以更新它们所代表的对象中的字段。

所以点击处理程序看起来像这样。

private void Item_Clicked(object sender, system.EventArgs e)
{
     if(sender.GetType() == typeof(System.Windows.Forms.Label))
     {
          System.Windows.Forms.Label label = (System.Windows.Forms.Label)sender;
          MyObject myObject = label.Tag;
          myObject.Value = true;
     }
}

在这种情况下这是可以接受的事情，还是有更好的方法来处理这个问题？

原文

I'm creating a group of form controls for each object in a list, is it OK to store a reference to the object in the controls Tag property?

I'm doing this so I can have a generic Click event on the controls, so when they are clicked I can update a field in the object that they represent.

So the click handler will look something like this.

private void Item_Clicked(object sender, system.EventArgs e)
{
     if(sender.GetType() == typeof(System.Windows.Forms.Label))
     {
          System.Windows.Forms.Label label = (System.Windows.Forms.Label)sender;
          MyObject myObject = label.Tag;
          myObject.Value = true;
     }
}

Is this an acceptable thing to do in this situation, or is there a better way to handle this?

分享到QQ

分享到微博

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

发布评论

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

强辩 2024-10-04 11:46:23

是的，这是合法的，并且是 Tag 属性设计的模式之一。

这里最大的危险是另一段代码尝试为自己的功能使用相同的 Tag 属性。这会造成 Tag 属性的竞争并导致运行时错误。更安全的方法是使用 Dictionary 实例在 Label 和 MyObject 之间创建私有映射。

private Dictionary<Label,MyObject> _map = new Dictionary<Label,MyObject>();
...
private void Item_Clicked(object sender, system.EventArgs e) 
{ 
     if(sender.GetType() == typeof(System.Windows.Forms.Label)) 
     { 
          System.Windows.Forms.Label label = (System.Windows.Forms.Label)sender;
          MyObject myObject = _map[label];
          myObject.Value = true; 
     } 
}

这种方法具有字典的额外开销，但会产生更可靠的代码（恕我直言）。

Yes this is legal to do and is one of the patterns the Tag property was designed for.

The biggest danger here is that another peice of code attempts to use the same Tag property for their own feature. This would create a race for the Tag property and lead to runtime bugs. A safer approach would be to create a private map between a Label and MyObject using a Dictionary instance.

private Dictionary<Label,MyObject> _map = new Dictionary<Label,MyObject>();
...
private void Item_Clicked(object sender, system.EventArgs e) 
{ 
     if(sender.GetType() == typeof(System.Windows.Forms.Label)) 
     { 
          System.Windows.Forms.Label label = (System.Windows.Forms.Label)sender;
          MyObject myObject = _map[label];
          myObject.Value = true; 
     } 
}

This approach has the extra overhead of a Dictionary but produces more reliable code (IMHO).

回复收藏 0 原文