PictureBox почему-то ускоряет движение

@Andoku · Регистрация: 12.06.2013

Author24 — интернет-сервис помощи студентам

Написал проект, в котором при нажатии клавиши 'w' создается и двигается PictureBox по форме. Движение реализовано с помощью таймера. При нажатии клавиши 'r' пропадает изображение и PictureBox должен возвращаться на прежнюю позицию. При повторном нажатии 'w' PictureBox снова должен появляться и начинать движение. Проблема в том, что при каждом нажатии 'w' PictureBox ускоряет движение. Может, кто подскажет, как исправить? Проект в прикрепленном файле.
Код класса, в котором весь основной функционал:

Кликните здесь для просмотра всего текста

C#

public class Bullet : System.Windows.Forms.Control
    {
        int y = 300;
 
        public void CreateBullet(Form fm)
        {
            if (AbstractClass.keyPress == 'w')
            {
                AbstractClass.bullet.Location = new System.Drawing.Point(300, y);
                AbstractClass.bullet.Size = new System.Drawing.Size(8, 8);
                AbstractClass.bullet.TabIndex = 0;
                AbstractClass.bullet.Load("UpBullet.bmp");
                AbstractClass.bullet.Parent = fm;
                BulletFly();
            }
 
            if (AbstractClass.keyPress == 'r')
            {
                y = 300;
                BulletDestroy();
            }
        }
 
        private void BulletFly()
        {
            AbstractClass.timer.Tick += new EventHandler(timer_Tick);
            AbstractClass.timer.Interval = 50;
            AbstractClass.timer.Enabled = true;
        }
 
        private void timer_Tick(object sender, EventArgs e)
        {
            y++;
            AbstractClass.bullet.Location = new System.Drawing.Point(300, y);
        }
 
        private void BulletDestroy()
        {
            AbstractClass.bullet.Image = null;
            AbstractClass.timer.Enabled = false;
            AbstractClass.timer.Dispose();
        }
    }

Код Form1:

Кликните здесь для просмотра всего текста

C#

public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }
 
        private void form1_KeyPress(object sender, KeyPressEventArgs e)
        {
            AbstractClass.keyPress = e.KeyChar;
 
            Controls.Add(AbstractClass.pict);
            AbstractClass.pict.CreateBullet(this);
        }
    }

Заранее благодарен!

@Spectral-Owl · 24.10.2013, 10:33

C#

public void CreateBullet(Form fm)
        {
            if (AbstractClass.keyPress == 'w')
            {
                if(AbstractClass.bullet.Image == nul)
                {
                AbstractClass.bullet.Location = new System.Drawing.Point(300, y);
                AbstractClass.bullet.Size = new System.Drawing.Size(8, 8);
                AbstractClass.bullet.TabIndex = 0;
                AbstractClass.bullet.Load("UpBullet.bmp");
                AbstractClass.bullet.Parent = fm;
                BulletFly();
            }
            }
...

или так:

C#

         
public void CreateBullet(Form fm)
        {
            if (AbstractClass.keyPress == 'w')
            {
                if(!AbstractClass.timer.Enabled)
                {
                AbstractClass.bullet.Location = new System.Drawing.Point(300, y);
                AbstractClass.bullet.Size = new System.Drawing.Size(8, 8);
                AbstractClass.bullet.TabIndex = 0;
                AbstractClass.bullet.Load("UpBullet.bmp");
                AbstractClass.bullet.Parent = fm;
                BulletFly();
            }
            }
...

и вообще, если вызываешь timer.Dispose() при BulletDestroy(), то наверное его желательно пересоздать, до того как использовать заново?

C#

private void BulletFly()
        {
            AbstractClass.timer = new неведомое.пространство.имён.Timer()
            AbstractClass.timer.Tick += new EventHandler(timer_Tick);
            AbstractClass.timer.Interval = 50;
            AbstractClass.timer.Enabled = true;
        }

@Andoku · 24.10.2013, 11:11 **[ТС]**

timer.Dispose() убрал, ибо никакого эффекта от него не увидел (кстати, кто сможет объяснить почему?)
Ускорятся перестал после следующей модификации метода BulletFly():

Кликните здесь для просмотра всего текста

C#

public class Bullet : System.Windows.Forms.Control
    {
        int y = 300;
        int count = 0;
 
        public void CreateBullet(Form fm)
        {
            if ((AbstractClass.keyPress == 'w') && (AbstractClass.timer.Enabled == false))
            {
                AbstractClass.bullet.Location = new System.Drawing.Point(300, y);
                AbstractClass.bullet.Size = new System.Drawing.Size(8, 8);
                AbstractClass.bullet.TabIndex = 0;
                AbstractClass.bullet.Load("UpBullet.bmp");
                AbstractClass.bullet.Parent = fm;
                BulletFly();
            }
 
            if (AbstractClass.keyPress == 'r')
            {
                y = 300;
                BulletDestroy();
            }
        }
 
        private void BulletFly()
        {
            if (count == 0)
            {
                AbstractClass.timer.Tick += new EventHandler(timer_Tick);
                AbstractClass.timer.Interval = 50;
            }
            count++;
            AbstractClass.timer.Enabled = true;
        }
 
        private void timer_Tick(object sender, EventArgs e)
        {
            y++;
            AbstractClass.bullet.Location = new System.Drawing.Point(300, y);
        }
 
        private void BulletDestroy()
        {
            AbstractClass.bullet.Image = null;
            AbstractClass.timer.Enabled = false;
            Controls.Remove(AbstractClass.bullet);
        }
    }

Похоже, с каждым вызовом метода BulletFly() добавлялся еще один обработчик события timer.Tick и именно поэтому шло ускорение

	24.10.2013, 11:11