Question

我正在C＃console中编写骰子滚动程序。我正在给两个输入

输入骰子的大小和
输入您想玩的时间。

假设骰子大小是我玩过的6倍和10倍。

Output is coming:
1 was rolled  2 times
2 was rolled  7 times
3 was rolled  8 times
4 was rolled  7 times
5 was rolled  4 times
6 was rolled  5 times

总计：33（对于每次执行都不固定，不会更改）

但我的要求是这总数应该是播放次数。在这里我正在玩10次，所以总数应该是10而不是33.这应该发生在每一个新的数字......如果我玩100次总和或总数应该是100而不是任何其他数字。休息一切都会保持不变，在我的计划中没有得到预期的金额。请有人修改它。这是我的代码：

Dice.cs：

public class Dice
{
    Int32 _DiceSize;
    public  Int32 _NoOfDice;     
    public Dice(Int32 dicesize)
    {
        this._DiceSize = dicesize;         
    }
    public string Roll()
    {
        if (_DiceSize<= 0)
        {
            throw new ApplicationException("Dice Size cant be less than 0 or 0");                 
        }
        if (_NoOfDice <= 0)
        {
            throw new ApplicationException("No of dice cant be less than 0 or 0");
        }
        Random rnd = new Random();
        Int32[] roll = new Int32[_DiceSize];
        for (Int32 i = 0; i < _DiceSize; i++)
        {
            roll[i] = rnd.Next(1,_NoOfDice);
        }
        StringBuilder result = new StringBuilder();
        Int32 Total = 0;
        Console.WriteLine("Rolling.......");
        for (Int32 i = 0; i < roll.Length; i++)
        {
            Total += roll[i];
            result.AppendFormat("{0}:\t was rolled\t{1}\t times\n", i + 1, roll[i]);
        }
        result.AppendFormat("\t\t\t......\n");
        result.AppendFormat("TOTAL: {0}", Total);
        return result.ToString();
    }
}
class Program
{
    static void Main(string[] args)
    {
        Console.WriteLine("Enter no of dice size");
        int dicesize = Convert.ToInt32(Console.ReadLine());
        Console.WriteLine("How many times want to play");
        int noofplay=Convert.ToInt32(Console.ReadLine());
        Dice obj = new Dice(dicesize);
        obj._NoOfDice = noofplay;
        obj.Roll();
        Console.WriteLine(obj.Roll());           
        Console.WriteLine("Press enter to exit");
        Console.ReadKey();
    }
}

Answer 1

在我看来，你正在向后推算数学......不应该是：

// to capture just the counts
int[] roll = new int[_DiceSize];
for (int i = 0; i < _NoOfDice; i++)
{
    roll[rnd.Next(roll.Length)]++;
}

或者如果你想要实际的卷：

// to capture individual rolls
int[] roll = new int[_NoOfDice];
for (int i = 0; i < _NoOfDice; i++)
{
    roll[i] = rnd.Next(_DiceSize) + 1; // note upper bound is exclusive, so +1
}

Answer 2

您正在每次迭代时创建一个新的Random实例。这不是一件好事，因为它会影响结果的均匀分布。将Random实例保留在字段中，而不是每次都创建一个新实例。

public class Dice {
    private Random rnd = new Random();

    // ... don't create a new random in `Roll` method. Use `rnd` directly.
}

Answer 3

首先，以下for循环是错误的：

for (Int32 i = 0; i < _DiceSize; i++)
{
   roll[i] = rnd.Next(1,_NoOfDice);
}

显然你切换了_DiceSize和_NoOfDice。正确的循环看起来像

for (Int32 i = 0; i < _NoOfDice; i++)
{
   roll[i] = rnd.Next(1,_DiceSize);
}

因此，行

Int32[] roll = new Int32[_DiceSize];

必须更改为

Int32[] roll = new Int32[_NoOfDice];

也许你应该考虑重命名这些变量，所以它更清楚，这意味着什么。

如果您以这种方式修改代码，您会提到您的分析将无法以您实现的方式工作。实际上，你所展示的是每次投掷的结果，如果我理解你的话，这不是你想要的。

更新：

对不起，我误解了你。您确实希望显示每个卷的结果。那么，为什么不将StringBuilder.AppendFormat移动到“滚动循环”？

更新＃2：

对我来说，下面的Die-class完全按你想要的方式工作：

public class Die
{
    private int maxValue;
    private int numberOfRolls;
    private Random random;

    public Die(int maxValue, int numberOfRolls)
    {
        this.maxValue = maxValue;
        this.numberOfRolls = numberOfRolls;
        this.random = new Random();
    }

    public string Roll()
    {
        StringBuilder resultString = new StringBuilder();

        for (int i = 0; i < this.numberOfRolls; i++)
        {
            resultString.AppendFormat("Roll #{0} - Result: {1}" + Environment.NewLine, i + 1, this.random.Next(1, maxValue + 1));
        }

        return resultString.ToString();
    } 
}

希望我能帮助你。

Answer 4

根据Mehrdad和Marc Gravell，这是您必须使用的完整代码。玩得开心。

  public class Dice
  {
    private Random rnd = new Random();

    Int32 _DiceSize;
    public Int32 _NoOfDice;
    public Dice(Int32 dicesize)
    {
      if (dicesize <= 0)
      {
        throw new ApplicationException("Dice Size cant be less than 0 or 0");
      }

      this._DiceSize = dicesize;
    }
    public string Roll()
    {

      if (_NoOfDice <= 0)
      {
        throw new ApplicationException("No of dice cant be less than 0 or 0");
      }
      // to capture just the counts
      int[] roll = new int[_DiceSize];
      for (int i = 0; i < _NoOfDice; i++)
      {
        roll[rnd.Next(roll.Length)]++;
      }
      StringBuilder result = new StringBuilder();
      Int32 Total = 0;
      Console.WriteLine("Rolling.......");
      for (Int32 i = 0; i < roll.Length; i++)
      {
        Total += roll[i];
        result.AppendFormat("{0}:\t was rolled\t{1}\t times\n", i + 1, roll[i]);
      }
      result.AppendFormat("\t\t\t......\n");
      result.AppendFormat("TOTAL: {0}", Total);
      return result.ToString();
    }
  }
  class Program
  {
    static void Main(string[] args)
    {
      Console.WriteLine("Enter no of dice size");
      int dicesize = Convert.ToInt32(Console.ReadLine());
      Console.WriteLine("How many times want to play");
      int noofplay = Convert.ToInt32(Console.ReadLine());
      Dice obj = new Dice(dicesize);
      obj._NoOfDice = noofplay;
      Console.WriteLine(obj.Roll());
      Console.WriteLine("Press enter to exit");
      Console.ReadKey();
    }
  }

骰子在C＃控制台应用程序中滚动，给出错误的总计

4 个答案: