为什么我的数据在我没有改变的时候却在改变？

Question

我正在开发一个 Microsoft Visual C# Form 应用程序，该应用程序将用于创建将进入 RPG 游戏的所有数据。我设计了一个结构并将其全部封装到一个 class 中，这样我就可以轻松地读写 to/from 一个 XML 文件。

在我的主窗体上，我有一个 "public static"/"Global" 变量，我的所有子窗体都可以从中复制信息...操作它需要...并将该信息发回

例如。我想要多种货币。处理货币的表单仅从全局变量复制货币数据，并且可以自由支配仅操纵该副本。只有当用户单击 "Apply" 或 "Accept" 按钮时，全局变量才会更新以反映这些更改。如果单击 "Cancel" 按钮，它应该只是关闭表单，并且在处理表单时复制的数据应该被扔到风中。

不幸的是，情况并非如此。每当我更改副本的数据时，它的更改似乎也会反映在全局变量上。这里有一个概念我在这里失踪并且不明白。有人请解释。我检查了我的代码，只有这两点数据应该更新。

来自 "Main" 表格的代码

public partial class frmMain : Form
{
    public static RPGDataCollection DATA = new RPGDataCollection();
    public static string FILE = "";

    public frmMain ()
    {
        InitializeComponent();
        FillDefaultData();
    }

    /// <summary>
    /// Sets item info status text.
    /// </summary>
    /// <param name="text">Text to be displayed.</param>
    private void SetItemInfo (string text)
    {
        lblItemInfo.Text = text;
    }

货币格式代码

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

using RPGData.ObjectTypes.DataGroups;

namespace RPGData.EntryForms
{
    public partial class frmCurrencies : Form
    {
        #region Fields

        private List<Currency> datCurrencies = new List<Currency>();
        private string strEntry = "";
        private bool blnGSC = false;
        private bool blnUpperFlag = false;
        private int intIndex = 0;

        #endregion

        #region Constructor

        public frmCurrencies ()
        {
            InitializeComponent();
        }

        #endregion

        #region Events

        private void frmCurrencies_Load (object sender, EventArgs e)
        {
            datCurrencies = frmMain.DATA.Currencies;
            DisplayData();
        }

        private void btnReplace_Click (object sender, EventArgs e)
        {
            intIndex = lstCurrencies.SelectedIndex;
            Currency c = datCurrencies[intIndex];

            if (txtEntry.Text.Trim().Length > 0)
            {
                SetValues();

                c.Name = strEntry;
                c.HandlesGSC = blnGSC;
                c.Amount = 0;

                datCurrencies[intIndex] = c;
            }

            ResetFields();
            DisplayData();
        }

        private void btnCancel_Click (object sender, EventArgs e)
        {
            this.Close();
        }

        private void btnAdd_Click (object sender, EventArgs e)
        {
            if (txtEntry.Text.Trim().Length > 0)
            {
                SetValues();

                Currency c = new Currency();
                c.Name = strEntry;
                c.HandlesGSC = blnGSC;
                c.Amount = 0;

                datCurrencies.Add(c);
            }

            ResetFields();
            DisplayData();
        }

        private void btnRemove_Click (object sender, EventArgs e)
        {
            intIndex = lstCurrencies.SelectedIndex;

            if (intIndex >= 0)
                datCurrencies.RemoveAt(intIndex);

            ResetFields();
            DisplayData();
        }

        private void btnApply_Click (object sender, EventArgs e)
        {
            frmMain.DATA.Currencies = datCurrencies;
        }

        private void btnAccept_Click (object sender, EventArgs e)
        {
            frmMain.DATA.Currencies = datCurrencies;
            this.Close();
        }

        private void lstCurrencies_SelectedIndexChanged (object sender, EventArgs e)
        {
            intIndex = lstCurrencies.SelectedIndex;
            Currency c = datCurrencies[intIndex];

            txtEntry.Text = c.Name;
            chkGSC.Checked = c.HandlesGSC;
        }

        #endregion

        #region Methods

        private void DisplayData ()
        {
            lstCurrencies.Items.Clear();

            for (int i = 0; i < datCurrencies.Count; i++)
            {
                string gsc = "";
                if (datCurrencies[i].HandlesGSC)
                    gsc = "*";
                else
                    gsc = " ";

                lstCurrencies.Items.Add("[ " + gsc + " ] " + datCurrencies[i].Name);
            }
        }

        private void ResetFields ()
        {
            strEntry = "";
            blnGSC = false;

            txtEntry.Text = strEntry;
            chkGSC.Checked = blnGSC;

            txtEntry.Focus();
        }

        private void SetValues ()
        {
            string entry = ToAllUpper(txtEntry.Text);

            strEntry = entry;
            blnGSC = chkGSC.Checked;
        }

        private string ToAllUpper (string str)
        {
            string entry = "";

            for (int i = 0; i < str.Length; i++)
            {
                string c = "";

                if (i == 0)
                    c = str.Substring(0, 1).ToUpper();

                else if (str.Substring(i, 1) == " ")
                {
                    c = " ";
                    blnUpperFlag = true;
                }

                else if (blnUpperFlag)
                {
                    c = str.Substring(i, 1).ToUpper();
                    blnUpperFlag = false;
                }
                else
                    c = str.Substring(i, 1);

                entry += c;
            }

            return entry;
        }

        #endregion
    }
}

如果你能给我任何帮助，帮助我理解可能发生的事情，那就太好了（或者你看到了我没有看到的错误或错误）。

谢谢！

Answer 1

您将引用复制到此处的列表：

datCurrencies = frmMain.DATA.Currencies;

完成此分配后，仍然只有一个列表，datCurrencies 指向这个全局列表。

相反，您需要创建一个深拷贝，即将内容从全局列表复制到本地列表。

Answer 2

这行代码 datCurrencies = frmMain.DATA.Currencies 实际上是创建了另一个对 frmMain.DATA.Currencies 的引用并且没有深度复制它。

所以您所做的所有更改 - 实际上是在原始对象上进行的。

您必须克隆它（创建深拷贝），而不仅仅是创建额外的引用。

例如，如果您的 Currency 是结构（值类型），则以下内容就足够了：

datCurrencies = new List<Currency>(frmMain.DATA.Currencies);

但是如果您的 Currency 是 class - 您可以考虑以下方法：在你的 Currency class 中创建 Clone 方法，它将 return 克隆当前对象，然后填充你的 datCurrencies 如：

datCurrencies = new List<Currency>(frmMain.DATA.Currencies.Count);
foreach(var currency in frmMain.DATA.Currencies)
    datCurrencies.Add(currency.Clone());

为什么我的数据在我没有改变的时候却在改变？

Why is my data changing when I am not changing it?

c#

static

winforms

来自 "Main" 表格的代码

货币格式代码