Question

我已经开始工作，但我正试图找到一种更简单的方法。

我需要一个显示两个骰子图片的程序，我需要一个Class，并且至少有一个方法正确使用通过引用传递的参数。

我在课堂上使用通过引用传递的两个参数获得了我的GetRoll方法，但是我能够实现这一工作的唯一方法是制作一些荒谬的if else语句。必须有更好的方法。有任何想法吗？这是我的表格：

{'temperature': 202.48803054780439,
 'no2': 4881.734909678366,
 'co': None,
 'humidity': 200}

这是我的班级：

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace DiceGame
{
    public partial class Form1 : Form
    {
        DiceClass objectRef;
        public Form1()
        {
            InitializeComponent();
            objectRef = new DiceClass();
        }    
        private void rollEm_Click(object sender, EventArgs e)
        {
            specialMessage.Text = "";
            objectRef.RollEm();
            string str1 = "";
            string str2 = "";
            objectRef.GetRoll(ref str1, ref str2);
            die1.Text = str1;
            die2.Text = str2;
            if (objectRef.BoxCars())
            {
                specialMessage.Text = "BOX CARS!!";
            }
            else
            {
                if (!objectRef.SnakeEyes())
                    return;
                specialMessage.Text = "SNAKE EYES!!";
            }
        }
    }
}

Answer 1

使用字典确定映射一次：

Dictionary<int, string> dieRapping = new Dictionary<int, string>() {
    { 1, nL + " l " },
    { 2, "l" + nL + nL + "  l" },
    { 3, "l l" + nL + nL + "l l" },
    { 4, "l l" + nL + nL + "l l" },
    { 5, "l l" + nL + " l " + nL + "l l" },
    { 6, "l l" + nL + "l l" + nL + "l l" }
};

现在，您只需使用dieRapping[someInteger]即可在整个时间内将整数转换为字符串。所以你的整个GetRoll方法就变成了这个：

public void GetRoll(ref string first, ref string second)
{
    first = dieRapping[firstDie];
    second = dieRapping[secondDie];
}

请注意，即使没有映射，如果您单独处理first和second，也可以为自己节省很多。由于它们不依赖于另一个，您可以先处理first，然后执行second：

if (firstDie == 1)
    first = one;
else if (firstDie == 2)
    first = two;
else …

if (secondDie == 1)
    second = one;
else if (secondDie == 2)
    second = two;
else …

或者，由于first和second的逻辑相同，您可以引入另一种方法：

public string GetSingleRoll(int value)
{
    if (value == 1)
        return one;
    else if (value == 2)
        return two;
    else …
}

然后你可以在GetRoll中调用该方法两次：

first = GetSingleRoll(firstDie);
second = GetSingleRoll(secondDie);

但这只是减少重复的一些方法。在您的情况下，使用映射可能是最佳解决方案。

Answer 2

//有一种更简单的方法让这个方法在没有所有if else语句的情况下工作吗？

是：

class DiceClass
{
    private static string nL = Environment.NewLine;
    List<string> vals = new List<string>
    {
        nL + " l ",
        "l" + nL + nL + "  l",
        "l l" + nL + nL + "l l",
        "l l" + nL + nL + "l l",
        "l l" + nL + " l " + nL + "l l",
        "l l" + nL + "l l" + nL + "l l"
    };
    private const int BOX = 6;
    private int firstDie;
    private int secondDie;
    Random randomNums = new Random();
    public DiceClass()
    {
        firstDie = 0;
        secondDie = 0;
    }

    public void RollEm()
    {
        firstDie = randomNums.Next(1, 7);
        secondDie = randomNums.Next(1, 7);
    }
    public bool BoxCars()
    {
        return firstDie == 6 && secondDie == 6;
    }

    public bool SnakeEyes()
    {
        return firstDie == 1 && secondDie == 1;
    }

    public void GetRoll(ref string first, ref string second)
    {
        //str1 = GenerateString(numOne);
        //str2 = GenerateString(numTwo);

        first = vals[firstDie < 1 ? vals.Count - 1 : firstDie - 1];
        second = vals[secondDie < 1 ? vals.Count - 1 : secondDie - 1];
    }
}

上面的代码实际上会将您的值保存在列表List<string> vals中，然后将其相应的值分配给first的{{1}}和second个参数。

有关此语法GetRoll的详细信息，请执行以下操作： ?: Operator (C# Reference)

参考Rolling Dice GUI C＃传递的参数

2 个答案: