Блог

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

 int c = a - b;
if (c < 0)
{
    Random ro = new Random();
    a = ro.Next(10)   1;
    b = ro.Next(10)   1;
    lb_getal1.Text = a.ToString();
    lb_getal2.Text = b.ToString();
}
if (txt_antwoord.Text == c.ToString())
{
    MessageBox.Show("You provided the right answer!");
    score  = 1;

    Random r = new Random();
    a = r.Next(10)   1;
    b = r.Next(10)   1;
    lb_getal1.Text = a.ToString();
    lb_getal2.Text = b.ToString();
}
else
{
    MessageBox.Show("You were wrong!");
}
if (score == 5)
{
    MessageBox.Show("You answered 5 answers correctly! Well done!");
    this.Close();
    RM_menu form = new RM_menu();
    form.Show();
}
  

В вашем коде есть несколько проблем. Первое, что нужно улучшить: используйте только один экземпляр Random .

Random Класс предоставляет псевдослучайные числа из списка. Если вы создаете экземпляр с конструктором по умолчанию new Random() , экземпляр инициализируется начальным значением, взятым из текущего времени. Поскольку между вашими двумя вызовами проходит не так много времени, оба экземпляра, вероятно, используют одно и то же начальное значение, и вы получите одинаковые значения для a и b снова.
Я предлагаю создать только один экземпляр и сохранить его в переменной-члене вашего класса.

Вторая проблема: если ваша первая попытка дает отрицательный результат c , вы повторяете попытку только один раз, но это не гарантирует, что на этот раз результат будет положительным. Лучший подход — сравнить a и b и переключить, если b больше.

Таким образом, ваш код может выглядеть следующим образом:

 public class YourGame : Form // I guess it's a Form
{
    private Random randomGenerator = new Random();
    private int a;
    private int b;
    private int score;

    public void CreateQuestion()
    {
        a = randomGenerator.Next(10)   1;
        b = randomGenerator .Next(10)   1;
        if (b > a)
        {
            // switch values
            int tmp = b;
            b = a;
            a = tmp;
        }

        lb_getal1.Text = a.ToString();
        lb_getal2.Text = b.ToString();            
    }

    private void OnAnswer()
    {
        int c = a - b; // will never be negative as we checked a and b above
        if (txt_antwoord.Text == c.ToString())
        {
            MessageBox.Show("You provided the right answer!");
            score  = 1;
            CreateQuestion();
        }
        else
            MessageBox.Show("You were wrong!");

        if (score == 5)
        {
            MessageBox.Show("You answered 5 answers correctly! Well done!");
            this.Close();
            RM_menu form = new RM_menu();
            form.Show();
        }
    }
}
  

Я не пытаюсь решить эту проблему, но пытаюсь дать некоторые рекомендации о том, как убедиться, что значение не будет меньше нуля при использовании Random .

 int a, b, c;
Random ro;

ro = new Random();
b = ro.Next(10);
a = ro.Next(b, 10);
c = a - b;
  

Как вы можете видеть, сначала мы устанавливаем случайное значение в b using ro.Next(10) , а затем указываем минимальное значение с помощью ro.Next(b, 10)

При этом гарантируется, что значение при выполнении не будет меньше нуля a-b .

вы можете использовать Math.Abs

 int c = Math.Abs (a-b);