c#使用动作创建一个.forEach()方法[重复]

Question

这个问题在这里已经有了答案：

为什么分配给参数不会改变对象？ (4 个回答)

5 天前关闭。

我正在尝试创建一个循环遍历数组并执行我用 lambda 表达式编写的一些操作的方法。 不知何故，我不认为我理解如何正确使用动作，因为我的代码只会产生一堆“空”对象。

有人知道我做错了什么吗？

        public void forEach(Action<Neuron> action)
        {
            for(int i = 0; i < this.Neurons.Length; i++)
            {                
                action(this.Neurons[i]);
            }            
        }

        public void CreateNeurons(int AmountOfNeurons)
        {
            this.Neurons = new Neuron[AmountOfNeurons];
            this.forEach(x => x = new Neuron(AmountOfNeurons));
        }

Answer 1

在您的委托中，设置x = ...只会更改该本地参数的值：它对数组本身没有任何作用。 如果你使用这样的for循环，你最终会遇到同样的问题：

var a = new[] { 1, 2, 3};

for (int i = 0; i < a.Length; i++)
{
    var x = a[i];
    x = 5; // this does nothing to `a[i]`
}

值得注意的是，如果您尝试这样做， foreach循环会警告您：

foreach (var x in a)
{
    x = 5; // CS1656: Cannot assign to 'x' because it is a foreach variable.
}

这支持 Dai 的评论：您真的不应该定义自己的forEach方法。 有很多更好、更惯用的方法可以用这样的方法来做任何你想做的事情。 例如，一个foreach结构会准确地告诉你你做错了什么。 使用 LINQ 创建包含一堆项目的数组通常更简单：

this.Neurons = Enumerable.Range(0, AmountOfNeurons)
    .Select(_ => new Neuron(AmountOfNeurons))
    .ToArray();