代码不断吞噬内存

Question

因此，基本上，这段代码会询问我要生成多少个项目，但此后只要继续运行就不会产生响应。 只是吃公羊。

我想要它做的是询问要生成的项目数量，然后说明其名称和统计信息。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace TestingRNG
{
class Program
{
    public static Random Random = new Random();
    public static int dropRate, itemAmmount;
    public static List<Items> Loot = new List<Items>();
    public static List<string> WeaponList = new List<string> { "Staff", "Sword", "Dagger", "Wand", "Axe", "Mace" };
    static void Main(string[] args)
    {
        Programs();
    }
    public static void Programs()
    {
        Console.WriteLine("How many items do you want to generate");
        itemAmmount = Int32.Parse(Console.ReadLine());

        for (int i = 0; 0 < itemAmmount; i++)
        {
            dropRate = Random.Next(1, 6);
            Loot.Add(new Items(dropRate));
        }
        foreach (var Item in Loot)
        {
            Console.WriteLine(Item.Name);
            if (WeaponList.Contains(Item.ItemType))
            {
                Console.WriteLine(Item.Damage);
            }
            else
            {
                Console.WriteLine(Item.HP);
                Console.WriteLine(Item.MANA);
            }
            Console.ReadLine();

        }
    }
}

}

Answer 1

for (int i = 0; 0 < itemAmmount; i++)

您没有检查循环计数器，而是将0与itemAmount进行比较。 那是一个无限循环。

尝试for (int i = 0; i < itemAmmount; i++) 。

您可以在Visual Studio中使用代码段来避免此类错误。 输入'for'并按两次Tab键，将插入for循环的代码段，可以重命名i和length ：

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

}

Answer 2

for (int i = 0; 0 < itemAmmount; i++)
{
    dropRate = Random.Next(1, 6);
    Loot.Add(new Items(dropRate));
}

您设置itemAmmount ，并且永远不要更改它。 循环退出的唯一方法是将itemAmmount减小（小于0）。

修复您的for循环，使其具有逻辑意义：

for(int i=0; i<itemAmmount; i++)