Как обновить значение переменной?

Question

[AlexsandrLomaev]

Всем доброго!

private void timer1_Tick(object sender, EventArgs e)
{
    switch(comboBox1.Text)
    {
        case "Дробовик":
            Damage = 36;
            Clip  = 6;
            Stock = 12;
            break;
        case "Автомат":
            Damage = 3;
            Clip  = 30;
            Stock = 60;
            break;
        case "Пистолет":
            Damage = 11;
            Clip  = 7;
            Stock = 21;
            break;
        case "Базука":
            Damage = 51;
            Clip  = 1;
            Stock = 2;
            break;
      }
}

таймер обновляет значение comboBox
но когда помещаю туда переменную с патронами вида -
label1 = Convert.ToString(Clip);
при клике на кнопку патроны должны отнимать -1 но из-за таймера оно возвращается в исходное значение, я предпологаю что я не правильно в таймер переменную вставляю.
прошу помощи

Comments

[freeExec]

1. удалите свою картинку и впишите текс
2. напиши что у вас в первом случае, какие данные приходят, какие данные вы куда сохранили
3. какие данные приходят во второй раз и что у вас сохранилось
4. какие данные вы читаете, что вы получаете по факту, а что вы хотели получить.

[AlexsandrLomaev]

freeExec, Всё содержимое таймера (активируется после загрузки формы)

private void timer1_Tick(object sender, EventArgs e)
        {
            switch (comboBox1.Text)
            {
                case "Дробовик":
                    Damage = 36;
                    Clip = 6;
                    Stock = 12;
                    break;

                case "Автомат":
                    Damage = 3;
                    Clip = 30;
                    Stock = 60;
                    break;

                case "Пистолет":
                    Damage = 11;
                    Clip = 7;
                    Stock = 21;
                    break;
                case "Базука":
                    Damage = 51;
                    Clip = 1;
                    Stock = 2;
                    break;
            }


        }

А вот само событие клика, тут проверка всего

private void button3_Click(object sender, EventArgs e)
        {
            if (comboBox1.Text == "Выбрать Оружие")
            {
                MessageBox.Show("Выберите оружие", "Warning", MessageBoxButtons.OK, MessageBoxIcon.Warning);
            }
            else
            {
                if (Clip == 0)
                {
                    MessageBox.Show("В обойме закончились потроны!", "Warning", MessageBoxButtons.OK, MessageBoxIcon.Warning);
                }
                else
                {
                    XP_bot = XP_bot -= Damage;
                    label7.Text = Convert.ToString(XP_bot);

                    Clip = Clip -= 1;
                    label4.Text = Convert.ToString(Clip);
                  

                }
            }
        }

[freeExec]

AlexsandrLomaev, Что это за фигня? Clip = Clip -= 1;

[AlexsandrLomaev]

freeExec, точно не скажу, но где-то искал ответ давно, и наткнулся на такой код.

[freeExec]

AlexsandrLomaev, в программировании без понимании написанного далеко не уедешь.

[AlexsandrLomaev]

freeExec, все понимаю, нравоучения и т.д, но может подскажете? Как обновлять содержимое лабел

[freeExec]

AlexsandrLomaev, содержимое лабел у вас нормально обновляется.

[AlexsandrLomaev]

freeExec, Я понимаю, но оно не дает при клике на бт изменять значение, а приводит его в исхоную,

Answer 1

[Василий Банников]

Каждый тик таймера вы устанавливаете определённое значение переменной.
В вашем случае нужно где-то сохранять данные об оставшемся количестве.
Предлагаю вам завести класс, который хранит количество всего и поместить их в словарь.
Будет что-то наподобие

private Dictionary<string, Item> Inventory = new () {
    ["Дробовик"] = new () { Damage = 36, Clip = 30, Stock = 60 },
    ["Автомат"] = new () { Damage = 30, Clip = 30, Stock = 60 },
    ["Базука"] = new () { Damage = 51, Clip = 1, Stock = 2 }
};
public Item SelectedItem {get; set;}
private void timer1_Tick(object sender, EventArgs e) {
  SelectedItem = Inventory[combobox.Text];
}

А потом там где вы выводите текст:
label1.Text = SelectedItem?.Clip.ToString() ?? "";