代码一直在吃记忆

时间:2015-10-07 18:31:38

标签: c# linq memory

所以基本上这段代码询问我想要生成多少项,但之后只是继续运行而没有响应。 只是吃了公羊。

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

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();

        }
    }
}

}

2 个答案:

答案 0 :(得分:5)

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

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

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

您可以在Visual Studio中使用代码段来避免此类错误。输入&#39; for&#39;并按Tab键两次,将插入for循环的代码段,可以重命名ilength

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

}

答案 1 :(得分: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++)