Повторно вызывайте функцию, пока не будет получен желаемый выход

Question

Я делаю автоматический статистический ролик и хочу, чтобы он продолжал работать до тех пор, пока не получу желаемый результат. Вот код:

class Program
{
    static void Main(string[] args)
    {
        _getStats();
    }

    static void _getStats()
    {
        Random rnd = new Random();
        int stat1 = 0;
        int stat2 = 0;
        int stat3 = 0;
        int stat4 = 0;
        int stat5 = 0;
        int stat6 = 0;

        for (int i = 0; i < 6; i++)
        {

            int roll1 = rnd.Next(1, 7);
            int roll2 = rnd.Next(1, 7);
            int roll3 = rnd.Next(1, 7);
            int roll4 = rnd.Next(1, 7);
            int tempStatTotal = 0;
            List<int> rollList = new List<int>();

            rollList.Add(roll1);
            rollList.Add(roll2);
            rollList.Add(roll3);
            rollList.Add(roll4);
            rollList.Sort();
            rollList.RemoveAt(0);

            foreach (int j in rollList)
            {
                tempStatTotal += j;
            }

            if (i == 0)
            {
                stat1 = tempStatTotal;
            }
            else if(i == 1)
            {
                stat2 = tempStatTotal;
            }
            else if (i == 2)
            {
                stat3 = tempStatTotal;
            }
            else if (i == 3)
            {
                stat4 = tempStatTotal;
            }
            else if (i == 4)
            {
                stat5 = tempStatTotal;
            }
            else
            {
                stat6 = tempStatTotal;
            }
        }

        if(stat1 == 18 || stat2 == 18 || stat3 == 18 || stat4== 18 || stat5 == 18 || stat6 == 18)
        {
            Console.WriteLine(stat1);
            Console.WriteLine(stat2);
            Console.WriteLine(stat3);
            Console.WriteLine(stat4);
            Console.WriteLine(stat5);
            Console.WriteLine(stat6);
        }
        else
        {
            Main(null); //Start the process all over again until one of the stats is 18
        }
    }
}

Когда я запускаю этот код, в некоторых случаях Консоль записывает в журнал 6 номеров, и 1 из них - 18. В большинстве случаев приложения ломаются и выдается ошибка System.StackOverflowException. Я понимаю, что это как-то связано с бесконечными циклами, и это происходит потому, что я рекурсивно вызываю свой метод, но как мне обойти это?

Answer 1

Используйте цикл do-while:

class Program
{
static void Main(string[] args)
{
    _getStats();
}

static void _getStats()
{
    do
    {
        Random rnd = new Random();
        int stat1 = 0;
        int stat2 = 0;
        int stat3 = 0;
        int stat4 = 0;
        int stat5 = 0;
        int stat6 = 0;

        for (int i = 0; i < 6; i++)
        {

            int roll1 = rnd.Next(1, 7);
            int roll2 = rnd.Next(1, 7);
            int roll3 = rnd.Next(1, 7);
            int roll4 = rnd.Next(1, 7);
            int tempStatTotal = 0;
            List<int> rollList = new List<int>();

            rollList.Add(roll1);
            rollList.Add(roll2);
            rollList.Add(roll3);
            rollList.Add(roll4);
            rollList.Sort();
            rollList.RemoveAt(0);

            foreach (int j in rollList)
            {
                tempStatTotal += j;
            }

            if (i == 0)
            {
                stat1 = tempStatTotal;
            }
            else if (i == 1)
            {
                stat2 = tempStatTotal;
            }
            else if (i == 2)
            {
                stat3 = tempStatTotal;
            }
            else if (i == 3)
            {
                stat4 = tempStatTotal;
            }
            else if (i == 4)
            {
                stat5 = tempStatTotal;
            }
            else
            {
                stat6 = tempStatTotal;
            }
        }

        if (stat1 == 18 || stat2 == 18 || stat3 == 18 || stat4 == 18 || stat5 == 18 || stat6 == 18)
        {
            Console.WriteLine(stat1);
            Console.WriteLine(stat2);
            Console.WriteLine(stat3);
            Console.WriteLine(stat4);
            Console.WriteLine(stat5);
            Console.WriteLine(stat6);
            break;
        }
    } while (true);
}
}

Answer 2

Ах, рекурсия.Это все веселье и игры, пока вы не получите исключение StackOverflowException.

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

static void _getStats()
{
    Random rnd = new Random();
    int stat1 = 0;
    int stat2 = 0;
    int stat3 = 0;
    int stat4 = 0;
    int stat5 = 0;
    int stat6 = 0;

    while (stat1 < 18 || stat2 < 18 || stat3 < 18 || stat4 < 18 || stat5 < 18 || stat6 < 18)
    {
        // rest of your code
    }

    // Move your printing after the while loop. Once one of the stats hits 18 or above,
    // print the results.
    Console.WriteLine(stat1);
    Console.WriteLine(stat2);
    Console.WriteLine(stat3);
    Console.WriteLine(stat4);
    Console.WriteLine(stat5);
    Console.WriteLine(stat6);
}