交换数组中的对象 - 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 上的手表，第 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 属性看起来如此奇怪的原因相同。

最终令人困惑的是，该属性被设计为复制并分配给，就像您通常使用值类型的字段一样（例如int、双等）。也就是说，要更改索引 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 中的索引。 （它使用的标准交换代码类似于我尝试过的以及每个人在回复中提到的内容。）

感谢大家的帮助。