重复使用时撤消操作失败

Question

我创建了一个撤消操作来将当前值设置为列表中的前一个值。它通过从列表中删除最近的索引并将值设置为其后面的值来完成此操作。目前，当初始化时列表中没有值时，以及列表中有多个值时，它都有效。

当数组中至少有一个索引时代码的工作方式，因为以前的单元测试通过了。

在列表中的最后一项也被删除之后，当命令运行两次时，我尝试让 InvalidOperationException 通过。所以 Add(5) -> Undo() -> Undo() 举个例子。同样将 if 语句从 valDict.Count > 0 更改为 Value > 0，也会出现同样的错误。

List<int> valDict = new List<int>();

 public void Undo()
    {
        repOp1 = "Undo";
        Console.WriteLine("1: " + valDict.Count);
        if ( valDict.Count > 0)
        {
            int temp = valDict.Count - 1;
            Console.WriteLine("2: " + temp);
            valDict.RemoveAt(temp);

            Console.WriteLine("3: " + valDict.Count);
            valDict.TrimExcess();
            //Below Line is flagged as the error
            Value = valDict[valDict.Count-1];
        }
        else
        {
           throw new InvalidOperationException();
        }

    }

有人可以用新的眼光看这个，指出一个可能的解决方案，但不是一个固定的实现，因为我需要自己修复实现。

Answer 1

您可以在获取值之前控制您的 List 不为空：list.Count==0 或 !list.Any()。