我该如何正确计算？

Question

我有一个 WPF 和 C# 和 SQLITE 系统（这里不重要），它计算人们一天喝的水量。用户有权更改 ML 中的杯子尺寸值和当天的元数据（在这种情况下并不重要）。

事物的计算是：

    takewatervalue = int.Parse(this.takeWater.Text) + takewatervalue;
    counter = takewatervalue * cupml;
    textCounter.Text = counter.ToString();

当人们没有在另一个窗口中更改 cupml 的值（如下所示）时，此计算可以正常工作，当他们更改时，会发生这种情况：

改变cupml值之前的例子：

    cupml = 200 // actual value of cupml choosen for the user

    takewatervalue = int.Parse(this.takeWater.Text)(user input = 1) + takewatervalue(0); // 
    counter = takewatervalue //1 * cupml //200; // = 200 ml -- this is right no problem here
    textCounter.Text = counter.ToString(); // showing to the user the counter and that's good

又一轮：

    cupml = 200 // actual value of cupml choosen for the user
    
    takewatervalue = int.Parse(this.takeWater.Text)(user input = 1) + takewatervalue(1); // 
    counter = takewatervalue //2 * cupml //200; // = 400 ml -- this is right no problem here
    textCounter.Text = counter.ToString(); // showing to the user the counter and that's good

但是当我们更改cupml时，会发生这种情况：

    actual conter value = 400 ML
    cupml = 300 // actual value of cupml choosen for the user in another window
    
    takewatervalue = int.Parse(this.takeWater.Text)(user input = 1) + takewatervalue(2); // 
    counter = takewatervalue //3 * cupml //300; // = 600 ml -- this is not right, the right value is 600 + 400(the old counter) = 1000 ML
    textCounter.Text = counter.ToString(); // showing the wrong counter;

有人知道用另一种方式计算吗？

完整的参考代码在这里有一个 SQLITE 和其他东西的数据库，但这里的重点是这个计算：https://github.com/lucasbarbosagit/BebaAguaAPP

Answer 1

在我看来，您计算喝水量的方式存在一些设计问题。 IE。您在不知道实际杯子容量的情况下存储喝过的杯子数量。同时，您可以更改杯子的大小，这实际上就像所有以前喝过的杯子都被更换了一样。

首先，我不会使用 UI 控件的字段作为存储（我会避免使用int.Parse）。

然后，从长远来看，我要么有一个毫升计数器，要么有一个列表 "drink" 事件，每次喝一杯水就会存储一个杯子容量。

因此，想想类似的东西（我还没有编译过）：

public class WaterConsumption
{
    private List<int> _DrunkCups = new List<int>();

    public int CupCapacity { get; set; }

    public void DrinkACupOfWater()
    {
        this._DrunkCups.Add(CupCapacity);
    }

    public int GetTotalWaterDrunk()
    {
        return this.DrunkCups.Sum()
    }
}

那么你的表单中上部字段的内容是GetTotalWaterDrunk()的结果，而你的表单下部字段的内容必须存储在CupCapacity中。

让我知道您是否清楚它是否解决了您的问题。