Windows Forms主窗体访问独立类内PictureBox的解决方案咨询

嘿，我来帮你搞定这个课程项目的问题！你现在的困扰是主窗体的Timer事件没法访问Coin类里的私有picCoin控件，没法实现硬币下落的重力逻辑对吧？这其实是个典型的面向对象封装问题，我们不用把picCoin改成公开的，只要给Coin类加几个合适的公共方法/属性就能完美解决，还能保持代码的整洁性。

推荐方案：给Coin类封装下落逻辑 #

最符合面向对象设计的方式是把硬币的下落行为封装在Coin类内部，主窗体只需要调用对应的方法即可，不用直接操作内部的PictureBox控件。

修改后的Coin类代码 #

class Coin { 
    private PictureBox picCoin = new PictureBox(); 
    public Coin() { 
        picCoin.BackColor = Color.Transparent; 
        picCoin.ImageLocation = @"C:\Users\sebfre1104\source\repos\SPEL\SPEL\Resources\hamburger.png"; 
        picCoin.Width = 100; 
        picCoin.Height = 100; 
        picCoin.SizeMode = PictureBoxSizeMode.StretchImage; 
    } 
    public void DrawTo(Form f) { 
        f.Controls.Add(picCoin); 
    } 
    public Rectangle getBounds() { 
        return picCoin.Bounds; 
    } 
    public void setposition(int x, int y) { 
        picCoin.Location = new Point(x, y); 
    }

    // 新增：处理下落的方法，封装重力逻辑
    public void MoveDown(int speed)
    {
        picCoin.Top += speed;
    }

    // 新增：从窗体移除硬币的方法
    public void RemoveFromForm()
    {
        picCoin.Parent?.Controls.Remove(picCoin);
    }
}

主窗体Timer事件修改 #

现在你可以在Timer的Tick事件里遍历所有硬币，调用MoveDown方法实现下落，还能顺便处理接取和出界逻辑：

private void TimerGravitation_Tick(object sender, EventArgs e) { 
    // 玩家的重力逻辑保持不变
    if(!picSpelare.Bounds.IntersectsWith(picMark.Bounds) && IsJumping==false) { 
        picSpelare.Top += 10; 
    }

    // 遍历所有硬币，应用重力下落（反向遍历避免删除元素时出错）
    for(int i = clist.Count - 1; i >= 0; i--)
    {
        Coin currentCoin = clist[i];
        // 让硬币下落，速度可以根据需求调整
        currentCoin.MoveDown(10);

        // 检测玩家是否接住硬币
        if(picSpelare.Bounds.IntersectsWith(currentCoin.getBounds()))
        {
            poäng++;
            // 更新分数显示（如果有对应的Label的话）
            // lblPoäng.Text = $"Poäng: {poäng}";
            // 移除硬币
            clist.RemoveAt(i);
            currentCoin.RemoveFromForm();
        }
        // 检测硬币是否超出窗体底部，自动移除
        else if(currentCoin.getBounds().Top > this.Height)
        {
            clist.RemoveAt(i);
            currentCoin.RemoveFromForm();
        }
    }
}

备选方案：暴露位置属性（简单但封装性稍弱） #

如果你觉得封装方法太麻烦，也可以给Coin类添加可读写的位置属性，让主窗体直接修改硬币的位置：

class Coin { 
    // ... 原有代码不变 ...

    // 新增Top属性，用于读取和设置硬币的垂直位置
    public int Top
    {
        get { return picCoin.Top; }
        set { picCoin.Top = value; }
    }

    // 也可以添加Left属性，方便后续调整水平位置
    public int Left
    {
        get { return picCoin.Left; }
        set { picCoin.Left = value; }
    }
}

然后在Timer事件里直接修改：

foreach(var coin in clist)
{
    coin.Top += 10;
}

不过这种方式会让主窗体直接操作硬币的位置，不如第一种方案把行为封装在Coin类里更规范。

额外提醒 #

别忘了在创建硬币的时候，要把它添加到clist列表里，比如在生成硬币的按钮事件或者另一个Timer里：

// 示例：生成一个随机位置的硬币
Random rnd = new Random();
Coin newCoin = new Coin();
newCoin.setposition(rnd.Next(0, this.Width - 100), 0); // 避免超出窗体宽度
newCoin.DrawTo(this);
clist.Add(newCoin);

内容的提问来源于stack exchange，提问作者Sebastian66