交换数组中的对象 - C#

Question

在 C# 中，我有一个 MenuItem 数组。我正在尝试使用以下代码交换数组的索引 2 和索引 3 中的两个对象，但没有成功：

MenuItem Temp = Items[2];  
Items[2] = Items[3];  
Items[3] = Temp;  

第二行和第三行不能在 C# 中运行肯定是有原因的，我可能还不明白。有人能再澄清一点吗？我是否必须深入并单独交换对象中的每个属性？

已编辑 - 抱歉。看起来我在尝试清理代码以进行发布时弄乱了代码。现已更正。

实际代码是：

MenuItem TempButton = MenuItems.Items[SelectedButton.CountId];  
MenuItems.Items[SelectedButton.CountId] = MenuItems.Items[SelectedButton.CountId + 1];  
MenuItems.Items[SelectedButton.CountId + 1] = TempButton;  

MenuItems.Items 是MenuItem 的数组

查看我放在 MenuItems.Items 上的 Watch，第 2 行或第 3 行没有任何反应。

MenuItems.Items 属性具有 get 和 set 函数，这可能会导致问题...将进一步调查...

Answer 1

您将设置Items[2] 设置为Temp，一开始是Items[2]，因此您实际上什么也没做。我不知道SelectedButton.CountId 应该是什么。

但如果你只想交换索引 2 和 3，你可以这样做：

Item Temp = Items[2];
Items[2] = Items[3];
Items[3] = Temp;

Answer 2

SelectedButton.CountId = 2？如果是这样，我会试试这个：

Item Temp = MenuItems.Items[2];  
MenuItems.Items[SelectedButton.CountId] = MenuItems.Items[3];  
MenuItems.Items[3] = Temp;  

注意最后一行有一个 3。

这样会更清楚：

Item Temp = MenuItems.Items[SelectedButton.CountId];  
MenuItems.Items[SelectedButton.CountId] = MenuItems.Items[3];  
MenuItems.Items[3] = Temp;  

Answer 3

我不知道SelectedButton.CountId 应该是什么，但您将Temp 放回原来的位置。而MenuItems.Items 似乎是与Items 完全不同的集合。

string[] items = { "one", "two", "three" };
string temp = items[1]; // temp = "two"
items[1] = items[2]; // items[1] = "three"
items[2] = temp; // items[2] = "two"

// items is now
// { "one", "three", "two" }

Answer 4

试试：

Item Temp = Items[SelectedButton.CountId];   
Items[SelectedButton.CountId] = MenuItems.Items[SelectedButton.CountId+1];   
Items[SelectedButton.CountId+1] = Temp;  

这应该以泡沫方式交换

Answer 5

我记得前段时间遇到过类似的混淆来源，DataRow.ItemArray 属性。这个属性非常违反直觉，原因与您示例中的 Items 属性看起来很奇怪一样。

最终令人困惑的是，该属性被设计为从复制并分配给，就像您通常使用值类型的字段（如@ 987654324@、double 等）。也就是说，要更改索引 2 处的元素，这是行不通的：

row.ItemArray[2] = "New Value";

上面的代码实际上会将行中的值复制到一个新数组中，获取该副本并将索引 2 处的值设置为“新值”，然后新数组将立即变为超出范围。该属性假定的工作方式是：

object[] items = row.ItemArray;
items[2] = "New Value";
row.ItemArray = items;

非常违反直觉，在我的书中（图书馆开发人员注意：不要这样做）。但听起来这可能是您在代码中看到的问题背后的问题。

换句话说，我认为您（现在）拥有的交换代码是正确的。问题出在谁有聪明的想法让Items 属性表现得好像它是一个值字段。

Answer 6

我遇到了同样的问题，因为我想在 WPF-TreeView 中上下移动元素。由于没有一个答案能解决我的问题，所以这里是我能找到的最好的答案。

    private void MoveLayerUp()
    {
        if(Layers.SelectedItem != null)
        {
            int index = Layers.Items.IndexOf(Layers.SelectedItem);
            if (index > 0)
            {
                var swap = Layers.Items[index - 1];
                Layers.Items.RemoveAt(index - 1);
                Layers.Items.Insert(index, swap);
            }
        }
    }

    private void MoveLayerDown()
    {
        if (Layers.SelectedItem != null)
        {
            int index = Layers.Items.IndexOf(Layers.SelectedItem);
            if (index < Layers.Items.Count-1)
            {
                var swap = Layers.Items[index + 1];
                Layers.Items.RemoveAt(index + 1);
                Layers.Items.Insert(index, swap);
            }
        }
    }

这解决了在集合中分配元素的问题。此外，它还有一个优点是当前选定的项目永远不会被触摸并保持选中状态。

Answer 7

解决了这个问题。我觉得我把问题复杂化了一点。

MenuItems.Items 是一个带有 get/set 函数的属性，它返回/设置一个私有 ArrayList。

我在 MenuItems 的类中创建了一个函数，它交换了私有 ArrayList 中的索引。 （它使用的标准交换代码类似于我尝试过的以及每个人在回复中提到的。）

感谢大家的帮助。