C # Как мне распознать столкновение между двумя объектами

Question

Form1.cs

namespace SpaceInvadersV3
{
public partial class Form1 : Form
{
    public bool isPressed;
    Shooter player;
    List<Missile> bullet;
    List<Enemy> pirate;
    Boundary bottom;
    Boundary top;
    Boundary left;
    Boundary right;



    public Form1()
    {
        InitializeComponent();

        player = new Shooter(450,460);
        bullet = new List<Missile>();
        pirate = new List<Enemy>();
        for (int i = 0; i < 10; i++)
        {
            Enemy temp = new Enemy();
            pirate.Add(temp);         
        }
    }

    private void timer1_Tick(object sender, EventArgs e)
    {
        player.Move();
        foreach (Missile b in bullet)
        {
            b.Move();
        }

        foreach (Enemy p in pirate)
        {
            p.Move();
        }

        pictureBox1.Invalidate();

        if (IsColliding(player, pirate) == true) 
        {
            gameOver();
        }
    }

Ошибка в "pirate" говорит о том, что он не может преобразовать из 'System.Collections.Generic.List<SpaceInvadersV3.Enemy>' в 'SpaceInvadersV3.Enemy' Я пытался изменить приведенную ниже функцию 'IsColliding' с (Enemy b) на (List<Enemy> b), но затем это не происходит распознает b.Bottom и говорит, что List<Enemy> не содержит определения для Bottom.

    // Keybinds
    private void Form1_KeyDown(object sender, KeyEventArgs e)
    {
        if (e.KeyCode == Keys.A)
        {
            player.goleft = true;
        }

        if (e.KeyCode == Keys.D)
        {
            player.goright = true;
        }

        if (e.KeyCode == Keys.W)
        {
            player.goup = true;
        }

        if (e.KeyCode == Keys.S)
        {
            player.godown = true;
        }

        if (e.KeyCode == Keys.Space)
        {
            Missile temp = new Missile(player.x, player.y);
            bullet.Add(temp);
        }

    }

    private void Form1_KeyUp(object sender, KeyEventArgs e)
    {
        if (e.KeyCode == Keys.A)
        {
            player.goleft = false;
        }

        if (e.KeyCode == Keys.D)
        {
            player.goright = false;
        }

        if (e.KeyCode == Keys.W)
        {
            player.goup = false;
        }

        if (e.KeyCode == Keys.S)
        {
            player.godown = false;
        }

    }
        // keybinds

    private void pictureBox1_Paint(object sender, PaintEventArgs e)
    {
        player.Draw(e.Graphics);

        foreach (Missile b in bullet)
        {
            b.Draw(e.Graphics);
        }

        foreach (Enemy p in pirate)
        {
            p.Draw(e.Graphics);
        }
    }

    private bool IsColliding(Shooter a, Enemy b)
    {
        bool colliding = true; // presume collision
        if (a.Top() > b.Bottom())
        {
            colliding = false;
        }
        return colliding;
    }

    private void gameOver()
    {
        timer1.Stop();
        MessageBox.Show("you died");
    }
}
}

Box.cs, где классы Enemy и Shooter наследуются от

using System.Drawing;

namespace SpaceInvadersV3
{
class Box
{
    public Image pic;
    public float x;
    public float y;
    public float speed;





    public Box()
    {
        x = 0;
        y = 0;
        speed = 0;
    }
    // Image Resizing Code
    public static Image resizeImage(Image imgToResize, Size size)
    {
        return (Image)(new Bitmap(imgToResize, size));
    }
    // image resizing code


    public void Draw(Graphics g)
    {
        g.DrawImage(pic, x, y);
    }

    public float Width()
    {
        return pic.Width;
    }

    public float Height()
    {
        return pic.Height;
    }

    public float Left()
    {
        return x;
    }

    public float Right()
    {
        return x + Width();
    }

    public float Top()
    {
        return y;
    }

    public float Bottom()
    {
        return y + Height();
    }



}
}

Не думаю, что классы Shooter и Enemy действительно актуальны, но если они вам понадобятся, я их опубликую. Спасибо за вашу помощь.

Answer 1

if (IsColliding(player, pirate) == true) 

Прежде всего, никогда этого не пиши. Выглядит дилетантски говорить «если это правда , что они сталкиваются». Скажите «если эти вещи сталкиваются»:

if (IsColliding(player, pirate))

Аналогично, предпочитайте if (!whatever) if (whatever == false).

Во-вторых, , пожалуйста, используйте существительные во множественном числе для коллекций . Это должно быть pirates, а не pirate. Вы хотите подчеркнуть, что есть коллекция из них для читателя.

Ошибка в «пиратском» говорит о том, что он не может конвертировать из «Списка» в «Врага»

Ваш IsColliding берет стрелка и врага, но вы даете ему стрелок и список врагов. IsColliding не знает, как с этим справиться.

Вы уже знаете, как это исправить . Вы хотели переместить каждого врага, поэтому вы написали:

foreach (Enemy p in pirate)
{
    p.Move();
}

Теперь вы хотите проверить каждого врага на столкновения, поэтому сделайте то же самое:

foreach (Enemy p in pirate)
{
    if (IsColliding(player, p)) { ... }
}

Продвинутая техника, которую вы в конечном итоге освоите, - это использование понимания запросов для последовательностей:

var collisions = from p in pirate 
                 where IsColliding(player, p) 
                 select p;
foreach (Enemy p in collisions)
{
  ... handle the collision...
}

Но научитесь ходить, прежде чем пытаться бежать.