是否可以减少基于重复排列的 if 语句？

Question

你好，我正在做这个数学游戏，对于我的最后一个场景，我做了很多重复的代码，但我不确定是否有办法简化它，我在下面提供了链接，所以也许一些经验丰富的程序员可能有一些更优雅的解决方案！例如，我试图生成类似 (a[]b)²[]c[]d 的每个排列，其中括号将被 +、-、* 或 / 替换。我一直在做的只是创建随机百分比 if 语句来选择像“(a+b)²/cd”这样的特定版本是否有可能比我一直在做的“蛮力”和可读性更低的方法？

if(UnityEngine.Random.Range(0,101)>50){
        // 50% of being (a+b)²(c)+d
    if(UnityEngine.Random.Range(0,101)>50){
        ans = ((int) Mathf.Pow((float) a+ (float) b, 2))*c+d;
        input.text = "("+a+"+"+b+")"+"²"+"("+c+")"+"+"+d+"=";
        Debug.Log("Problem ID: 78");
        // 50% of being (a+b)²(c)-d
    } else {
        ans = ((int) Mathf.Pow((float) a+ (float) b, 2))*c-d;
        input.text = "("+a+"+"+b+")"+"²"+"("+c+")"+"-"+d+"=";
        Debug.Log("Problem ID: 79");
    }
    // 50% of being (a-b)²(c)[]d
} else {
    // 50% of being (a-b)²(c)+d
    if(UnityEngine.Random.Range(0,101)>50){
        ans = ((int) Mathf.Pow((float) a- (float) b, 2))*c+d;
        input.text = "("+a+"-"+b+")"+"²"+"("+c+")"+"+"+d+"=";
        Debug.Log("Problem ID: 80");
        // 50% of being (a-b)²(c)-d
    } else {
        ans = ((int) Mathf.Pow((float) a- (float) b, 2))*c-d;
        input.text = "("+a+"-"+b+")"+"²"+"("+c+")"+"-"+d+"=";
        Debug.Log("Problem ID: 81");
    }

（下面的 Pastebin 了解更多上下文） https://pastebin.pl/view/d1bfb99e

Answer 1

我很欣赏你想让你的代码更具可读性的愿望。基本思想是拆分 (a) 定义、(b) 选择和 (c) 应用您的运算符。

• 第 1 步：定义Operators。每个都Operator结合了一个数学运算（例如Addwould be (a, b) => a + b）和一个符号（例如Addwould be "+"）。

  class Operator
  {
      public Func<int, int, int> Calculate { get; }
      public string Symbol { get; }
  
      public Operator(Func<int, int, int> calculate, string symbol)
      {
          Calculate = calculate;
          Symbol = symbol;
      }
  }
  
  private Operator Add = new Operator((a, b) => (a + b), "+");
  private Operator Subtract = new Operator((a, b) => (a - b), "-");
  

• 第 2 步：然后您随机选择您的运算符（我使用了System.Random，因为我不熟悉 Unity，但可以随意将其替换为您选择的随机数生成器）：

  var rnd = new Random();
  
  private (Operator op1, Operator op2, int problemId) RandomlyChooseProblem()
  {
      switch (rnd.Next(4))
      {
          case 0: return (Add, Add, 78);
          case 1: return (Add, Subtract, 79);
          case 2: return (Subtract, Add, 80);
          case 3: return (Subtract, Subtract, 81);
          default: throw new InvalidOperationException("This should not happen.");
      }
  }
  

• 第 3 步：您应用它们：

  var (op1, op2, problemId) = RandomlyChooseProblem();
  
  ans = op2.Calculate((int)Math.Pow(op1.Calculate(a, b), 2) * c, d);
  input.text = $"(a{op1.Symbol}b)²*c{op2.Symbol}d");
  Debug.Log($"Problem ID: {problemId}");
  

添加新的运算符（例如Multiply）或新的问题变体（例如(Add, Multiply, 82)）现在只是一行代码。