Question

我正在开发一个Microsoft Visual C＃Form Application，用于创建将进入RPG游戏的所有数据。我已经设计了一个结构并将其全部包含在一个类中，因此我可以轻松地读取和写入XML文件。

在我的主表单上，我有一个＆＃34; public static＆＃34; /＆＃34; Global＆＃34;变量，我的所有子表单都可以复制信息...操纵它需要的内容...并将该信息发送回它。

例如。我想要多种货币。处理货币的表单仅复制全局变量中的货币数据，并且可以自由统治仅操作该副本。只有当用户点击＆＃34;应用＆＃34;或＆＃34;接受＆＃34;按钮应更新全局变量以反映这些更改。如果＆＃34;取消＆＃34;单击按钮它应该只关闭表单，复制的数据应该在表单处理时被抛到风中。

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

代码来自＆＃34; Main＆＃34;表格

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实际上创建了另一个对frmMain.DATA.Currencies的引用，并没有深度复制它。

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

您必须克隆它（创建深层副本），而不仅仅是创建其他参考。

例如，如果您的Currency是struct（value-type），那么以下就足够了：

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

但如果您的Currency是上课 - 您可以考虑采用以下方法：在Clone类中创建将返回当前对象克隆的方法，然后填充Currency，如：

datCurrencies

Answer 2

您可以将参考复制到此处的列表中：

datCurrencies = frmMain.DATA.Currencies;

Afetr this asignment仍然只有一个列表，datCurrencies指向这一个全局列表。

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

当我不更改数据时，为什么我的数据会发生变化？

代码来自＆＃34; Main＆＃34;表格

货币表格代码

2 个答案: