在 Visual Studio 中使用带有图片框的标签

Question

我正在练习 C# 编程，并且正在学习我在网上找到的教程。我对 Visual Studio 女士产生了兴趣，目前一直在自学。我已经对教程进行了自己的编辑，但尽可能接近教程。一切似乎都很好，除了代码中有一部分我无论做什么或研究都无法弄清楚。在教程中，它让我创建了 30 个图片框。每个图片框都标有一个单词名称，例如块或砖块或您选择的任何名称。我认为这是为了更容易一次使用所有图片框而不是一次执行一个 if/else 语句。我明白只说“如果图片框有标签“砖块”然后做 XYZ 会更省时间。 无论如何，我按照代码到了 T 但 visualbasic 说

"可能是非预期的引用比较；要进行值比较，请将左侧转换为类型 'string'

这是给我的代码部分

foreach (Control x in this.Controls)
        {
            if (x is PictureBox && x.Tag == "blockies")
            {
                if (pBall.Bounds.IntersectsWith(x.Bounds))
                {
                    this.Controls.Remove(x);
                    pBally = -pBally;
                    score++;
                }
            }
        }

它有一条从 x.Tag 到 blockies 的绿色波浪线” 谢谢你的帮助。 附言。窗体

Answer 1

我发现使用 Linq 更具可读性。

解决比较警告问题和在枚举集合时从集合中删除元素的问题：

foreach(var pb in this.Controls
                    .OfType<PictureBox>()
                    .Where(x => (string)x.Tag == "blockies")
                    .Where(x => pBall.Bounds.IntersectsWith(x.Bounds))
                    .ToList())
{
    this.Controls.Remove(pb);
}

Answer 2

那是因为Tag 不是字符串。你需要改为x.Tag.ToString() == "blockies"：

foreach (Control x in this.Controls)
{
    if (x is PictureBox && x.Tag != null && x.Tag.ToString() == "blockies")
    {
        if (pBall.Bounds.IntersectsWith(x.Bounds))
        {
            this.Controls.Remove(x);
            pBally = -pBally;
            score++;
        }
    }
}

Answer 3

如果你打开Control.Tag，你会看到它的定义。

public object Tag { get; set; }

您正在尝试将对象与字符串进行比较，因此会显示此错误消息。

你应该做的是：

if (x is PictureBox && x.Tag.ToString() == "blockies")

还有一点你应该尽量避免箭头代码结构，它真的很难阅读：

    foreach (Control x in this.Controls)
    {
        if (!(x is PictureBox))
           continue;

        //this is needed if you want to use some specific property of the PictureBox.
        PictureBox ctl = (PictureBox)x;

        if(ctl.Tag.ToString() != "blockies")
            continue;

        if (!pBall.Bounds.IntersectsWith(ctl.Bounds))
             continue;

         //Also this line will create you a problem, because you will change the Control collection 
         //when you try to enumerate it. This should throw you an exception. Better make the control not visible.
         //this.Controls.Remove(x);

         x.Visible = false;

         pBally = -pBally;
         score++;
    }

在这种情况下，您的代码具有更好的可读性。